Squish more warnings in xstat
authorSimon Wilkinson <sxw@inf.ed.ac.uk>
Wed, 22 Jul 2009 21:07:32 +0000 (22:07 +0100)
committerDerrick Brashear <shadow@dementia.org>
Thu, 23 Jul 2009 17:35:00 +0000 (10:35 -0700)
Use a correct prototype for a_ProbeHandler in xstat_cm_Init
Fix a printf warning
rx_getAllAddr and afs_cb_interface.addr disagree on the sign of addresses
afs_interface.addr is a [], which we then use & to get the address of. Fix
this to stop the compiler from sulking.

Reviewed-on: http://gerrit.openafs.org/http://gerrit.openafs.org/184
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>

src/xstat/xstat_cm.c
src/xstat/xstat_fs_callback.c

index 2aadde3..6d4b7b0 100644 (file)
@@ -379,7 +379,7 @@ xstat_cm_LWP(void *unused)
 
 int
 xstat_cm_Init(int a_numServers, struct sockaddr_in *a_socketArray,
-             int a_ProbeFreqInSecs, int (*a_ProbeHandler) (), int a_flags,
+             int a_ProbeFreqInSecs, int (*a_ProbeHandler) (void), int a_flags,
              int a_numCollections, afs_int32 * a_collIDP)
 {
 
@@ -422,7 +422,7 @@ xstat_cm_Init(int a_numServers, struct sockaddr_in *a_socketArray,
                a_ProbeFreqInSecs);
        arg_errfound = 1;
     }
-    if (a_ProbeHandler == (int (*)())0) {
+    if (a_ProbeHandler == NULL) {
        fprintf(stderr, "[%s] Null probe handler function argument\n", rn);
        arg_errfound = 1;
     }
@@ -474,7 +474,7 @@ xstat_cm_Init(int a_numServers, struct sockaddr_in *a_socketArray,
        malloc(a_numServers * sizeof(struct xstat_cm_ConnectionInfo));
     if (xstat_cm_ConnInfo == (struct xstat_cm_ConnectionInfo *)0) {
        fprintf(stderr,
-               "[%s] Can't allocate %d connection info structs (%d bytes)\n",
+               "[%s] Can't allocate %d connection info structs (%lu bytes)\n",
                rn, a_numServers,
                (a_numServers * sizeof(struct xstat_cm_ConnectionInfo)));
        return (-1);            /*No cleanup needs to be done yet */
index ee4bab0..71778a7 100644 (file)
@@ -55,7 +55,7 @@ init_afs_cb(void)
 #else
     afs_uuid_create(&afs_cb_interface.uuid);
 #endif
-    count = rx_getAllAddr(&afs_cb_interface.addr_in, AFS_MAX_INTERFACE_ADDR);
+    count = rx_getAllAddr((afs_uint32 *)afs_cb_interface.addr_in, AFS_MAX_INTERFACE_ADDR);
     if (count <= 0)
        afs_cb_interface.numberOfInterfaces = 0;
     else