]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
salvager: Clear summary in RecordHeader
authorAndrew Deason <adeason@sinenomine.net>
Tue, 21 Jun 2011 23:08:21 +0000 (18:08 -0500)
committerDerrick Brashear <shadow@dementia.org>
Sat, 9 Jul 2011 05:07:25 +0000 (22:07 -0700)
Not every field in the summary header in RecordHeader is set, leaving
some used uninitialized when we copy to the given volumeSummaryp (like
'deleted'). Zero out the header before we do anything.

Reviewed-on: http://gerrit.openafs.org/4876
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
(cherry picked from commit 926ce3d35018050bdbe8e00011884954d6b0fd80)

Change-Id: Ia617153da8762394659b7c7816e6f3d24af6c4d2
Reviewed-on: http://gerrit.openafs.org/4944
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
src/vol/vol-salvage.c

index 0e31f43834e4149d19ab38531a7cadf0a6a803b9..7f99998cbec66a0ec82195f2dda9c59a0dd709c4 100644 (file)
@@ -1542,6 +1542,8 @@ RecordHeader(struct DiskPartition64 *dp, const char *name,
 
     params = (struct SalvageScanParams *)rock;
 
+    memset(&summary, 0, sizeof(summary));
+
     singleVolumeNumber = params->singleVolumeNumber;
     salvinfo = params->salvinfo;