Changeset 71431 in vbox for trunk/src/VBox/HostServices/GuestControl

Timestamp:

Mar 21, 2018 1:30:12 PM (7 years ago)

Author:

vboxsync

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

121399

Message:

Guest Control/service: Partly reverted r121279 (also let the host know that the current message has been skipped.), not needed.

File:

• trunk/src/VBox/HostServices/GuestControl/service.cpp (modified) (2 diffs)

trunk/src/VBox/HostServices/GuestControl/service.cpp

@@ -1278 +1278 @@
  * from the internal command list.
  *
- * This also tells the host callback(s) waiting for a reply for this command that this command
- * has been skipped by returning VERR_NOT_SUPPORTED to the host.
- *
  * @return VBox status code.
  * @param  u32ClientID                  The client's ID.
@@ -1302 +1299 @@
     if (itClientState != mClientStateMap.end())
     {
-        const HostCommand *pCurCmd = itClientState->second.GetCurrent();
-        if (pCurCmd)
-        {
-            /* Tell the host that the guest did not handle the current command. */
-            uint32_t cHstParms = 0;
-            VBOXHGCMSVCPARM aHstParms[4];
-            aHstParms[cHstParms++].setUInt32(pCurCmd->mContextID);
-            aHstParms[cHstParms++].setUInt32(0); /* Notification type (None / generic). */
-            aHstParms[cHstParms++].setUInt32((uint32_t)VERR_NOT_SUPPORTED); /** @todo int vs. uint32_t! */
-            aHstParms[cHstParms++].setPointer(NULL, 0); /* Payload (none). */
-
-            itClientState->second.DequeueCurrent();
-
-            rc = hostCallback(GUEST_MSG_REPLY, cHstParms, aHstParms);
-        }
-        else
-            rc = VERR_NOT_FOUND; /* Should never happen. */
+        itClientState->second.DequeueCurrent();
+        rc = VINF_SUCCESS;
     }
     else
-        rc = VERR_NOT_FOUND; /* Ditto. */
+        rc = VERR_NOT_FOUND;
 
     LogFlowFunc(("[Client %RU32] Skipped current message, rc=%Rrc\n", u32ClientID, rc));