From: Simon Wilkinson <sxw@your-file-system.com>
Date: Tue, 26 Feb 2013 12:30:00 +0000 (+0000)
Subject: ptserver: Zero ubik header before writing to it
X-Git-Tag: openafs-stable-1_8_0pre1~1407
X-Git-Url: https://git.openafs.org/?p=openafs.git;a=commitdiff_plain;h=96131593b329a319b22368a9ef37788fb4e5d4f4

ptserver: Zero ubik header before writing to it

When using pt_util's ubik shim, if we're creating a new ubik label
make sure to zero the whole structure before writing it out to disk.
Otherwise we get a shorts worth of stack garbage in the resulting
file.

Caught by coverity (#986010)

Change-Id: I0abb8b7968d4f25c97dc0927d63dd01099b829b7
Reviewed-on: http://gerrit.openafs.org/9274
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Reviewed-by: Chas Williams - CONTRACTOR <chas@cmf.nrl.navy.mil>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
---

diff --git a/src/ptserver/ptubik.c b/src/ptserver/ptubik.c
index 6ca0eef..f059d2c 100644
--- a/src/ptserver/ptubik.c
+++ b/src/ptserver/ptubik.c
@@ -33,6 +33,7 @@ ubik_BeginTrans(struct ubik_dbase *dbase, afs_int32 transMode,
     struct ubik_hdr thdr;
 
     if (!init) {
+	memset(&thdr, 0, sizeof(thdr));
 	thdr.version.epoch = htonl(2);
 	thdr.version.counter = htonl(0);
 	thdr.magic = htonl(UBIK_MAGIC);