From c58ae34f72d25434cd59228806ca59a6ff39a903 Mon Sep 17 00:00:00 2001 From: Jeffrey Altman Date: Mon, 15 Aug 2011 00:34:00 -0400 Subject: [PATCH] Windows: cm_ShutdownSCache corrections Obtain cm_scache.dirlock, cm_scache.rw and cm_scacheLock in the correct order. Do not release cm_scache.rw when it is not held. Since the cm_scacheLock is being dropped, preserve the value of scp->allNextp prior to dropping the lock. Change-Id: I025a8d76f3f7b94ae00bfd4e000750a90d38b343 Reviewed-on: http://gerrit.openafs.org/5279 Reviewed-by: Jeffrey Altman Tested-by: Jeffrey Altman --- src/WINNT/afsd/cm_scache.c | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/src/WINNT/afsd/cm_scache.c b/src/WINNT/afsd/cm_scache.c index 081cada94..42cd5c76f 100644 --- a/src/WINNT/afsd/cm_scache.c +++ b/src/WINNT/afsd/cm_scache.c @@ -583,20 +583,24 @@ cm_SuspendSCache(void) long cm_ShutdownSCache(void) { - cm_scache_t * scp; + cm_scache_t * scp, * nextp; cm_GiveUpAllCallbacksAllServersMulti(FALSE); lock_ObtainWrite(&cm_scacheLock); for ( scp = cm_data.allSCachesp; scp; - scp = scp->allNextp ) { + scp = nextp ) { + nextp = scp->allNextp; + lock_ReleaseWrite(&cm_scacheLock); +#ifdef USE_BPLUS + lock_ObtainWrite(&scp->dirlock); +#endif + lock_ObtainWrite(&scp->rw); + lock_ObtainWrite(&cm_scacheLock); + if (scp->randomACLp) { - lock_ReleaseWrite(&cm_scacheLock); - lock_ObtainWrite(&scp->rw); - lock_ObtainWrite(&cm_scacheLock); cm_FreeAllACLEnts(scp); - lock_ReleaseWrite(&scp->rw); } if (scp->cbServerp) { @@ -612,6 +616,7 @@ cm_ShutdownSCache(void) freeBtree(scp->dirBplus); scp->dirBplus = NULL; scp->dirDataVersion = CM_SCACHE_VERSION_BAD; + lock_ReleaseWrite(&scp->dirlock); lock_FinalizeRWLock(&scp->dirlock); #endif lock_FinalizeRWLock(&scp->rw); -- 2.39.5