X-Git-Url: https://git.openafs.org/?p=openafs.git;a=blobdiff_plain;f=src%2FWINNT%2Fafsd%2Fcm_callback.c;h=950a2dbe70ef7015eb69ce460b915966b45b7ca8;hp=bded4b5ac9cc368e4facfacb369713cbcc97ef1f;hb=7c01e673176e9de75620a9765aacc25324ce36a4;hpb=f2b18e7af85093f88b8afb8d7a344fd15fc90164

diff --git a/src/WINNT/afsd/cm_callback.c b/src/WINNT/afsd/cm_callback.c
index bded4b5..950a2db 100644
--- a/src/WINNT/afsd/cm_callback.c
+++ b/src/WINNT/afsd/cm_callback.c
@@ -90,33 +90,34 @@ void cm_RecordRacingRevoke(cm_fid_t *fidp, long cancelFlags)
 
 /*
  * When we lose a callback, may have to send change notification replies.
+ * Do not call with a lock on the scp.
  */
 void cm_CallbackNotifyChange(cm_scache_t *scp)
 {
     osi_Log2(afsd_logp, "CallbackNotifyChange FileType %d Flags %lX",
               scp->fileType, scp->flags);
 
-	if (scp->fileType == CM_SCACHETYPE_DIRECTORY) {
-		if (scp->flags & CM_SCACHEFLAG_ANYWATCH)
-			smb_NotifyChange(0,
-			 FILE_NOTIFY_GENERIC_DIRECTORY_FILTER,
-			 scp, NULL, NULL, TRUE);
-	} else {
-		cm_fid_t tfid;
-		cm_scache_t *dscp;
-
-		tfid.cell = scp->fid.cell;
-		tfid.volume = scp->fid.volume;
-		tfid.vnode = scp->parentVnode;
-		tfid.unique = scp->parentUnique;
-		dscp = cm_FindSCache(&tfid);
-		if (dscp &&
-			dscp->flags & CM_SCACHEFLAG_ANYWATCH)
-			smb_NotifyChange(0,
-			 FILE_NOTIFY_GENERIC_FILE_FILTER,
-			 dscp, NULL, NULL, TRUE);
-		if (dscp) cm_ReleaseSCache(dscp);
-	}
+    if (scp->fileType == CM_SCACHETYPE_DIRECTORY) {
+        if (scp->flags & CM_SCACHEFLAG_ANYWATCH)
+            smb_NotifyChange(0,
+                             FILE_NOTIFY_GENERIC_DIRECTORY_FILTER,
+                             scp, NULL, NULL, TRUE);
+    } else {
+        cm_fid_t tfid;
+        cm_scache_t *dscp;
+
+        tfid.cell = scp->fid.cell;
+        tfid.volume = scp->fid.volume;
+        tfid.vnode = scp->parentVnode;
+        tfid.unique = scp->parentUnique;
+        dscp = cm_FindSCache(&tfid);
+        if (dscp &&
+             dscp->flags & CM_SCACHEFLAG_ANYWATCH)
+            smb_NotifyChange( 0,
+                              FILE_NOTIFY_GENERIC_FILE_FILTER,
+                              dscp,   NULL, NULL, TRUE);
+        if (dscp) cm_ReleaseSCache(dscp);
+    }
 }
 
 /* called with no locks held for every file ID that is revoked directly by
@@ -906,13 +907,13 @@ void cm_CheckCBExpiration(void)
         for(scp = cm_hashTablep[i]; scp; scp=scp->nextp) {
             scp->refCount++;
             lock_ReleaseWrite(&cm_scacheLock);
-            lock_ObtainMutex(&scp->mx);
             if (scp->cbExpires > 0 && (scp->cbServerp == NULL || now > scp->cbExpires)) {
                 osi_Log1(afsd_logp, "Callback Expiration Discarding SCache scp %x", scp);
                 cm_CallbackNotifyChange(scp);
+                lock_ObtainMutex(&scp->mx);
                 cm_DiscardSCache(scp);
+                lock_ReleaseMutex(&scp->mx);
             }
-            lock_ReleaseMutex(&scp->mx);
             lock_ObtainWrite(&cm_scacheLock);
             osi_assert(scp->refCount-- > 0);
         }