vlserver: bulkaddrs are unsigned ints
authorSimon Wilkinson <sxw@your-file-system.com>
Tue, 12 Feb 2013 13:15:16 +0000 (13:15 +0000)
committerDerrick Brashear <shadow@your-file-system.com>
Wed, 20 Feb 2013 20:30:14 +0000 (12:30 -0800)
bulkaddrs_val is a pointer to an array of unsigned ints, not to
an array of ints. Fix the sizeof() used in the call to malloc to silence
a clang warning.

Change-Id: If39f34756984200fed1cd3f3de5833012b86af21
Reviewed-on: http://gerrit.openafs.org/9149
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>

src/vlserver/vlprocs.c

index ec0329c..715e2a0 100644 (file)
@@ -2112,7 +2112,7 @@ SVL_GetAddrs(struct rx_call *rxcall,
 
     VLog(5, ("GetAddrs\n"));
     addrsp->bulkaddrs_val = taddrp =
-       malloc(sizeof(afs_int32) * (MAXSERVERID + 1));
+       malloc(sizeof(afs_uint32) * (MAXSERVERID + 1));
     nservers = *nentries = addrsp->bulkaddrs_len = 0;
 
     if (!taddrp) {
@@ -2595,7 +2595,7 @@ SVL_GetAddrsU(struct rx_call *rxcall,
        goto abort;
     }
     addrsp->bulkaddrs_val = taddrp =
-       malloc(sizeof(afs_int32) * (MAXSERVERID + 1));
+       malloc(sizeof(afs_uint32) * (MAXSERVERID + 1));
     nservers = *nentries = addrsp->bulkaddrs_len = 0;
     if (!taddrp) {
        code = VL_NOMEM;