Changeset 23841 in vbox for trunk/src/VBox/Main/ConsoleImplTeleporter.cpp

Timestamp:

Oct 17, 2009 7:07:43 PM (15 years ago)

Author:

vboxsync

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

53599

Message:

Main: Fixed assertion and double error message when cancelling a teleporter waiting for incoming connections.

File:

• trunk/src/VBox/Main/ConsoleImplTeleporter.cpp (modified) (3 diffs)

trunk/src/VBox/Main/ConsoleImplTeleporter.cpp

@@ -816 +816 @@
  *                              Running (false) state,
  * @param   pProgress           Pointer to the progress object.
+ *
+ * @remarks The caller expects error information to be set on failure.
+ * @todo    Check that all the possible failure paths sets error info...
  */
 int
@@ -918 +921 @@
                 else if (vrc == VERR_TCP_SERVER_SHUTDOWN)
                 {
-                    /** @todo this crap isn't work right wrt error info. Aaaarrrg! */
                     BOOL fCancelled = TRUE;
                     hrc = pProgress->COMGETTER(Canceled)(&fCancelled);
@@ -931 +933 @@
                         vrc = VERR_TIMEOUT;
                     }
-                    pProgress->setResultCode(E_FAIL); /* ugly! */
                     LogRel(("Teleporter: RTTcpServerListen aborted - %Rrc\n", vrc));
                     fPowerOff = true;