const char paths for ubik_ServerInit
authorSimon Wilkinson <sxw@inf.ed.ac.uk>
Wed, 11 Nov 2009 10:28:29 +0000 (10:28 +0000)
committerDerrick Brashear <shadow|account-1000005@unknown>
Wed, 11 Nov 2009 17:17:25 +0000 (09:17 -0800)
ubik_ServerInit* take a pathname, which should really be a const.
It already is in many of the callers, some of which remove the
const by casting, the others throw errors.

Make pathName const for all of ubik_ServerInitByInfo, ubik_ServerInit
and ubik_ServerInitCommon.
Update all of our callers to remove the now unecessary casting.
Remove the now uneccessary warning inhibition on vlserver/vlserver.c

Change-Id: I009103dc49e4473eec65ebd75aab727915c26f3c
Reviewed-on: http://gerrit.openafs.org/811
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>

README.WARNINGS
src/kauth/kaserver.c
src/ptserver/ptserver.c
src/ptserver/ubik.c
src/ubik/ubik.c
src/ubik/ubik.p.h
src/vlserver/Makefile.in

index c151e49..409bdd9 100644 (file)
@@ -80,7 +80,6 @@ uss/uss_vol.c              : all             Format issues
 uss/uss_kauth.c      : all          : Des keys, ubik_Call
 uss/lex.yy.c         : all           : Unused symbols
 viced/fsprobe.c      : all          : ubik_Call
-vlserver/vlserver.c  : all          : GetExtendedCellInfo
 vol/salvager.c       : all           : consts & undefined variable from header
 volser/vos.c         : all          : ubik_Call_New
 volser/vol-dump.c    : format        : afs_sfsize_t
index cab9a76..8b0c80a 100644 (file)
@@ -385,11 +385,11 @@ main(int argc, char *argv[])
     if (servers)
        code =
            ubik_ServerInit(myHost, htons(AFSCONF_KAUTHPORT), serverList,
-                           (char *)dbpath, &KA_dbase);
+                           dbpath, &KA_dbase);
     else
        code =
            ubik_ServerInitByInfo(myHost, htons(AFSCONF_KAUTHPORT), &cellinfo,
-                                 clones, (char *)dbpath, &KA_dbase);
+                                 clones, dbpath, &KA_dbase);
 
     if (code) {
        afs_com_err(whoami, code, "Ubik init failed");
index 2c958a9..803a0d2 100644 (file)
@@ -522,7 +522,7 @@ main(int argc, char **argv)
 
     code =
        ubik_ServerInitByInfo(myHost, htons(AFSCONF_PROTPORT), &info, clones,
-                             (char *)pr_dbaseName, &dbase);
+                             pr_dbaseName, &dbase);
     if (code) {
        afs_com_err(whoami, code, "Ubik init failed");
        PT_EXIT(2);
index 4cec9f6..b5cdf61 100644 (file)
@@ -27,7 +27,7 @@ struct ubik_dbase *dbase;
 
 int
 ubik_ServerInit(afs_int32 myHost, short myPort, afs_int32 serverList[],
-                char *pathName, struct ubik_dbase **dbase)
+                const char *pathName, struct ubik_dbase **dbase)
 {
     return (0);
 }
index 09374f8..f72032f 100644 (file)
@@ -385,7 +385,7 @@ ContactQuorum_DISK_SetVersion(register struct ubik_trans *atrans, int aflags,
 int
 ubik_ServerInitCommon(afs_int32 myHost, short myPort,
                      struct afsconf_cell *info, char clones[],
-                     afs_int32 serverList[], char *pathName,
+                     afs_int32 serverList[], const char *pathName,
                      struct ubik_dbase **dbase)
 {
     register struct ubik_dbase *tdb;
@@ -560,7 +560,7 @@ ubik_ServerInitCommon(afs_int32 myHost, short myPort,
 int
 ubik_ServerInitByInfo(afs_int32 myHost, short myPort,
                      struct afsconf_cell *info, char clones[],
-                     char *pathName, struct ubik_dbase **dbase)
+                     const char *pathName, struct ubik_dbase **dbase)
 {
     afs_int32 code;
 
@@ -575,7 +575,7 @@ ubik_ServerInitByInfo(afs_int32 myHost, short myPort,
  */
 int
 ubik_ServerInit(afs_int32 myHost, short myPort, afs_int32 serverList[],
-               char *pathName, struct ubik_dbase **dbase)
+               const char *pathName, struct ubik_dbase **dbase)
 {
     afs_int32 code;
 
index 32d5ebe..315a80f 100644 (file)
@@ -455,10 +455,11 @@ extern afs_int32 ubik_nBuffers;
 struct afsconf_cell;
 extern int ubik_ServerInitByInfo(afs_int32 myHost, short myPort,
                                 struct afsconf_cell *info, char clones[],
-                                char *pathName, struct ubik_dbase **dbase);
+                                const char *pathName,
+                                struct ubik_dbase **dbase);
 extern int ubik_ServerInit(afs_int32 myHost, short myPort, 
                           afs_int32 serverList[],
-                          char *pathName, struct ubik_dbase **dbase);
+                          const char *pathName, struct ubik_dbase **dbase);
 extern int ubik_BeginTrans(register struct ubik_dbase *dbase,
                           afs_int32 transMode, struct ubik_trans **transPtr);
 extern int ubik_BeginTransReadAny(register struct ubik_dbase *dbase,
index b4c5d70..8f4b189 100644 (file)
@@ -127,7 +127,7 @@ vlserver.h vl_errors.c: vl_errors.et vlserver.p.h
        $(RM) -f vlserver.h vl_errors.c; ${COMPILE_ET} -p ${srcdir} vl_errors -h vlserver
 
 vlserver.o: vlserver.c
-       $(CC) $(CFLAGS) @CFLAGS_NOERROR@ -c $<
+
 system: install
 
 install: vlserver cnvldb vldb_check libvldb.a vl_opcodes.h vlserver.h \