From: Derrick Brashear <shadow@dementia.org>
Date: Wed, 4 Aug 2010 13:00:58 +0000 (-0400)
Subject: discon getparentvcache leak fix
X-Git-Tag: openafs-devel-1_5_76~41
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=85e29294eef44fcc52b0583c70debcae452e829d;p=packages%2Fo%2Fopenafs.git

discon getparentvcache leak fix

don't leak an unref'd vcache pointer if we are returning an error
(EAGAIN). thanks to Simon Wilkinson for catching this.

Change-Id: I40c5aa0fd50e1f2c76b09ba073a7109bca706d8c
Reviewed-on: http://gerrit.openafs.org/2511
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/afs/afs_disconnected.c b/src/afs/afs_disconnected.c
index db244bf3d..46a665c9d 100644
--- a/src/afs/afs_disconnected.c
+++ b/src/afs/afs_disconnected.c
@@ -531,8 +531,10 @@ afs_GetParentVCache(struct vcache *avc, int deleted, struct VenusFid *afid,
     }
 
 end:
-    if (code && *adp)
+    if (code && *adp) {
 	afs_PutVCache(*adp);
+	*adp = NULL;
+    }
     return code;
 }