From d55f70792e98e01b3540cdcfad6751a05bf9e645 Mon Sep 17 00:00:00 2001 From: Andrew Deason Date: Tue, 21 Jun 2011 18:08:21 -0500 Subject: [PATCH] salvager: Clear summary in RecordHeader 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 Tested-by: BuildBot (cherry picked from commit 926ce3d35018050bdbe8e00011884954d6b0fd80) Change-Id: Ia617153da8762394659b7c7816e6f3d24af6c4d2 Reviewed-on: http://gerrit.openafs.org/4944 Reviewed-by: Derrick Brashear Tested-by: Derrick Brashear --- src/vol/vol-salvage.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/vol/vol-salvage.c b/src/vol/vol-salvage.c index 0e31f4383..7f99998cb 100644 --- a/src/vol/vol-salvage.c +++ b/src/vol/vol-salvage.c @@ -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; -- 2.39.5