Changeset 24580 in vbox for trunk/src

Timestamp:

Nov 11, 2009 2:29:17 PM (15 years ago)

Author:

vboxsync

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

54715

Message:

PDMLdr.cpp: Call VMSetError before freeing the filename used in the message formatting.

File:

• trunk/src/VBox/VMM/PDMLdr.cpp (modified) (1 diff)

trunk/src/VBox/VMM/PDMLdr.cpp

@@ -529 +529 @@
         AssertRC(rc2);
     }
+
+    /* Don't consider VERR_PDM_MODULE_NAME_CLASH and VERR_NO_MEMORY above as these are very unlikely. */
+    if (RT_FAILURE(rc))
+        rc = VMSetError(pVM, rc, RT_SRC_POS, N_("Cannot load GC module %s"), pszFilename);
+
     RTMemFree(pModule);
     RTMemTmpFree(pszFile);
-
-    /* Don't consider VERR_PDM_MODULE_NAME_CLASH and VERR_NO_MEMORY above as these are very unlikely. */
-    if (RT_FAILURE(rc))
-        return VMSetError(pVM, rc, RT_SRC_POS, N_("Cannot load GC module %s"), pszFilename);
     return rc;
 }