From 9c7f4490af8582c943f115474df645e825ad4cff Mon Sep 17 00:00:00 2001 From: Stephan Wiesand Date: Fri, 30 May 2014 15:05:28 +0200 Subject: [PATCH] libadmin: Remove redundant memset call 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. Reviewed-on: http://gerrit.openafs.org/11180 Reviewed-by: Jeffrey Altman Tested-by: BuildBot (cherry picked from commit f05abc47938beddb436fd0b9677f7b59fc64a215) Change-Id: I7338e42a45d0871af2ad404191ab622605172ac3 Reviewed-on: http://gerrit.openafs.org/11195 Tested-by: BuildBot Reviewed-by: Perry Ruiter Reviewed-by: Benjamin Kaduk Reviewed-by: Stephan Wiesand --- src/libadmin/vos/afs_vosAdmin.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/libadmin/vos/afs_vosAdmin.c b/src/libadmin/vos/afs_vosAdmin.c index 55978e5fb..bed8c494f 100644 --- a/src/libadmin/vos/afs_vosAdmin.c +++ b/src/libadmin/vos/afs_vosAdmin.c @@ -381,8 +381,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 */ -- 2.39.5