Changeset 14855 in vbox for trunk/src/VBox

Timestamp:

Dec 1, 2008 1:28:58 PM (16 years ago)

Author:

vboxsync

Message:

#3162: Fixed a newly introduced memory leak. Added VDShutdown() to test cases.

Location:

trunk/src/VBox/Devices/Storage

Files:

• VBoxHDD-new.cpp (modified) (1 diff)
• testcase/tstVD-2.cpp (modified) (1 diff)
• testcase/tstVD.cpp (modified) (1 diff)

trunk/src/VBox/Devices/Storage/VBoxHDD-new.cpp

@@ -715 +715 @@
         if (pBackends[i]->hPlugin != NIL_RTLDRMOD)
             RTLdrClose(pBackends[i]->hPlugin);
+
+    RTMemFree(pBackends);
     return VINF_SUCCESS;
 }

trunk/src/VBox/Devices/Storage/testcase/tstVD-2.cpp

@@ -150 +150 @@
     }
 
+    rc = VDShutdown();
+    if (RT_FAILURE(rc))
+    {
+        RTPrintf("tstVD-2: unloading backends failed! rc=%Rrc\n", rc);
+        g_cErrors++;
+    }
     /*
      * Summary

trunk/src/VBox/Devices/Storage/testcase/tstVD.cpp

@@ -1060 +1060 @@
     RTFileDelete("tmpVDRename-s003.vmdk");
 
-    /*
+    rc = VDShutdown();
+    if (RT_FAILURE(rc))
+    {
+        RTPrintf("tstVD: unloading backends failed! rc=%Rrc\n", rc);
+        g_cErrors++;
+    }
+     /*
      * Summary
      */