From bfa668c14730b2335c29abd7b8aa20b8e6df338b Mon Sep 17 00:00:00 2001 From: Benjamin Kaduk Date: Mon, 12 Jan 2015 15:14:48 -0500 Subject: [PATCH] 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 Tested-by: BuildBot --- src/vol/volume.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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); -- 2.39.5