From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Sat, 10 Nov 2007 22:50:44 +0000 (+0000)
Subject: windows-readonly-volume-callbacks-20071110
X-Git-Tag: BP-openafs-windows-kdfs-ifs~362
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=b5935d47fb5b06c4f1c74c3671af4b1a481b8ab3;p=packages%2Fo%2Fopenafs.git

windows-readonly-volume-callbacks-20071110


avoid a deadlock
---

diff --git a/src/WINNT/afsd/cm_volume.c b/src/WINNT/afsd/cm_volume.c
index 3ee4812ee..e2d9d62ba 100644
--- a/src/WINNT/afsd/cm_volume.c
+++ b/src/WINNT/afsd/cm_volume.c
@@ -1517,17 +1517,15 @@ cm_VolumeRenewROCallbacks(void)
 
             cm_InitReq(&req);
 
+            lock_ReleaseRead(&cm_volumeLock);
             if (cm_GetSCache(&fid, &scp, cm_rootUserp, &req) == 0) {
-                lock_ReleaseRead(&cm_volumeLock);
                 lock_ObtainMutex(&scp->mx);
                 cm_GetCallback(scp, cm_rootUserp, &req, 1);
                 lock_ReleaseMutex(&scp->mx);
                 cm_ReleaseSCache(scp);
-                lock_ObtainRead(&cm_volumeLock);
             }
+            lock_ObtainRead(&cm_volumeLock);
         }
     }
     lock_ReleaseRead(&cm_volumeLock);
 }
-
-