Changeset 55337 in vbox for trunk/src/VBox

Timestamp:

Apr 20, 2015 12:37:46 PM (10 years ago)

Author:

vboxsync

Message:

USB: Walk the async URB list only under the critical section or we might end up accessing invalid data. Should hopefully fix a crash seen by a customer

File:

• trunk/src/VBox/Devices/USB/VUSBDevice.cpp (modified) (2 diffs)

trunk/src/VBox/Devices/USB/VUSBDevice.cpp

@@ -1047 +1047 @@
      * Iterate the URBs and cancel them.
      */
+    RTCritSectEnter(&pDev->CritSectAsyncUrbs);
     PVUSBURB pUrb = pDev->pAsyncUrbHead;
     while (pUrb)
@@ -1119 +1120 @@
         }
     }
-
+    RTCritSectLeave(&pDev->CritSectAsyncUrbs);
     return VINF_SUCCESS;
 }