From: Simon Wilkinson Date: Tue, 26 Feb 2013 12:26:36 +0000 (+0000) Subject: ptserver: Rename ubik.c as ptubik.c X-Git-Tag: upstream/1.8.0_pre1^2~1409 X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=daed548f70a2187c679232e4f79d703389419c4e;p=packages%2Fo%2Fopenafs.git ptserver: Rename ubik.c as ptubik.c Coverity gets confused between ubik/ubik.c and ptserver/ubik.c, and produces a load of false positives. Rename the ptserver ubik shim (which is only used by pt_util) in order to reduce this confusion. Change-Id: Ida606fccebcb4edfed706405424ff3ee4858020f Reviewed-on: http://gerrit.openafs.org/9273 Reviewed-by: Chas Williams - CONTRACTOR Tested-by: BuildBot Reviewed-by: Jeffrey Altman --- diff --git a/src/ptserver/Makefile.in b/src/ptserver/Makefile.in index aa0c3a496..3d018e812 100644 --- a/src/ptserver/Makefile.in +++ b/src/ptserver/Makefile.in @@ -205,11 +205,11 @@ testpt: testpt.o libprot.a ${TOP_LIBDIR}/libcmd.a $(LIBS) testpt.o: testpt.c ${INCLS} ${TOP_INCDIR}/afs/cmd.h AFS_component_version_number.c -pt_util: pt_util.o ptutils.o ubik.o utils.o map.o libprot.a $(LIBS) - $(AFS_LDRULE) pt_util.o ptutils.o ubik.o utils.o map.o libprot.a \ +pt_util: pt_util.o ptutils.o ptubik.o utils.o map.o libprot.a $(LIBS) + $(AFS_LDRULE) pt_util.o ptutils.o ptubik.o utils.o map.o libprot.a \ ${TOP_LIBDIR}/libcmd.a $(LIBS) $(LIB_roken) ${XLIBS} -ubik.o: ubik.c ${INCLS} +ptubik.o: ptubik.c ${INCLS} prerror.h: pterror.h ${INSTALL_DATA} $? $@ diff --git a/src/ptserver/NTMakefile b/src/ptserver/NTMakefile index 9e5e49d8c..bc5846e9c 100644 --- a/src/ptserver/NTMakefile +++ b/src/ptserver/NTMakefile @@ -105,7 +105,7 @@ PTCLIENT_EXEOBJS =\ PT_UTIL_EXEOBJS = \ $(OUT)\pt_util.obj \ $(OUT)\ptutils.obj \ - $(OUT)\ubik.obj \ + $(OUT)\ptubik.obj \ $(OUT)\utils.obj \ $(OUT)\map.obj \ $(OUT)\pt_util.res diff --git a/src/ptserver/ptubik.c b/src/ptserver/ptubik.c new file mode 100644 index 000000000..6ca0eef9e --- /dev/null +++ b/src/ptserver/ptubik.c @@ -0,0 +1,182 @@ +/* $Id$ */ + +#include +#include +#include + +#include + +#include +#include +#define UBIK_INTERNALS +#include +#include + +#include "ptint.h" +#include "ptserver.h" + +extern int dbase_fd; +struct ubik_dbase *dbase; + +int +ubik_ServerInit(afs_uint32 myHost, short myPort, afs_uint32 serverList[], + const char *pathName, struct ubik_dbase **dbase) +{ + return (0); +} + +int +ubik_BeginTrans(struct ubik_dbase *dbase, afs_int32 transMode, + struct ubik_trans **transPtr) +{ + static int init = 0; + struct ubik_hdr thdr; + + if (!init) { + thdr.version.epoch = htonl(2); + thdr.version.counter = htonl(0); + thdr.magic = htonl(UBIK_MAGIC); + thdr.size = htons(HDRSIZE); + lseek(dbase_fd, 0, 0); + write(dbase_fd, &thdr, sizeof(thdr)); + fsync(dbase_fd); + init = 1; + } + return (0); +} + +int +ubik_BeginTransReadAny(struct ubik_dbase *dbase, afs_int32 transMode, + struct ubik_trans **transPtr) +{ + return (0); +} + +int +ubik_AbortTrans(struct ubik_trans *transPtr) +{ + return (0); +} + +int +ubik_EndTrans(struct ubik_trans *transPtr) +{ + return (0); +} + +int +ubik_Tell(struct ubik_trans *transPtr, afs_int32 * fileid, + afs_int32 * position) +{ + return (0); +} + +int +ubik_Truncate(struct ubik_trans *transPtr, afs_int32 length) +{ + return (0); +} + +int +ubik_SetLock(struct ubik_trans *atrans, afs_int32 apos, afs_int32 alen, + int atype) +{ + return (0); +} + +int +ubik_WaitVersion(struct ubik_dbase *adatabase, + struct ubik_version *aversion) +{ + return (0); +} + +int +ubik_CheckCache(struct ubik_trans *atrans, ubik_updatecache_func cbf, void *rock) +{ + return (*cbf)(atrans, rock); +} + +void +panic(char *format, ...) +{ + va_list ap; + + va_start(ap, format); + vprintf(format, ap); + va_end(ap); + + abort(); + printf("BACK FROM ABORT\n"); /* shouldn't come back from floating pt exception */ + exit(1); /* never know, though */ +} + +int +ubik_GetVersion(struct ubik_trans *dummy, struct ubik_version *ver) +{ + memset(ver, 0, sizeof(struct ubik_version)); + return (0); +} + + +int +ubik_Seek(struct ubik_trans *tt, afs_int32 afd, afs_int32 pos) +{ + if (lseek(dbase_fd, pos + HDRSIZE, 0) < 0) { + perror("ubik_Seek"); + return (-1); + } + return (0); +} + +int +ubik_Write(struct ubik_trans *tt, void *buf, afs_int32 len) +{ + int status; + + status = write(dbase_fd, buf, len); + if (status < len) { + perror("ubik_Write"); + return (1); + } + return (0); +} + +int +ubik_Read(struct ubik_trans *tt, void *buf, afs_int32 len) +{ + int status; + + status = read(dbase_fd, buf, len); + if (status < 0) { + perror("ubik_Read"); + return (1); + } + if (status < len) + memset((char *)buf + status, 0, len - status); + return (0); +} + + +/* Global declarations from ubik.c */ +afs_int32 ubik_quorum = 0; +struct ubik_dbase *ubik_dbase = 0; +struct ubik_stats ubik_stats; +afs_uint32 ubik_host[UBIK_MAX_INTERFACE_ADDR]; +afs_int32 ubik_epochTime = 0; +afs_int32 urecovery_state = 0; + +struct rx_securityClass *ubik_sc[3]; + + +/* Other declarations */ + +int +afsconf_GetNoAuthFlag(struct afsconf_dir *adir) +{ + return (1); +} + + +char *prdir = "/dev/null"; +struct prheader cheader; diff --git a/src/ptserver/ubik.c b/src/ptserver/ubik.c deleted file mode 100644 index 6ca0eef9e..000000000 --- a/src/ptserver/ubik.c +++ /dev/null @@ -1,182 +0,0 @@ -/* $Id$ */ - -#include -#include -#include - -#include - -#include -#include -#define UBIK_INTERNALS -#include -#include - -#include "ptint.h" -#include "ptserver.h" - -extern int dbase_fd; -struct ubik_dbase *dbase; - -int -ubik_ServerInit(afs_uint32 myHost, short myPort, afs_uint32 serverList[], - const char *pathName, struct ubik_dbase **dbase) -{ - return (0); -} - -int -ubik_BeginTrans(struct ubik_dbase *dbase, afs_int32 transMode, - struct ubik_trans **transPtr) -{ - static int init = 0; - struct ubik_hdr thdr; - - if (!init) { - thdr.version.epoch = htonl(2); - thdr.version.counter = htonl(0); - thdr.magic = htonl(UBIK_MAGIC); - thdr.size = htons(HDRSIZE); - lseek(dbase_fd, 0, 0); - write(dbase_fd, &thdr, sizeof(thdr)); - fsync(dbase_fd); - init = 1; - } - return (0); -} - -int -ubik_BeginTransReadAny(struct ubik_dbase *dbase, afs_int32 transMode, - struct ubik_trans **transPtr) -{ - return (0); -} - -int -ubik_AbortTrans(struct ubik_trans *transPtr) -{ - return (0); -} - -int -ubik_EndTrans(struct ubik_trans *transPtr) -{ - return (0); -} - -int -ubik_Tell(struct ubik_trans *transPtr, afs_int32 * fileid, - afs_int32 * position) -{ - return (0); -} - -int -ubik_Truncate(struct ubik_trans *transPtr, afs_int32 length) -{ - return (0); -} - -int -ubik_SetLock(struct ubik_trans *atrans, afs_int32 apos, afs_int32 alen, - int atype) -{ - return (0); -} - -int -ubik_WaitVersion(struct ubik_dbase *adatabase, - struct ubik_version *aversion) -{ - return (0); -} - -int -ubik_CheckCache(struct ubik_trans *atrans, ubik_updatecache_func cbf, void *rock) -{ - return (*cbf)(atrans, rock); -} - -void -panic(char *format, ...) -{ - va_list ap; - - va_start(ap, format); - vprintf(format, ap); - va_end(ap); - - abort(); - printf("BACK FROM ABORT\n"); /* shouldn't come back from floating pt exception */ - exit(1); /* never know, though */ -} - -int -ubik_GetVersion(struct ubik_trans *dummy, struct ubik_version *ver) -{ - memset(ver, 0, sizeof(struct ubik_version)); - return (0); -} - - -int -ubik_Seek(struct ubik_trans *tt, afs_int32 afd, afs_int32 pos) -{ - if (lseek(dbase_fd, pos + HDRSIZE, 0) < 0) { - perror("ubik_Seek"); - return (-1); - } - return (0); -} - -int -ubik_Write(struct ubik_trans *tt, void *buf, afs_int32 len) -{ - int status; - - status = write(dbase_fd, buf, len); - if (status < len) { - perror("ubik_Write"); - return (1); - } - return (0); -} - -int -ubik_Read(struct ubik_trans *tt, void *buf, afs_int32 len) -{ - int status; - - status = read(dbase_fd, buf, len); - if (status < 0) { - perror("ubik_Read"); - return (1); - } - if (status < len) - memset((char *)buf + status, 0, len - status); - return (0); -} - - -/* Global declarations from ubik.c */ -afs_int32 ubik_quorum = 0; -struct ubik_dbase *ubik_dbase = 0; -struct ubik_stats ubik_stats; -afs_uint32 ubik_host[UBIK_MAX_INTERFACE_ADDR]; -afs_int32 ubik_epochTime = 0; -afs_int32 urecovery_state = 0; - -struct rx_securityClass *ubik_sc[3]; - - -/* Other declarations */ - -int -afsconf_GetNoAuthFlag(struct afsconf_dir *adir) -{ - return (1); -} - - -char *prdir = "/dev/null"; -struct prheader cheader; diff --git a/src/tptserver/Makefile.in b/src/tptserver/Makefile.in index 97910d3d2..5a217c602 100644 --- a/src/tptserver/Makefile.in +++ b/src/tptserver/Makefile.in @@ -136,15 +136,15 @@ testpt: testpt.o $(LIBS_client) testpt.o: ${PTSERVER}/testpt.c ${INCLS} $(AFS_CCRULE) $(PTSERVER)/testpt.c -pt_util: pt_util.o ptutils.o ubik.o utils.o map.o $(LIBS_client) - $(LT_LDRULE_static) pt_util.o ptutils.o ubik.o utils.o map.o \ +pt_util: pt_util.o ptutils.o ptubik.o utils.o map.o $(LIBS_client) + $(LT_LDRULE_static) pt_util.o ptutils.o ptubik.o utils.o map.o \ $(LIBS_client) $(LIB_hcrypto) $(LIB_roken) ${MT_LIBS} ${XLIBS} pt_util.o: ${PTSERVER}/pt_util.c $(AFS_CCRULE) $(PTSERVER)/pt_util.c -ubik.o: ${PTSERVER}/ubik.c ${INCLS} - $(AFS_CCRULE) $(PTSERVER)/ubik.c +ptubik.o: ${PTSERVER}/ptubik.c ${INCLS} + $(AFS_CCRULE) $(PTSERVER)/ptubik.c pterror.h: pterror.c pterror.c: ${PTSERVER}/pterror.et diff --git a/src/tptserver/NTMakefile b/src/tptserver/NTMakefile index b23134424..f53592367 100644 --- a/src/tptserver/NTMakefile +++ b/src/tptserver/NTMakefile @@ -130,7 +130,7 @@ PTCLIENT_EXERES = $(OUT)\ptclient.res PT_UTIL_EXEOBJS = \ $(OUT)\pt_util.obj \ - $(OUT)\ubik.obj + $(OUT)\ptubik.obj $(PT_UTIL_EXEOBJS): $(SRC_DIR)\$$(@B).c $(C2OBJ) -I$(SRC_DIR) $**