Changeset 24088 in vbox for trunk/src/VBox

Timestamp:

Oct 26, 2009 4:08:22 PM (16 years ago)

Author:

vboxsync

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

53965

Message:

Main/Console: better fail than return NULL machine reference, which will usually lead to client crash

File:

• trunk/src/VBox/Main/ConsoleImpl.cpp (modified) (1 diff)

trunk/src/VBox/Main/ConsoleImpl.cpp

@@ -1315 +1315 @@
     /* mMachine is constant during life time, no need to lock */
     mMachine.queryInterfaceTo(aMachine);
+
+    /* callers expect to get a valid reference, better fail than crash them */
+    if (mMachine.isNull())
+        return E_FAIL;
 
     return S_OK;