macos 10.6 warning fixes

due to 32/64 and a newer compiler, macos 10.6 finds more warnings.
these are fixes for some of them.

Reviewed-on: http://gerrit.openafs.org/533
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>

diff --git a/src/butc/dump.c b/src/butc/dump.c
index 60975bce66af04b2abb5f3752889dac3af9e2296..5a1228c2cfbd9068be4ed417bbbfbacf94683405 100644 (file)
--- a/src/butc/dump.c
+++ b/src/butc/dump.c
@@ -299,9 +299,9 @@ dumpVolume(struct tc_dumpDesc * curDump, struct dumpRock * dparamsPtr)
        /* Create and Write the volume header */
        makeVolumeHeader(&hostVolumeHeader, dparamsPtr, fragmentNumber);
        hostVolumeHeader.contd = ((fragmentNumber == 1) ? 0 : TC_VOLCONTD);
-       volumeHeader_hton(&hostVolumeHeader, buffer);
+       volumeHeader_hton(&hostVolumeHeader, (struct volumeHeader *)buffer);
 
-       rc = butm_WriteFileData(tapeInfoPtr, buffer, 1,
+       rc = butm_WriteFileData(tapeInfoPtr, (struct volumeHeader *)buffer, 1,
                                sizeof(hostVolumeHeader));
        if (rc) {
            ErrorLog(1, taskId, rc, tapeInfoPtr->error,
@@ -644,10 +644,10 @@ xbsaDumpVolume(struct tc_dumpDesc * curDump, struct dumpRock * dparamsPtr)
     /* Create and Write the volume header */
     makeVolumeHeader(&hostVolumeHeader, dparamsPtr, 1);
     hostVolumeHeader.contd = 0;
-    volumeHeader_hton(&hostVolumeHeader, buffer);
+    volumeHeader_hton(&hostVolumeHeader, (struct volumeHeader *)buffer);
 
-    rc = xbsa_WriteObjectData(&butxInfo, buffer, sizeof(struct volumeHeader),
-                             &bytesWritten);
+    rc = xbsa_WriteObjectData(&butxInfo, (struct volumeHeader *)buffer,
+                             sizeof(struct volumeHeader), &bytesWritten);
     if (rc != XBSA_SUCCESS) {
        ErrorLog(1, taskId, rc, 0,
                 "Unable to write VolumeHeader data to the server\n");

diff --git a/src/des/make_fp.c b/src/des/make_fp.c
index 74e8b7472792e37f770ce0fd99ef4417df2af36d..211e0f96316269dbb3dffe10249af0276e01f34f 100644 (file)
--- a/src/des/make_fp.c
+++ b/src/des/make_fp.c
@@ -46,7 +46,7 @@ gen(FILE * stream)
 
 #ifdef AFS_DARWIN80_ENV
   for(j = 0;; j++) {
-    fprintf(stream, _darwin_whichstr[j]);
+    fprintf(stream, "%s", _darwin_whichstr[j]);
     if (j == 2)
        break;
 #endif /* AFS_DARWIN80_ENV */

diff --git a/src/des/make_ip.c b/src/des/make_ip.c
index 5d699bbb406ba972d10d2b2311087367e3a30874..2fed60e509d6435d088f8742c2b1b35d83101c9d 100644 (file)
--- a/src/des/make_ip.c
+++ b/src/des/make_ip.c
@@ -40,7 +40,7 @@ gen(FILE * stream)
 
 #ifdef AFS_DARWIN80_ENV
   for(j = 0;; j++) {
-    fprintf(stream, _darwin_whichstr[j]);
+    fprintf(stream, "%s", _darwin_whichstr[j]);
     if (j == 2)
        break;
 #endif /* AFS_DARWIN80_ENV */

diff --git a/src/des/read_pssword.c b/src/des/read_pssword.c
index 18fa9e0ec658699a6207546bbcde2d65148c95c5..de4b43a472b061507c8ac7b30f2c304c1136cc8f 100644 (file)
--- a/src/des/read_pssword.c
+++ b/src/des/read_pssword.c
@@ -250,7 +250,7 @@ des_read_pw_string(char *s, int maxa, char *prompt, int verify)
 #endif
 #endif
     while (!ok) {
-       (void)printf(prompt);
+       (void)printf("%s", prompt);
        (void)fflush(stdout);
 #ifdef CROSSMSDOS
        h19line(s, sizeof(s), 0);

diff --git a/src/kauth/admin_tools.c b/src/kauth/admin_tools.c
index 982c128f96462dced3ce7827f6e452d5c5600aa3..45842649b1708db3e7fc82ddb4ec7d41ea43eb3d 100644 (file)
--- a/src/kauth/admin_tools.c
+++ b/src/kauth/admin_tools.c
@@ -1290,7 +1290,8 @@ static int
 MyBeforeProc(struct cmd_syndesc *as, void *arock)
 {
     struct ktc_encryptionKey key;
-    struct ktc_principal auth_server, auth_token, client;
+    struct ktc_principal auth_server, client;
+    struct ktc_token auth_token;
     char realm[MAXKTCREALMLEN];
 
     struct ktc_token token, *pToken;

diff --git a/src/kauth/ka-forwarder.c b/src/kauth/ka-forwarder.c
index 3bfa4b11dd37ecacc60b1ff4281b3ed23dc4eeac..f894353bc98846c2ce91873a391cd5bbd13e2954 100644 (file)
--- a/src/kauth/ka-forwarder.c
+++ b/src/kauth/ka-forwarder.c
@@ -207,7 +207,8 @@ main(int argc, char **argv)
     for (;;) {
        char buf[BUFFER_SIZE], *bufp, *sendptr;
        struct sockaddr_in from, reply, *to;
-       size_t fromlen, sendlen;
+       size_t sendlen;
+       socklen_t fromlen;
 
        bufp = buf + 8;
        fromlen = sizeof(from);

diff --git a/src/kauth/kaprocs.c b/src/kauth/kaprocs.c
index ec23611638cedd309f4d58723a200cb5280aee68..c2deadc84d98ac1cd698f46477c0fa3eb235379f 100644 (file)
--- a/src/kauth/kaprocs.c
+++ b/src/kauth/kaprocs.c
@@ -299,7 +299,7 @@ init_kaprocs(const char *lclpath, int initFlags)
        return code;
     }
 
-    kaux_opendb(lclpath);      /* aux database stores failure counters */
+    kaux_opendb((char *)lclpath);/* aux database stores failure counters */
     rebuildDatabase = 0;       /* only do this during init */
     kaprocsInited = 1;
     return 0;

diff --git a/src/kauth/kaserver.c b/src/kauth/kaserver.c
index 4606202306341c766aad73c4480bea6d73d73aa3..cce632ea508d4e79809bd712bbeee00431ca5a0e 100644 (file)
--- a/src/kauth/kaserver.c
+++ b/src/kauth/kaserver.c
@@ -384,11 +384,11 @@ main(int argc, char *argv[])
     if (servers)
        code =
            ubik_ServerInit(myHost, htons(AFSCONF_KAUTHPORT), serverList,
-                           dbpath, &KA_dbase);
+                           (char *)dbpath, &KA_dbase);
     else
        code =
            ubik_ServerInitByInfo(myHost, htons(AFSCONF_KAUTHPORT), &cellinfo,
-                                 clones, dbpath, &KA_dbase);
+                                 clones, (char *)dbpath, &KA_dbase);
 
     if (code) {
        afs_com_err(whoami, code, "Ubik init failed");

diff --git a/src/lwp/lwp.c b/src/lwp/lwp.c
index 66eb7446505254d393df87a6b53d744017721322..0fe7514f7a59786822d535eb36052e71786d107c 100644 (file)
--- a/src/lwp/lwp.c
+++ b/src/lwp/lwp.c
@@ -934,9 +934,9 @@ Dispatcher(void)
 #endif
        printf("stackcheck = %u: stack = %u \n", lwp_cpptr->stackcheck,
               *(int *)lwp_cpptr->stack);
-       printf("topstack = 0x%x: stackptr = 0x%x: stacksize = 0x%x\n",
-              (uintptr_t)lwp_cpptr->context.topstack, 
-              (uintptr_t)lwp_cpptr->stack,
+       printf("topstack = 0x%" AFS_PTR_FMT ": stackptr = 0x%" AFS_PTR_FMT ": stacksize = 0x%x\n",
+              (void *)(uintptr_t)lwp_cpptr->context.topstack,
+              (void *)(uintptr_t)lwp_cpptr->stack,
               lwp_cpptr->stacksize);
 
        switch (lwp_overflowAction) {

diff --git a/src/ptserver/ptserver.c b/src/ptserver/ptserver.c
index 4b046e908a0cfec3a8d654d2e9a344cc1e5d5a80..0a764ba4ca1382a551a763236176943f8d2c1e02 100644 (file)
--- a/src/ptserver/ptserver.c
+++ b/src/ptserver/ptserver.c
@@ -522,7 +522,7 @@ main(int argc, char **argv)
 
     code =
        ubik_ServerInitByInfo(myHost, htons(AFSCONF_PROTPORT), &info, clones,
-                             pr_dbaseName, &dbase);
+                             (char *)pr_dbaseName, &dbase);
     if (code) {
        afs_com_err(whoami, code, "Ubik init failed");
        PT_EXIT(2);

diff --git a/src/vol/fssync-debug.c b/src/vol/fssync-debug.c
index 126e5c7c7fa935e8f69f63ec143b402330d2368a..304b93a8ba5894a3fd83c98b7209c52c08541de1 100644 (file)
--- a/src/vol/fssync-debug.c
+++ b/src/vol/fssync-debug.c
@@ -1046,7 +1046,7 @@ VnQuery(struct cmd_syndesc * as, void * rock)
        }
 #endif /* !AFS_DEMAND_ATTACH_FS */
 
-       printf("\twriter          = %u\n", (uintptr_t)v.writer);
+       printf("\twriter          = %"AFS_PTR_FMT"\n", (uintptr_t)v.writer);
        printf("\tvcp             = %p\n", v.vcp);
        printf("\thandle          = %p\n", v.handle);
 

diff --git a/src/vol/salvager.c b/src/vol/salvager.c
index d10130d85b1a795befebd42fc7f167a5f94cf052..5a93a5f894fd65a35275f63be6ef697d53f67dc9 100644 (file)
--- a/src/vol/salvager.c
+++ b/src/vol/salvager.c
@@ -268,7 +268,7 @@ handleit(struct cmd_syndesc *as, void *arock)
     }
 
     if ((ti = as->parms[18].items)) {  /* -datelogs */
-       TimeStampLogFile(AFSDIR_SERVER_SLVGLOG_FILEPATH);
+      TimeStampLogFile((char *)AFSDIR_SERVER_SLVGLOG_FILEPATH);
     }
 #endif
 
@@ -410,7 +410,7 @@ main(int argc, char **argv)
         * multiple salvagers appending to the log.
         */
 
-       CheckLogFile(AFSDIR_SERVER_SLVGLOG_FILEPATH);
+       CheckLogFile((char *)AFSDIR_SERVER_SLVGLOG_FILEPATH);
 #ifndef AFS_NT40_ENV
 #ifdef AFS_LINUX20_ENV
        fcntl(fileno(logFile), F_SETFL, O_APPEND);      /* Isn't this redundant? */