Changeset 58036 in vbox for trunk/src/VBox/HostDrivers/VBoxUSB

Timestamp:

Oct 6, 2015 8:13:01 AM (9 years ago)

Author:

vboxsync

Message:

USB/Win: Do not leak memory if request fails.

File:

• trunk/src/VBox/HostDrivers/VBoxUSB/win/dev/VBoxUsbRt.cpp (modified) (1 diff)

trunk/src/VBox/HostDrivers/VBoxUSB/win/dev/VBoxUsbRt.cpp

@@ -831 +831 @@
             {
                 AssertMsgFailed((__FUNCTION__": VBoxUsbToolUrbPost failed Status (0x%x) usb Status (0x%x)\n", Status, pUrb->UrbHeader.Status));
+                /* Free up the allocated memory. */
+                vboxUsbMemFree(pDevExt->Rt.pVBIfaceInfo[InterfaceNumber].pInterfaceInfo);
+                pDevExt->Rt.pVBIfaceInfo[InterfaceNumber].pInterfaceInfo = NULL;
+                if (pDevExt->Rt.pVBIfaceInfo[InterfaceNumber].pPipeInfo)
+                {
+                    vboxUsbMemFree(pDevExt->Rt.pVBIfaceInfo[InterfaceNumber].pPipeInfo);
+                    pDevExt->Rt.pVBIfaceInfo[InterfaceNumber].pPipeInfo = NULL;
+                }
             }
         }