From: Benjamin Kaduk <kaduk@mit.edu>
Date: Mon, 12 Jan 2015 20:14:48 +0000 (-0500)
Subject: Normalize on vp->hashid for hash table usage
X-Git-Tag: upstream/1.8.0_pre1^2~275
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=bfa668c14730b2335c29abd7b8aa20b8e6df338b;p=packages%2Fo%2Fopenafs.git

Normalize on vp->hashid for hash table usage

At present the hashid is set to the same value as the volume ID
(i.e., V_id(vp) a.k.a. vp->header->diskstuff.id), but we should
not leak across the abstraction barrier without cause.

Change-Id: I6a727e60c34bdc938f4ae2e815c7513802a4dbc9
Reviewed-on: http://gerrit.openafs.org/11664
Reviewed-by: Daria Brashear <shadow@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
---

diff --git a/src/vol/volume.c b/src/vol/volume.c
index a7c5d7521..49793a6c1 100644
--- a/src/vol/volume.c
+++ b/src/vol/volume.c
@@ -3490,7 +3490,7 @@ attach2(Error * ec, VolumeId volumeId, char *path, struct DiskPartition64 *partp
 	V_checkoutMode(vp) = mode;
     }
 
-    AddVolumeToHashTable(vp, V_id(vp));
+    AddVolumeToHashTable(vp, vp->hashid);
 #ifdef AFS_DEMAND_ATTACH_FS
     if (VCanUnlockAttached() && (V_attachFlags(vp) & VOL_LOCKED)) {
 	VUnlockVolume(vp);