Changeset 25428 in vbox for trunk/src/VBox/HostDrivers/Support

Timestamp:

Dec 16, 2009 1:59:51 PM (15 years ago)

Author:

vboxsync

Message:

Never call unlock if supdrvLdrLock failed during cleanup.

File:

• trunk/src/VBox/HostDrivers/Support/SUPDrv.c (modified) (2 diffs)

trunk/src/VBox/HostDrivers/Support/SUPDrv.c

@@ -783 +783 @@
      * Loaded images needs to be dereferenced and possibly freed up.
      */
-    supdrvLdrLock(pDevExt);
+    int rcLock = supdrvLdrLock(pDevExt);
+    if (rcLock != VINF_SUCCESS)
+        Log(("supdrvLdrLock failed with rc=%d\n", rcLock));
     Log2(("freeing images:\n"));
     if (pSession->pLdrUsage)
@@ -802 +804 @@
         }
     }
-    supdrvLdrUnlock(pDevExt);
+    if (rcLock == VINF_SUCCESS)
+        supdrvLdrUnlock(pDevExt);
     Log2(("freeing images - done\n"));