Changeset 76680 in vbox for trunk

Timestamp:

Jan 7, 2019 3:41:09 PM (6 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

127987

Message:

USB: Teardown the I/O thread and sync request queue during detach to prevent the I/O thread from accessing the roothub member which was cleared already

File:

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

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

@@ -1236 +1236 @@
         pRh->pDefaultAddress = NULL;
 
+    /*
+     * Destroy I/O thread and request queue last because they might still be used
+     * when cancelling URBs.
+     */
+    vusbDevUrbIoThreadDestroy(pDev);
+
+    int rc = RTReqQueueDestroy(pDev->hReqQueueSync);
+    AssertRC(rc);
+    pDev->hReqQueueSync = NIL_RTREQQUEUE;
+
     pDev->pHub->pOps->pfnDetach(pDev->pHub, pDev);
     pDev->i16Port = -1;
@@ -1267 +1277 @@
         RTCritSectDelete(&pDev->aPipes[i].CritSectCtrl);
     }
-
-    /*
-     * Destroy I/O thread and request queue last because they might still be used
-     * when cancelling URBs.
-     */
-    vusbDevUrbIoThreadDestroy(pDev);
-
-    int rc = RTReqQueueDestroy(pDev->hReqQueueSync);
-    AssertRC(rc);
 
     if (pDev->hSniffer != VUSBSNIFFER_NIL)