Changeset 24708 in vbox for trunk

Timestamp:

Nov 16, 2009 6:18:53 PM (15 years ago)

Author:

vboxsync

Message:

AsyncCompletion: Remove the entry from the endpoint where it belongs to and not the current endpoint. Fixes weird crashes with multiple images

File:

• trunk/src/VBox/VMM/PDMAsyncCompletionFileCache.cpp (modified) (1 diff)

trunk/src/VBox/VMM/PDMAsyncCompletionFileCache.cpp

@@ -312 +312 @@
                     {
                         PPDMACFILECACHEENTRY pFree = pGhostListDst->pTail;
+                        PPDMACFILEENDPOINTCACHE pEndpointCacheFree = &pFree->pEndpoint->DataCache;
+
+                        RTSemRWRequestWrite(pEndpointCacheFree->SemRWEntries, RT_INDEFINITE_WAIT);
+
                         pdmacFileCacheEntryRemoveFromList(pFree);
+
                         STAM_PROFILE_ADV_START(&pCache->StatTreeRemove, Cache);
-                        RTAvlrFileOffsetRemove(pCurr->pEndpoint->DataCache.pTree, pFree->Core.Key);
+                        RTAvlrFileOffsetRemove(pEndpointCacheFree->pTree, pFree->Core.Key);
                         STAM_PROFILE_ADV_STOP(&pCache->StatTreeRemove, Cache);
+
+                        RTSemRWReleaseWrite(pEndpointCacheFree->SemRWEntries);
                         RTMemFree(pFree);
                     }