From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Mon, 7 Apr 2008 07:00:16 +0000 (+0000)
Subject: windows-freelance-two-20080407
X-Git-Tag: openafs-devel-1_5_61~1133
X-Git-Url: https://git.openafs.org/?p=openafs.git;a=commitdiff_plain;h=8f57819a99273812c1d9ba7ec504689b05b83b14

windows-freelance-two-20080407

LICENSE MIT

do not add the scp into the hash table twice.
---

diff --git a/src/WINNT/afsd/cm_scache.c b/src/WINNT/afsd/cm_scache.c
index 70586b4..918b126 100644
--- a/src/WINNT/afsd/cm_scache.c
+++ b/src/WINNT/afsd/cm_scache.c
@@ -724,9 +724,8 @@ long cm_GetSCache(cm_fid_t *fidp, cm_scache_t **outScpp, cm_user_t *userp,
         }
         lock_ReleaseMutex(&cm_Freelance_Lock);
 
-        if (scp == NULL) 
+        if (scp == NULL)
             scp = cm_GetNewSCache();
-
 	if (scp == NULL) {
 	    osi_Log0(afsd_logp,"cm_GetSCache unable to obtain *new* scache entry");
             lock_ReleaseWrite(&cm_scacheLock);
@@ -749,9 +748,11 @@ long cm_GetSCache(cm_fid_t *fidp, cm_scache_t **outScpp, cm_user_t *userp,
         scp->dotdotFid.unique=1;
         scp->dotdotFid.vnode=1;
         scp->flags |= (CM_SCACHEFLAG_PURERO | CM_SCACHEFLAG_RO);
-        scp->nextp=cm_data.scacheHashTablep[hash];
-        cm_data.scacheHashTablep[hash]=scp;
-        scp->flags |= CM_SCACHEFLAG_INHASH;
+        if (!(scp->flags & CM_SCACHEFLAG_INHASH)) {
+            scp->nextp=cm_data.scacheHashTablep[hash];
+            cm_data.scacheHashTablep[hash]=scp;
+            scp->flags |= CM_SCACHEFLAG_INHASH;
+        }
         scp->refCount = 1;
 	osi_Log1(afsd_logp,"cm_GetSCache (freelance) sets refCount to 1 scp 0x%x", scp);
         scp->fileType = fileType;