From: Andrew Deason <adeason@sinenomine.net>
Date: Tue, 21 Jun 2011 23:08:21 +0000 (-0500)
Subject: salvager: Clear summary in RecordHeader
X-Git-Tag: upstream/1.8.0_pre1^2~3608
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=926ce3d35018050bdbe8e00011884954d6b0fd80;p=packages%2Fo%2Fopenafs.git

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.

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

diff --git a/src/vol/vol-salvage.c b/src/vol/vol-salvage.c
index 263117647..0f7f0b91f 100644
--- a/src/vol/vol-salvage.c
+++ b/src/vol/vol-salvage.c
@@ -1546,6 +1546,8 @@ RecordHeader(struct DiskPartition64 *dp, const char *name,
 
     params = (struct SalvageScanParams *)rock;
 
+    memset(&summary, 0, sizeof(summary));
+
     singleVolumeNumber = params->singleVolumeNumber;
     salvinfo = params->salvinfo;