Changeset 3013 in vbox for trunk

Timestamp:

Jun 4, 2007 9:48:38 AM (18 years ago)

Author:

vboxsync

Message:

don't assert on normal errors, crashing VBoxSVC is extremely annoying.

File:

• trunk/src/VBox/Main/HostUSBDeviceImpl.cpp (modified) (1 diff)

trunk/src/VBox/Main/HostUSBDeviceImpl.cpp

@@ -670 +670 @@
         /// @todo we will probably want to re-run all filters on failure
         //  instead of asserting
-        AssertComRC (rc);
+        /*AssertComRC (rc); - actually we shall not assert on fully legal error coditions because it
+                              crashes VBoxSVC and upsets everything. Asserts are for preconditions
+                              violations not for runtime errors like incorrect usbfs permission. */
 
         LogFlowThisFunc (("Done machine->onUSBDeviceAttach()=%08X\n", rc));