Changeset 56770 in vbox

Timestamp:

Jul 3, 2015 12:00:07 PM (10 years ago)

Author:

vboxsync

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

101419

Message:

tstRTR0DbgKrnlInfo: don't leak the stuff and don't panic the host with strict builds.

File:

• trunk/src/VBox/Runtime/testcase/tstRTR0DbgKrnlInfo.cpp (modified) (4 diffs)

trunk/src/VBox/Runtime/testcase/tstRTR0DbgKrnlInfo.cpp

@@ -65 +65 @@
      * The big switch.
      */
+    bool fSavedMayPanic = RTAssertSetMayPanic(false); /* Don't crash the host with strict builds! */
+    RTDBGKRNLINFO hKrnlInfo = NIL_RTDBGKRNLINFO;
     switch (uOperation)
     {
@@ -76 +78 @@
         case TSTRTR0DBGKRNLINFO_BASIC:
         {
-            RTDBGKRNLINFO hKrnlInfo;
             RTR0TESTR0_CHECK_RC_BREAK(RTR0DbgKrnlInfoOpen(&hKrnlInfo, 1), VERR_INVALID_PARAMETER);
             RTR0TESTR0_CHECK_RC_BREAK(RTR0DbgKrnlInfoOpen(NULL, 0), VERR_INVALID_PARAMETER);
@@ -91 +92 @@
             RTR0TESTR0_CHECK_RC_BREAK(RTR0DbgKrnlInfoQuerySymbol(hKrnlInfo, "TestModule", "Test", &pvSymbol), VERR_MODULE_NOT_FOUND);
             RTR0TESTR0_CHECK_RC_BREAK(RTR0DbgKrnlInfoQuerySymbol(hKrnlInfo, NULL, NULL, &pvSymbol), VERR_INVALID_PARAMETER);
+
+            RTDBGKRNLINFO hTmp = hKrnlInfo;
+            hKrnlInfo = NIL_RTDBGKRNLINFO;
+            RTR0DbgKrnlInfoRelease(hKrnlInfo);
+            uint32_t cRefs;
+            RTR0TESTR0_CHECK_MSG((cRefs = RTR0DbgKrnlInfoRelease(hKrnlInfo)) == 0, ("cRefs=%#x", cRefs));
             break;
         }
@@ -101 +108 @@
             break;
     }
+    if (hKrnlInfo != NIL_RTDBGKRNLINFO)
+        RTR0DbgKrnlInfoRelease(hKrnlInfo);
+    RTAssertSetMayPanic(fSavedMayPanic);
 
     /* The error indicator is the '!' in the message buffer. */