Changeset 86402 in vbox for trunk

Timestamp:

Oct 1, 2020 8:26:56 PM (4 years ago)

Author:

vboxsync

Message:

tstVBoxAPIXPCOM: Fix leak in VM listing. Added VBOX_XPCOM_HOME hack to it runs as a testcase. bugref:9841

File:

• trunk/src/VBox/Main/testcase/tstVBoxAPIXPCOM.cpp (modified) (5 diffs)

trunk/src/VBox/Main/testcase/tstVBoxAPIXPCOM.cpp

@@ -105 +105 @@
      */
     IMachine **machines = NULL;
-    PRUint32 machineCnt = 0;
-
-    rc = virtualBox->GetMachines(&machineCnt, &machines);
+    PRUint32 cMachines = 0;
+
+    rc = virtualBox->GetMachines(&cMachines, &machines);
     if (NS_SUCCEEDED(rc))
     {
@@ -113 +113 @@
          * Iterate through the collection
          */
-        for (PRUint32 i = 0; i < machineCnt; ++ i)
+        for (PRUint32 i = 0; i < cMachines; ++ i)
         {
             IMachine *machine = machines[i];
@@ -168 +168 @@
             }
         }
+        nsMemory::Free(machines);
     }
     printf("----------------------------------------------------\n\n");
@@ -433 +434 @@
 ///////////////////////////////////////////////////////////////////////////////
 
-int main()
+int main(int argc, char **argv)
 {
     /*
@@ -450 +451 @@
         return -1;
     }
+
+#if 1 /* Please ignore this! It is very very crude. */
+    char szTmp[8192];
+    if (!getenv("VBOX_XPCOM_HOME"))
+    {
+        strcpy(szTmp, argv[0]);
+        *strrchr(szTmp, '/') = '\0';
+        strcat(szTmp, "/..");
+        fprintf(stderr, "tstVBoxAPIXPCOM: VBOX_XPCOM_HOME is not set, using '%s' instead\n", szTmp);
+        setenv("VBOX_XPCOM_HOME", szTmp, 1);
+    }
+#endif
+    (void)argc; (void)argv;
 
     nsresult rc;