From: Benjamin Kaduk <kaduk@mit.edu>
Date: Fri, 10 Jan 2014 21:27:44 +0000 (-0500)
Subject: Fix vcache/vnode mismatch in lookup for DARWIN
X-Git-Tag: upstream/1.8.0_pre1^2~848
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=67232bd9d6db545cc3f0839c59ecc667a3f9965e;p=packages%2Fo%2Fopenafs.git

Fix vcache/vnode mismatch in lookup for DARWIN

We have a vcache and need a vnode, so AFSTOV() is the right macro,
not VTOAFS().

It's kind of unfortunate that --enable-checking doesn't catch this.

Change-Id: I924649918dd0149b5fd7e4380b5f2e49d21fc8c4
Reviewed-on: http://gerrit.openafs.org/10702
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
---

diff --git a/src/afs/VNOPS/afs_vnop_lookup.c b/src/afs/VNOPS/afs_vnop_lookup.c
index 0faae09de..ee95ac57d 100644
--- a/src/afs/VNOPS/afs_vnop_lookup.c
+++ b/src/afs/VNOPS/afs_vnop_lookup.c
@@ -861,7 +861,7 @@ afs_DoBulkStat(struct vcache *adp, long dirCookie, struct vrequest *areqp)
 		    else
 			tvcp->f.m.Type = VREG;
 		    /* finalize to a best guess */
-		    afs_darwin_finalizevnode(tvcp, VTOAFS(adp), NULL, 0, 1);
+		    afs_darwin_finalizevnode(tvcp, AFSTOV(adp), NULL, 0, 1);
 		    /* re-acquire usecount that finalizevnode disposed of */
 		    vnode_ref(AFSTOV(tvcp));
 #endif