Changeset 75854 in vbox

Timestamp:

Nov 30, 2018 7:36:54 PM (6 years ago)

Author:

vboxsync

Message:

GuestCtrlImpl.cpp: You cannot change uFlags from uint32_t to int32_t because of HGCMSvcGetU32, not legacy/whatever reasons.

File:

• trunk/src/VBox/Main/src-client/GuestCtrlImpl.cpp (modified) (1 diff)

trunk/src/VBox/Main/src-client/GuestCtrlImpl.cpp

@@ -179 +179 @@
             HGCMSvcGetPv(&pSvcCb->mpaParms[4], &dataCb.pvData, &dataCb.cbData);
 
-            if (   (         dataCb.uStatus == PROC_STS_ERROR)
-                   /** @todo Note: Due to legacy reasons we cannot change uFlags to
-                    *              int32_t, so just cast it for now. */
-                && ((int32_t)dataCb.uFlags  == VERR_TOO_MUCH_DATA))
+            if (   dataCb.uStatus == PROC_STS_ERROR
+                && (int32_t)dataCb.uFlags == VERR_TOO_MUCH_DATA)
             {
                 LogFlowFunc(("Requested command with too much data, skipping dispatching ...\n"));