Changeset 12658 in vbox

Timestamp:

Sep 22, 2008 8:37:53 PM (17 years ago)

Author:

vboxsync

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

36950

Message:

Main: Removed too strict assertion in Session::GetRemoteConsole().

File:

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

trunk/src/VBox/Main/SessionImpl.cpp ¶

@@ -270 +270 @@
     AutoReadLock alock (this);
 
-    AssertReturn (mState == SessionState_Open, E_FAIL);
+    AssertReturn (mState != SessionState_Closed, E_FAIL);
 
     AssertMsgReturn (mType == SessionType_Direct && !!mConsole,
                      ("This is not a direct session!\n"), E_FAIL);
+
+    /* return a failure if the session already transitioned to Closing
+     * but the server hasn't processed Machine::OnSessionEnd() yet. */
+    if (mState != SessionState_Open)
+        return E_UNEXPECTED;
 
     mConsole.queryInterfaceTo (aConsole);