From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Sat, 7 Oct 2006 23:29:12 +0000 (+0000)
Subject: DEVEL15-windows-scache-ref-leak-20061007
X-Git-Tag: openafs-devel-1_5_9~33
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=7486e2f4c2e23208f6d4662511410168b89644fc;p=packages%2Fo%2Fopenafs.git

DEVEL15-windows-scache-ref-leak-20061007


return the cm_scache_t whose refCount was increased


(cherry picked from commit feb5f27c754325dcc5fd9eb13f305bc1c0172977)
---

diff --git a/src/WINNT/afsd/cm_scache.c b/src/WINNT/afsd/cm_scache.c
index edfa338ca..50f8ddeac 100644
--- a/src/WINNT/afsd/cm_scache.c
+++ b/src/WINNT/afsd/cm_scache.c
@@ -767,15 +767,15 @@ cm_scache_t * cm_FindSCacheParent(cm_scache_t * scp)
     parent_fid.unique = scp->parentUnique;
 
     if (cm_FidCmp(&scp->fid, &parent_fid)) {
-	for (i=0; i<cm_data.hashTableSize; i++) {
-	    for (pscp = cm_data.hashTablep[i]; pscp; pscp = pscp->nextp) {
-		if (!cm_FidCmp(&pscp->fid, &parent_fid)) {
-		    cm_HoldSCacheNoLock(pscp);
-		    break;
-		}
+	i = CM_SCACHE_HASH(&parent_fid);
+	for (pscp = cm_data.hashTablep[i]; pscp; pscp = pscp->nextp) {
+	    if (!cm_FidCmp(&pscp->fid, &parent_fid)) {
+		cm_HoldSCacheNoLock(pscp);
+		break;
 	    }
 	}
     }
+
     lock_ReleaseRead(&cm_scacheLock);
 
     return pscp;