xstat: Don't cast returns from malloc()
authorSimon Wilkinson <sxw@your-file-system.com>
Thu, 17 May 2012 12:29:42 +0000 (13:29 +0100)
committerDerrick Brashear <shadow@dementix.org>
Thu, 24 May 2012 16:18:58 +0000 (09:18 -0700)
malloc() returns a (void *) on all of our current platforms. So,
don't bother casting the return value before assigning it - it is
unnecessary noise.

Change-Id: Iaddc6c44041dc77a576f32133b2aa80aefce5cb2
Reviewed-on: http://gerrit.openafs.org/7469
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementix.org>

src/xstat/xstat_cm.c
src/xstat/xstat_cm_test.c
src/xstat/xstat_fs.c
src/xstat/xstat_fs_callback.c
src/xstat/xstat_fs_test.c

index c7227d2..31ced2b 100644 (file)
@@ -448,7 +448,7 @@ xstat_cm_Init(int a_numServers, struct sockaddr_in *a_socketArray,
     xstat_cm_ProbeFreqInSecs = a_ProbeFreqInSecs;
     xstat_cm_numCollections = a_numCollections;
     collIDBytes = xstat_cm_numCollections * sizeof(afs_int32);
-    xstat_cm_collIDP = (afs_int32 *) (malloc(collIDBytes));
+    xstat_cm_collIDP = malloc(collIDBytes);
     memcpy(xstat_cm_collIDP, a_collIDP, collIDBytes);
     if (xstat_cm_debug) {
        printf("[%s] Asking for %d collection(s): ", rn,
@@ -470,8 +470,8 @@ xstat_cm_Init(int a_numServers, struct sockaddr_in *a_socketArray,
      * Allocate the necessary data structures and initialize everything
      * else.
      */
-    xstat_cm_ConnInfo = (struct xstat_cm_ConnectionInfo *)
-       malloc(a_numServers * sizeof(struct xstat_cm_ConnectionInfo));
+    xstat_cm_ConnInfo = 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 (%" AFS_SIZET_FMT " bytes)\n",
index 50c4956..71e6fc5 100644 (file)
@@ -1459,8 +1459,7 @@ RunTheTest(struct cmd_syndesc *a_s, void *arock)
     if (debugging_on)
        printf("%s: Allocating socket array for %d Cache Manager(s)\n", rn,
               numCMs);
-    CMSktArray = (struct sockaddr_in *)
-       malloc(numCMs * sizeof(struct sockaddr_in));
+    CMSktArray = malloc(numCMs * sizeof(struct sockaddr_in));
     if (CMSktArray == (struct sockaddr_in *)0) {
        printf("%s: Can't allocate socket array for %d Cache Managers\n", rn,
               numCMs);
@@ -1494,7 +1493,7 @@ RunTheTest(struct cmd_syndesc *a_s, void *arock)
      */
     if (debugging_on)
        printf("Allocating %d long(s) for coll ID\n", numCollIDs);
-    collIDP = (afs_int32 *) (malloc(numCollIDs * sizeof(afs_int32)));
+    collIDP = malloc(numCollIDs * sizeof(afs_int32));
     currCollIDP = collIDP;
     curr_item = a_s->parms[P_COLL_IDS].items;
     for (currCollIDIdx = 0; currCollIDIdx < numCollIDs; currCollIDIdx++) {
index e157bb9..e35469e 100644 (file)
@@ -477,7 +477,7 @@ xstat_fs_Init(int a_numServers, struct sockaddr_in *a_socketArray,
     xstat_fs_ProbeFreqInSecs = a_ProbeFreqInSecs;
     xstat_fs_numCollections = a_numCollections;
     collIDBytes = xstat_fs_numCollections * sizeof(afs_int32);
-    xstat_fs_collIDP = (afs_int32 *) (malloc(collIDBytes));
+    xstat_fs_collIDP = malloc(collIDBytes);
     memcpy(xstat_fs_collIDP, a_collIDP, collIDBytes);
     if (xstat_fs_debug) {
        printf("[%s] Asking for %d collection(s): ", rn,
@@ -499,8 +499,8 @@ xstat_fs_Init(int a_numServers, struct sockaddr_in *a_socketArray,
      * Allocate the necessary data structures and initialize everything
      * else.
      */
-    xstat_fs_ConnInfo = (struct xstat_fs_ConnectionInfo *)
-       malloc(a_numServers * sizeof(struct xstat_fs_ConnectionInfo));
+    xstat_fs_ConnInfo = malloc(a_numServers
+                              * sizeof(struct xstat_fs_ConnectionInfo));
     if (xstat_fs_ConnInfo == (struct xstat_fs_ConnectionInfo *)0) {
        fprintf(stderr,
                "[%s] Can't allocate %d connection info structs (%" AFS_SIZET_FMT " bytes)\n",
index 35b6a7c..05c3e4d 100644 (file)
@@ -696,7 +696,7 @@ SRXAFSCB_GetLocalCell(struct rx_call * a_call, char **a_name)
 {
     char *t_name;
 
-    t_name = (char *)malloc(AFSNAMEMAX);
+    t_name = malloc(AFSNAMEMAX);
     if (!t_name)
        return ENOMEM;
     strcpy(t_name, "This is xstat_fs");
index c90fd03..413d15c 100644 (file)
@@ -726,7 +726,7 @@ RunTheTest(struct cmd_syndesc *a_s, void *dummy)
      */
     if (debugging_on)
        printf("Allocating %d long(s) for coll ID\n", numCollIDs);
-    collIDP = (afs_int32 *) (malloc(numCollIDs * sizeof(afs_int32)));
+    collIDP = malloc(numCollIDs * sizeof(afs_int32));
     currCollIDP = collIDP;
     curr_item = a_s->parms[P_COLL_IDS].items;
     for (currCollIDIdx = 0; currCollIDIdx < numCollIDs; currCollIDIdx++) {