libadmin: Remove redundant memset call
authorStephan Wiesand <stephan.wiesand@desy.de>
Fri, 30 May 2014 13:05:28 +0000 (15:05 +0200)
committerD Brashear <shadow@your-file-system.com>
Sat, 31 May 2014 11:25:51 +0000 (07:25 -0400)
Commit bf78bf2c115659b78c34d3bc9d1934bcff21c8cc added initialisation
of the nbulkentries structure to 0, to avoid freeing garbage due to a
goto fail_... before the structure is initialised. As pointed out by
Andrew Deason, there already is an equivalent memset call later in the
code which is now redundant. Remove it.

Change-Id: I236e6de2a79f4f483be314654225bd52316f5a02
Reviewed-on: http://gerrit.openafs.org/11180
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>

src/libadmin/vos/afs_vosAdmin.c

index 12b1164..dfcef7f 100644 (file)
@@ -372,8 +372,6 @@ vos_BackupVolumeCreateMultiple(const void *cellHandle,
        prefix_len = strlen(volumePrefix);
     }
 
-    memset((void *)&arrayEntries, 0, sizeof(arrayEntries));
-
     /*
      * Get a list of all the volumes in the cell
      */