VirtualBox

Changeset 68458 in vbox


Ignore:
Timestamp:
Aug 18, 2017 10:31:10 AM (7 years ago)
Author:
vboxsync
Message:

Introducing macros for initializing the VBoxGuestHGCMCallInfo structure.

Location:
trunk
Files:
11 edited

Legend:

Unmodified
Added
Removed

  • trunk/include/VBox/VBoxGuest2.h

    r62476 r68458  
    8585AssertCompileSize(VBoxGuestHGCMCallInfo, 16);
    8686
     87/**
     88 * Initialize a HGCM header (VBoxGuestHGCMCallInfo).
     89 *
     90 * @param   a_pHdr          The header to initalize.
     91 * @param   a_idClient      The client connection ID to call thru.
     92 * @param   a_idFunction    The function we're calling
     93 * @param   a_cParameters   Number of parameters.
     94 */
     95# define VBGL_HGCM_HDR_INIT(a_pHdr, a_idClient, a_idFunction, a_cParameters) \
     96    do { \
     97        (a_pHdr)->result         = VERR_INTERNAL_ERROR; \
     98        (a_pHdr)->u32ClientID    = (a_idClient); \
     99        (a_pHdr)->u32Function    = (a_idFunction); \
     100        (a_pHdr)->cParms         = (a_cParameters); \
     101    } while (0)
     102
    87103
    88104/**
    … …  
    105121# pragma pack()
    106122
     123/**
     124 * Initialize a HGCM header (VBoxGuestHGCMCallInfo), with timeout (interruptible).
     125 *
     126 * @param   a_pHdr          The header to initalize.
     127 * @param   a_idClient      The client connection ID to call thru.
     128 * @param   a_idFunction    The function we're calling
     129 * @param   a_cParameters   Number of parameters.
     130 * @param   a_cMsTimeout    The timeout in milliseconds.
     131 */
     132# define VBGL_HGCM_HDR_INIT_TIMED(a_pHdr, a_idClient, a_idFunction, a_cParameters, a_cMsTimeout) \
     133    do { \
     134        (a_pHdr)->u32Timeout        = (a_cMsTimeout); \
     135        (a_pHdr)->fInterruptible    = true; \
     136        (a_pHdr)->info.result       = VERR_INTERNAL_ERROR; \
     137        (a_pHdr)->info.u32ClientID  = (a_idClient); \
     138        (a_pHdr)->info.u32Function  = (a_idFunction); \
     139        (a_pHdr)->info.cParms       = (a_cParameters); \
     140    } while (0)
     141
    107142/** @} */
    108143

  • trunk/src/VBox/Additions/WINNT/Graphics/Video/mp/wddm/VBoxMPCr.cpp

    r63062 r68458  
    840840    int rc;
    841841
    842     parms.hdr.result      = VERR_WRONG_ORDER;
    843     parms.hdr.u32ClientID = u32ClientID;
    844     parms.hdr.u32Function = SHCRGL_GUEST_FN_SET_VERSION;
    845     parms.hdr.cParms      = SHCRGL_CPARMS_SET_VERSION;
     842    VBGL_HGCM_HDR_INIT(&parms.hdr, u32ClientID, SHCRGL_GUEST_FN_SET_VERSION, SHCRGL_CPARMS_SET_VERSION);
    846843
    847844    parms.vMajor.type      = VMMDevHGCMParmType_32bit;
    … …  
    870867    int rc;
    871868
    872     parms.hdr.result      = VERR_WRONG_ORDER;
    873     parms.hdr.u32ClientID = u32ClientID;
    874     parms.hdr.u32Function = SHCRGL_GUEST_FN_GET_CAPS_LEGACY;
    875     parms.hdr.cParms      = SHCRGL_CPARMS_GET_CAPS_LEGACY;
     869    VBGL_HGCM_HDR_INIT(&parms.hdr, u32ClientID, SHCRGL_GUEST_FN_GET_CAPS_LEGACY, SHCRGL_CPARMS_GET_CAPS_LEGACY);
    876870
    877871    parms.Caps.type      = VMMDevHGCMParmType_32bit;
    … …  
    910904    int rc;
    911905
    912     parms.hdr.result      = VERR_WRONG_ORDER;
    913     parms.hdr.u32ClientID = u32ClientID;
    914     parms.hdr.u32Function = SHCRGL_GUEST_FN_GET_CAPS_NEW;
    915     parms.hdr.cParms      = SHCRGL_CPARMS_GET_CAPS_NEW;
     906    VBGL_HGCM_HDR_INIT(&parms.hdr, u32ClientID, SHCRGL_GUEST_FN_GET_CAPS_NEW, SHCRGL_CPARMS_GET_CAPS_NEW);
    916907
    917908    parms.Caps.type      = VMMDevHGCMParmType_LinAddr;
    … …  
    948939    int rc;
    949940
    950     parms.hdr.result      = VERR_WRONG_ORDER;
    951     parms.hdr.u32ClientID = u32ClientID;
    952     parms.hdr.u32Function = SHCRGL_GUEST_FN_SET_PID;
    953     parms.hdr.cParms      = SHCRGL_CPARMS_SET_PID;
     941    VBGL_HGCM_HDR_INIT(&parms.hdr, u32ClientID, SHCRGL_GUEST_FN_SET_PID, SHCRGL_CPARMS_SET_PID);
    954942
    955943    parms.u64PID.type     = VMMDevHGCMParmType_64bit;

  • trunk/src/VBox/Additions/common/VBoxGuestLib/VBoxGuestR3LibClipboard.cpp

    r62521 r68458  
    9797    VBoxClipboardGetHostMsg Msg;
    9898
    99     Msg.hdr.result = VERR_WRONG_ORDER;
    100     Msg.hdr.u32ClientID = idClient;
    101     Msg.hdr.u32Function = VBOX_SHARED_CLIPBOARD_FN_GET_HOST_MSG;
    102     Msg.hdr.cParms = 2;
     99    VBGL_HGCM_HDR_INIT(&Msg.hdr, idClient, VBOX_SHARED_CLIPBOARD_FN_GET_HOST_MSG, 2);
    103100    VbglHGCMParmUInt32Set(&Msg.msg, 0);
    104101    VbglHGCMParmUInt32Set(&Msg.formats, 0);
    … …  
    146143    VBoxClipboardReadData Msg;
    147144
    148     Msg.hdr.result = VERR_WRONG_ORDER;
    149     Msg.hdr.u32ClientID = idClient;
    150     Msg.hdr.u32Function = VBOX_SHARED_CLIPBOARD_FN_READ_DATA;
    151     Msg.hdr.cParms = 3;
     145    VBGL_HGCM_HDR_INIT(&Msg.hdr, idClient, VBOX_SHARED_CLIPBOARD_FN_READ_DATA, 3);
    152146    VbglHGCMParmUInt32Set(&Msg.format, fFormat);
    153147    VbglHGCMParmPtrSet(&Msg.ptr, pv, cb);
    … …  
    186180    VBoxClipboardFormats Msg;
    187181
    188     Msg.hdr.result = VERR_WRONG_ORDER;
    189     Msg.hdr.u32ClientID = idClient;
    190     Msg.hdr.u32Function = VBOX_SHARED_CLIPBOARD_FN_FORMATS;
    191     Msg.hdr.cParms = 1;
     182    VBGL_HGCM_HDR_INIT(&Msg.hdr, idClient, VBOX_SHARED_CLIPBOARD_FN_FORMATS, 1);
    192183    VbglHGCMParmUInt32Set(&Msg.formats, fFormats);
    193184
    … …  
    214205{
    215206    VBoxClipboardWriteData Msg;
    216     Msg.hdr.result = VERR_WRONG_ORDER;
    217     Msg.hdr.u32ClientID = idClient;
    218     Msg.hdr.u32Function = VBOX_SHARED_CLIPBOARD_FN_WRITE_DATA;
    219     Msg.hdr.cParms = 2;
     207    VBGL_HGCM_HDR_INIT(&Msg.hdr, idClient, VBOX_SHARED_CLIPBOARD_FN_WRITE_DATA, 2);
    220208    VbglHGCMParmUInt32Set(&Msg.format, fFormat);
    221209    VbglHGCMParmPtrSet(&Msg.ptr, pv, cb);

  • trunk/src/VBox/Additions/common/VBoxGuestLib/VBoxGuestR3LibDragAndDrop.cpp

    r62845 r68458  
    5656*   Forward declarations                                                                                                         *
    5757*********************************************************************************************************************************/
    58 
     58/** @todo r=bird: What's this? publicly accessible symbol that's not in a header? Shouldn't this be a static function perhaps? */
    5959VBGLR3DECL(int) VbglR3DnDHGSendProgress(PVBGLR3GUESTDNDCMDCTX pCtx, uint32_t uStatus, uint8_t uPercent, int rcErr);
    6060
    … …  
    7272    VBOXDNDNEXTMSGMSG Msg;
    7373    RT_ZERO(Msg);
    74     Msg.hdr.result      = VERR_WRONG_ORDER;
    75     Msg.hdr.u32ClientID = pCtx->uClientID;
    76     Msg.hdr.u32Function = GUEST_DND_GET_NEXT_HOST_MSG;
    77     Msg.hdr.cParms      = 3;
    78 
     74    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_GET_NEXT_HOST_MSG, 3);
    7975    Msg.uMsg.SetUInt32(0);
    8076    Msg.cParms.SetUInt32(0);
    … …  
    119115    VBOXDNDHGACTIONMSG Msg;
    120116    RT_ZERO(Msg);
    121     Msg.hdr.result      = VERR_WRONG_ORDER;
    122     Msg.hdr.u32ClientID = pCtx->uClientID;
    123     Msg.hdr.u32Function = uMsg;
    124 
    125117    if (pCtx->uProtocol < 3)
    126118    {
    127         Msg.hdr.cParms = 7;
    128 
     119        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, uMsg, 7);
    129120        Msg.u.v1.uScreenId.SetUInt32(0);
    130121        Msg.u.v1.uX.SetUInt32(0);
    … …  
    137128    else
    138129    {
    139         Msg.hdr.cParms = 8;
    140 
     130        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, uMsg, 8);
    141131        Msg.u.v3.uContext.SetUInt32(0);
    142132        Msg.u.v3.uScreenId.SetUInt32(0);
    … …  
    188178    VBOXDNDHGLEAVEMSG Msg;
    189179    RT_ZERO(Msg);
    190     Msg.hdr.u32ClientID = pCtx->uClientID;
    191     Msg.hdr.u32Function = HOST_DND_HG_EVT_LEAVE;
    192 
    193180    if (pCtx->uProtocol < 3)
    194     {
    195         Msg.hdr.cParms = 0;
    196     }
     181        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_HG_EVT_LEAVE, 0);
    197182    else
    198183    {
    199         Msg.hdr.cParms = 1;
    200 
     184        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_HG_EVT_LEAVE, 1);
    201185        /** @todo Context ID not used yet. */
    202186        Msg.u.v3.uContext.SetUInt32(0);
    … …  
    216200    VBOXDNDHGCANCELMSG Msg;
    217201    RT_ZERO(Msg);
    218     Msg.hdr.result      = VERR_WRONG_ORDER;
    219     Msg.hdr.u32ClientID = pCtx->uClientID;
    220     Msg.hdr.u32Function = HOST_DND_HG_EVT_CANCEL;
    221 
    222202    if (pCtx->uProtocol < 3)
    223     {
    224         Msg.hdr.cParms = 0;
    225     }
     203        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_HG_EVT_CANCEL, 0);
    226204    else
    227205    {
    228         Msg.hdr.cParms = 1;
    229 
     206        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_HG_EVT_CANCEL, 1);
    230207        /** @todo Context ID not used yet. */
    231208        Msg.u.v3.uContext.SetUInt32(0);
    … …  
    253230    VBOXDNDHGSENDDIRMSG Msg;
    254231    RT_ZERO(Msg);
    255     Msg.hdr.result      = VERR_WRONG_ORDER;
    256     Msg.hdr.u32ClientID = pCtx->uClientID;
    257     Msg.hdr.u32Function = HOST_DND_HG_SND_DIR;
    258 
    259232    if (pCtx->uProtocol < 3)
    260233    {
    261         Msg.hdr.cParms = 3;
    262 
     234        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_HG_SND_DIR, 3);
    263235        Msg.u.v1.pvName.SetPtr(pszDirname, cbDirname);
    264236        Msg.u.v1.cbName.SetUInt32(cbDirname);
    … …  
    267239    else
    268240    {
    269         Msg.hdr.cParms = 4;
    270 
     241        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_HG_SND_DIR, 4);
    271242        /** @todo Context ID not used yet. */
    272243        Msg.u.v3.uContext.SetUInt32(0);
    … …  
    321292    VBOXDNDHGSENDFILEDATAMSG Msg;
    322293    RT_ZERO(Msg);
    323     Msg.hdr.result      = VERR_WRONG_ORDER;
    324     Msg.hdr.u32ClientID = pCtx->uClientID;
    325     Msg.hdr.u32Function = HOST_DND_HG_SND_FILE_DATA;
    326 
    327294    if (pCtx->uProtocol <= 1)
    328295    {
     296        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_HG_SND_FILE_DATA, 5);
    329297        Msg.u.v1.pvName.SetPtr(pszFilename, cbFilename);
    330298        Msg.u.v1.cbName.SetUInt32(0);
    … …  
    332300        Msg.u.v1.cbData.SetUInt32(0);
    333301        Msg.u.v1.fMode.SetUInt32(0);
    334 
    335         Msg.hdr.cParms = 5;
    336302    }
    337303    else if (pCtx->uProtocol == 2)
    338304    {
     305        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_HG_SND_FILE_DATA, 3);
    339306        Msg.u.v2.uContext.SetUInt32(0);
    340307        Msg.u.v2.pvData.SetPtr(pvData, cbData);
    341308        Msg.u.v2.cbData.SetUInt32(cbData);
    342 
    343         Msg.hdr.cParms = 3;
    344309    }
    345310    else if (pCtx->uProtocol >= 3)
    346311    {
     312        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_HG_SND_FILE_DATA, 5);
    347313        Msg.u.v3.uContext.SetUInt32(0);
    348314        Msg.u.v3.pvData.SetPtr(pvData, cbData);
    … …  
    350316        Msg.u.v3.pvChecksum.SetPtr(NULL, 0);
    351317        Msg.u.v3.cbChecksum.SetUInt32(0);
    352 
    353         Msg.hdr.cParms = 5;
    354318    }
    355319    else
    … …  
    408372    VBOXDNDHGSENDFILEHDRMSG Msg;
    409373    RT_ZERO(Msg);
    410     Msg.hdr.result      = VERR_WRONG_ORDER;
    411     Msg.hdr.u32ClientID = pCtx->uClientID;
    412     Msg.hdr.u32Function = HOST_DND_HG_SND_FILE_HDR;
    413 
    414374    int rc;
    415 
    416375    if (pCtx->uProtocol <= 1)
    417     {
    418376        rc = VERR_NOT_SUPPORTED;
    419     }
    420377    else
    421378    {
    422         Msg.hdr.cParms = 6;
    423 
     379        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_HG_SND_FILE_HDR, 6);
    424380        Msg.uContext.SetUInt32(0); /** @todo Not used yet. */
    425381        Msg.pvName.SetPtr(pszFilename, cbFilename);
    … …  
    780736    VBOXDNDHGSENDDATAMSG Msg;
    781737    RT_ZERO(Msg);
    782     Msg.hdr.result      = VERR_WRONG_ORDER;
    783     Msg.hdr.u32ClientID = pCtx->uClientID;
    784     Msg.hdr.u32Function = HOST_DND_HG_SND_DATA;
    785 
    786738    int rc;
    787739    if (pCtx->uProtocol < 3)
    788740    {
    789         Msg.hdr.cParms  = 5;
    790 
     741        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_HG_SND_DATA, 5);
    791742        Msg.u.v1.uScreenId.SetUInt32(0);
    792743        Msg.u.v1.pvFormat.SetPtr(pDataHdr->pvMetaFmt, pDataHdr->cbMetaFmt);
    … …  
    849800    else /* Protocol v3 and up. */
    850801    {
    851         Msg.hdr.cParms = 5;
    852 
     802        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_HG_SND_DATA, 5);
    853803        Msg.u.v3.uContext.SetUInt32(0);
    854804        Msg.u.v3.pvData.SetPtr(pvData, cbData);
    … …  
    893843    VBOXDNDHGSENDDATAHDRMSG Msg;
    894844    RT_ZERO(Msg);
    895     Msg.hdr.result      = VERR_WRONG_ORDER;
    896     Msg.hdr.u32ClientID = pCtx->uClientID;
    897     Msg.hdr.u32Function = HOST_DND_HG_SND_DATA_HDR;
    898     Msg.hdr.cParms      = 12;
    899 
     845    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_HG_SND_DATA_HDR, 12);
    900846    Msg.uContext.SetUInt32(0);
    901847    Msg.uFlags.SetUInt32(0);
    … …  
    944890    VBOXDNDHGSENDMOREDATAMSG Msg;
    945891    RT_ZERO(Msg);
    946     Msg.hdr.result      = VERR_WRONG_ORDER;
    947     Msg.hdr.u32ClientID = pCtx->uClientID;
    948     Msg.hdr.u32Function = HOST_DND_HG_SND_MORE_DATA;
    949     Msg.hdr.cParms      = 2;
    950 
     892    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_HG_SND_MORE_DATA, 2);
    951893    Msg.pvData.SetPtr(pvData, cbData);
    952894    Msg.cbData.SetUInt32(0);
    … …  
    12351177    VBOXDNDGHREQPENDINGMSG Msg;
    12361178    RT_ZERO(Msg);
    1237     Msg.hdr.result      = VERR_WRONG_ORDER;
    1238     Msg.hdr.u32ClientID = pCtx->uClientID;
    1239     Msg.hdr.u32Function = HOST_DND_GH_REQ_PENDING;
    1240 
    12411179    if (pCtx->uProtocol < 3)
    12421180    {
    1243         Msg.hdr.cParms = 1;
    1244 
     1181        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_GH_REQ_PENDING, 1);
    12451182        Msg.u.v1.uScreenId.SetUInt32(0);
    12461183    }
    12471184    else
    12481185    {
    1249         Msg.hdr.cParms = 2;
    1250 
     1186        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_GH_REQ_PENDING, 2);
    12511187        /** @todo Context ID not used yet. */
    12521188        Msg.u.v3.uContext.SetUInt32(0);
    … …  
    12891225    VBOXDNDGHDROPPEDMSG Msg;
    12901226    RT_ZERO(Msg);
    1291     Msg.hdr.result      = VERR_WRONG_ORDER;
    1292     Msg.hdr.u32ClientID = pCtx->uClientID;
    1293     Msg.hdr.u32Function = HOST_DND_GH_EVT_DROPPED;
    1294 
    12951227    if (pCtx->uProtocol < 3)
    12961228    {
    1297         Msg.hdr.cParms = 3;
    1298 
     1229        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_GH_EVT_DROPPED, 3);
    12991230        Msg.u.v1.pvFormat.SetPtr(pszFormat, cbFormat);
    13001231        Msg.u.v1.cbFormat.SetUInt32(0);
    … …  
    13031234    else
    13041235    {
    1305         Msg.hdr.cParms = 4;
    1306 
     1236        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, HOST_DND_GH_EVT_DROPPED, 4);
    13071237        Msg.u.v3.uContext.SetUInt32(0);
    13081238        Msg.u.v3.pvFormat.SetPtr(pszFormat, cbFormat);
    … …  
    14171347        VBOXDNDCONNECTMSG Msg;
    14181348        RT_ZERO(Msg);
    1419         Msg.hdr.result      = VERR_WRONG_ORDER;
    1420         Msg.hdr.u32ClientID = pCtx->uClientID;
    1421         Msg.hdr.u32Function = GUEST_DND_CONNECT;
    1422 
    14231349        if (pCtx->uProtocol < 3)
    14241350        {
    1425             Msg.hdr.cParms = 2;
    1426 
     1351            VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_CONNECT, 2);
    14271352            Msg.u.v2.uProtocol.SetUInt32(pCtx->uProtocol);
    14281353            Msg.u.v2.uFlags.SetUInt32(0); /* Unused at the moment. */
    … …  
    14301355        else
    14311356        {
    1432             Msg.hdr.cParms = 3;
    1433 
     1357            VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_CONNECT, 3);
    14341358            /** @todo Context ID not used yet. */
    14351359            Msg.u.v3.uContext.SetUInt32(0);
    … …  
    16091533    VBOXDNDHGACKOPMSG Msg;
    16101534    RT_ZERO(Msg);
    1611     Msg.hdr.result      = VERR_WRONG_ORDER;
    1612     Msg.hdr.u32ClientID = pCtx->uClientID;
    1613     Msg.hdr.u32Function = GUEST_DND_HG_ACK_OP;
    1614 
    16151535    LogFlowFunc(("uProto=%RU32\n", pCtx->uProtocol));
    1616 
    16171536    if (pCtx->uProtocol < 3)
    16181537    {
    1619         Msg.hdr.cParms = 1;
    1620 
     1538        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_HG_ACK_OP, 1);
    16211539        Msg.u.v1.uAction.SetUInt32(uAction);
    16221540    }
    16231541    else
    16241542    {
    1625         Msg.hdr.cParms = 2;
    1626 
     1543        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_HG_ACK_OP, 2);
    16271544        /** @todo Context ID not used yet. */
    16281545        Msg.u.v3.uContext.SetUInt32(0);
    … …  
    16411558    AssertPtrReturn(pCtx,       VERR_INVALID_POINTER);
    16421559    AssertPtrReturn(pcszFormat, VERR_INVALID_POINTER);
     1560    if (!RTStrIsValidEncoding(pcszFormat))
     1561        return VERR_INVALID_PARAMETER;
     1562
     1563    const uint32_t cbFormat = (uint32_t)strlen(pcszFormat) + 1; /* Include termination */
    16431564
    16441565    VBOXDNDHGREQDATAMSG Msg;
    16451566    RT_ZERO(Msg);
    1646     Msg.hdr.result      = VERR_WRONG_ORDER;
    1647     Msg.hdr.u32ClientID = pCtx->uClientID;
    1648     Msg.hdr.u32Function = GUEST_DND_HG_REQ_DATA;
    1649 
    1650     if (!RTStrIsValidEncoding(pcszFormat))
    1651         return VERR_INVALID_PARAMETER;
    1652 
    1653     const uint32_t cbFormat = (uint32_t)strlen(pcszFormat) + 1; /* Include termination */
    1654 
    16551567    if (pCtx->uProtocol < 3)
    16561568    {
    1657         Msg.hdr.cParms = 1;
    1658 
     1569        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_HG_REQ_DATA, 1);
    16591570        Msg.u.v1.pvFormat.SetPtr((void*)pcszFormat, cbFormat);
    16601571    }
    16611572    else
    16621573    {
    1663         Msg.hdr.cParms = 3;
    1664 
     1574        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_HG_REQ_DATA, 3);
    16651575        /** @todo Context ID not used yet. */
    16661576        Msg.u.v3.uContext.SetUInt32(0);
    … …  
    16831593    VBOXDNDHGEVTPROGRESSMSG Msg;
    16841594    RT_ZERO(Msg);
    1685     Msg.hdr.result      = VERR_WRONG_ORDER;
    1686     Msg.hdr.u32ClientID = pCtx->uClientID;
    1687     Msg.hdr.u32Function = uStatus;
    1688 
    16891595    if (pCtx->uProtocol < 3)
    16901596    {
    1691         Msg.hdr.cParms = 3;
    1692 
     1597        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, uStatus, 3); /** @todo r=bird: Do we really mean to execute 'uStatus' here? */
    16931598        Msg.u.v1.uStatus.SetUInt32(uStatus);
    16941599        Msg.u.v1.uPercent.SetUInt32(uPercent);
    … …  
    16971602    else
    16981603    {
    1699         Msg.hdr.cParms = 4;
    1700 
     1604        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, uStatus, 4); /** @todo r=bird: Do we really mean to execute 'uStatus' here? */
    17011605        /** @todo Context ID not used yet. */
    17021606        Msg.u.v3.uContext.SetUInt32(0);
    … …  
    17141618
    17151619#ifdef VBOX_WITH_DRAG_AND_DROP_GH
     1620
    17161621VBGLR3DECL(int) VbglR3DnDGHSendAckPending(PVBGLR3GUESTDNDCMDCTX pCtx,
    17171622                                          uint32_t uDefAction, uint32_t uAllActions, const char* pcszFormats, uint32_t cbFormats)
    … …  
    17261631    VBOXDNDGHACKPENDINGMSG Msg;
    17271632    RT_ZERO(Msg);
    1728     Msg.hdr.result      = VERR_WRONG_ORDER;
    1729     Msg.hdr.u32ClientID = pCtx->uClientID;
    1730     Msg.hdr.u32Function = GUEST_DND_GH_ACK_PENDING;
    1731 
    17321633    if (pCtx->uProtocol < 3)
    17331634    {
    1734         Msg.hdr.cParms = 3;
    1735 
     1635        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_GH_ACK_PENDING, 3);
    17361636        Msg.u.v1.uDefAction.SetUInt32(uDefAction);
    17371637        Msg.u.v1.uAllActions.SetUInt32(uAllActions);
    … …  
    17401640    else
    17411641    {
    1742         Msg.hdr.cParms = 5;
    1743 
     1642        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_GH_ACK_PENDING, 5);
    17441643        /** @todo Context ID not used yet. */
    17451644        Msg.u.v3.uContext.SetUInt32(0);
    … …  
    17751674        VBOXDNDGHSENDDATAHDRMSG Msg;
    17761675        RT_ZERO(Msg);
    1777         Msg.hdr.result      = VERR_WRONG_ORDER;
    1778         Msg.hdr.u32ClientID = pCtx->uClientID;
    1779         Msg.hdr.u32Function = GUEST_DND_GH_SND_DATA_HDR;
    1780         Msg.hdr.cParms      = 12;
    1781 
     1676        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_GH_SND_DATA_HDR, 12);
    17821677        Msg.uContext.SetUInt32(0);                           /** @todo Not used yet. */
    17831678        Msg.uFlags.SetUInt32(0);                             /** @todo Not used yet. */
    … …  
    18051700        VBOXDNDGHSENDDATAMSG Msg;
    18061701        RT_ZERO(Msg);
    1807         Msg.hdr.result      = VERR_WRONG_ORDER;
    1808         Msg.hdr.u32ClientID = pCtx->uClientID;
    1809         Msg.hdr.u32Function = GUEST_DND_GH_SND_DATA;
    1810 
    18111702        if (pCtx->uProtocol >= 3)
    18121703        {
    1813             Msg.hdr.cParms = 5;
    1814 
     1704            VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_GH_SND_DATA, 5);
    18151705            Msg.u.v3.uContext.SetUInt32(0);      /** @todo Not used yet. */
    18161706            Msg.u.v3.pvChecksum.SetPtr(NULL, 0); /** @todo Not used yet. */
    … …  
    18191709        else
    18201710        {
    1821             Msg.hdr.cParms = 2;
     1711            VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_GH_SND_DATA, 2);
    18221712
    18231713            /* Total amount of bytes to send (meta data + all directory/file objects). */
    … …  
    18781768    VBOXDNDGHSENDDIRMSG Msg;
    18791769    RT_ZERO(Msg);
    1880     Msg.hdr.result      = VERR_WRONG_ORDER;
    1881     Msg.hdr.u32ClientID = pCtx->uClientID;
    1882     Msg.hdr.u32Function = GUEST_DND_GH_SND_DIR;
    1883 
    18841770    if (pCtx->uProtocol < 3)
    18851771    {
    1886         Msg.hdr.cParms = 3;
    1887 
     1772        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_GH_SND_DIR, 3);
    18881773        Msg.u.v1.pvName.SetPtr((void *)strPath.c_str(), (uint32_t)cbPath);
    18891774        Msg.u.v1.cbName.SetUInt32((uint32_t)cbPath);
    … …  
    18921777    else
    18931778    {
    1894         Msg.hdr.cParms = 4;
    1895 
     1779        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_GH_SND_DIR, 4);
    18961780        /** @todo Context ID not used yet. */
    18971781        Msg.u.v3.uContext.SetUInt32(0);
    … …  
    19331817        VBOXDNDGHSENDFILEHDRMSG MsgHdr;
    19341818        RT_ZERO(MsgHdr);
    1935         MsgHdr.hdr.result      = VERR_WRONG_ORDER;
    1936         MsgHdr.hdr.u32ClientID = pCtx->uClientID;
    1937         MsgHdr.hdr.u32Function = GUEST_DND_GH_SND_FILE_HDR;
    1938         MsgHdr.hdr.cParms      = 6;
    1939 
     1819        VBGL_HGCM_HDR_INIT(&MsgHdr.hdr, pCtx->uClientID, GUEST_DND_GH_SND_FILE_HDR, 6);
    19401820        MsgHdr.uContext.SetUInt32(0);                                                    /* Context ID; unused at the moment. */
    19411821        MsgHdr.pvName.SetPtr((void *)strPath.c_str(), (uint32_t)(strPath.length() + 1));
    … …  
    19611841        VBOXDNDGHSENDFILEDATAMSG Msg;
    19621842        RT_ZERO(Msg);
    1963         Msg.hdr.result      = VERR_WRONG_ORDER;
    1964         Msg.hdr.u32ClientID = pCtx->uClientID;
    1965         Msg.hdr.u32Function = GUEST_DND_GH_SND_FILE_DATA;
    1966 
    19671843        switch (pCtx->uProtocol)
    19681844        {
    19691845            case 3:
    19701846            {
    1971                 Msg.hdr.cParms = 5;
    1972 
     1847                VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_GH_SND_FILE_DATA, 5);
    19731848                Msg.u.v3.uContext.SetUInt32(0);
    19741849                Msg.u.v3.pvChecksum.SetPtr(NULL, 0);
    … …  
    19791854            case 2:
    19801855            {
    1981                 Msg.hdr.cParms  = 3;
    1982 
     1856                VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_GH_SND_FILE_DATA, 3);
    19831857                Msg.u.v2.uContext.SetUInt32(0);
    19841858                break;
    … …  
    19871861            default: /* Protocol v1 */
    19881862            {
    1989                 Msg.hdr.cParms  = 5;
    1990 
     1863                VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_GH_SND_FILE_DATA, 5);
    19911864                Msg.u.v1.pvName.SetPtr((void *)strPath.c_str(), (uint32_t)(strPath.length() + 1));
    19921865                Msg.u.v1.cbName.SetUInt32((uint32_t)(strPath.length() + 1));
    … …  
    22032076    VBOXDNDGHEVTERRORMSG Msg;
    22042077    RT_ZERO(Msg);
    2205     Msg.hdr.result      = VERR_WRONG_ORDER;
    2206     Msg.hdr.u32ClientID = pCtx->uClientID;
    2207     Msg.hdr.u32Function = GUEST_DND_GH_EVT_ERROR;
    2208 
    22092078    if (pCtx->uProtocol < 3)
    22102079    {
    2211         Msg.hdr.cParms = 1;
    2212 
     2080        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_GH_EVT_ERROR, 1);
    22132081        Msg.u.v1.rc.SetUInt32((uint32_t)rcErr); /* uint32_t vs. int. */
    22142082    }
    22152083    else
    22162084    {
    2217         Msg.hdr.cParms = 2;
    2218 
     2085        VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_DND_GH_EVT_ERROR, 2);
    22192086        /** @todo Context ID not used yet. */
    22202087        Msg.u.v3.uContext.SetUInt32(0);
    … …  
    22382105    return rc;
    22392106}
     2107
    22402108#endif /* VBOX_WITH_DRAG_AND_DROP_GH */
    22412109

  • trunk/src/VBox/Additions/common/VBoxGuestLib/VBoxGuestR3LibGuestCtrl.cpp

    r62847 r68458  
    104104
    105105    HGCMMsgCmdWaitFor Msg;
    106 
    107     Msg.hdr.result      = VERR_WRONG_ORDER;
    108     Msg.hdr.u32ClientID = uClientId;
    109     Msg.hdr.u32Function = GUEST_MSG_WAIT; /* Tell the host we want our next command. */
    110     Msg.hdr.cParms      = 2;              /* Just peek for the next message! */
    111 
     106    VBGL_HGCM_HDR_INIT(&Msg.hdr, uClientId,
     107                       GUEST_MSG_WAIT,      /* Tell the host we want our next command. */
     108                       2);                  /* Just peek for the next message! */
    112109    VbglHGCMParmUInt32Set(&Msg.msg, 0);
    113110    VbglHGCMParmUInt32Set(&Msg.num_parms, 0);
    … …  
    145142    HGCMMsgCmdFilterSet Msg;
    146143
    147     Msg.hdr.result      = VERR_WRONG_ORDER;
    148     Msg.hdr.u32ClientID = uClientId;
    149     Msg.hdr.u32Function = GUEST_MSG_FILTER_SET; /* Tell the host we want to set a filter. */
    150     Msg.hdr.cParms      = 4;
    151 
     144    /* Tell the host we want to set a filter. */
     145    VBGL_HGCM_HDR_INIT(&Msg.hdr, uClientId, GUEST_MSG_FILTER_SET, 4);
    152146    VbglHGCMParmUInt32Set(&Msg.value, uValue);
    153147    VbglHGCMParmUInt32Set(&Msg.mask_add, uMaskAdd);
    … …  
    170164VBGLR3DECL(int) VbglR3GuestCtrlMsgFilterUnset(uint32_t uClientId)
    171165{
     166    /* Tell the host we want to unset the filter. */
    172167    HGCMMsgCmdFilterUnset Msg;
    173 
    174     Msg.hdr.result      = VERR_WRONG_ORDER;
    175     Msg.hdr.u32ClientID = uClientId;
    176     Msg.hdr.u32Function = GUEST_MSG_FILTER_UNSET; /* Tell the host we want to unset the filter. */
    177     Msg.hdr.cParms      = 1;
    178 
     168    VBGL_HGCM_HDR_INIT(&Msg.hdr, uClientId, GUEST_MSG_FILTER_UNSET, 1);
    179169    VbglHGCMParmUInt32Set(&Msg.flags, 0 /* Flags, unused */);
    180170
    … …  
    202192
    203193    HGCMMsgCmdReply Msg;
    204 
    205     Msg.hdr.result      = VERR_WRONG_ORDER;
    206     Msg.hdr.u32ClientID = pCtx->uClientID;
    207     Msg.hdr.u32Function = GUEST_MSG_REPLY;
    208     Msg.hdr.cParms      = 4;
    209 
     194    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_REPLY, 4);
    210195    VbglHGCMParmUInt32Set(&Msg.context, pCtx->uContextID);
    211196    VbglHGCMParmUInt32Set(&Msg.rc, (uint32_t)rc); /* int vs. uint32_t */
    … …  
    235220    HGCMMsgCmdSkip Msg;
    236221
    237     Msg.hdr.result      = VERR_WRONG_ORDER;
    238     Msg.hdr.u32ClientID = uClientId;
    239     Msg.hdr.u32Function = GUEST_MSG_SKIP; /* Tell the host we want to skip
    240                                              the current assigned command. */
    241     Msg.hdr.cParms      = 1;
    242 
     222    /* Tell the host we want to skip the current assigned command. */
     223    VBGL_HGCM_HDR_INIT(&Msg.hdr, uClientId, GUEST_MSG_SKIP, 1);
    243224    VbglHGCMParmUInt32Set(&Msg.flags, 0 /* Flags, unused */);
    244225
    … …  
    260241{
    261242    HGCMMsgCancelPendingWaits Msg;
    262 
    263     Msg.hdr.result      = VERR_WRONG_ORDER;
    264     Msg.hdr.u32ClientID = uClientId;
    265     Msg.hdr.u32Function = GUEST_CANCEL_PENDING_WAITS;
    266     Msg.hdr.cParms      = 0;
    267 
     243    VBGL_HGCM_HDR_INIT(&Msg.hdr, uClientId, GUEST_CANCEL_PENDING_WAITS, 0);
    268244    int rc = vbglR3DoIOCtl(VBOXGUEST_IOCTL_HGCM_CALL(sizeof(Msg)), &Msg, sizeof(Msg));
    269245    if (RT_SUCCESS(rc))
    … …  
    291267
    292268    HGCMMsgSessionClose Msg;
    293 
    294     Msg.hdr.result      = VERR_WRONG_ORDER;
    295     Msg.hdr.u32ClientID = pCtx->uClientID;
    296     Msg.hdr.u32Function = GUEST_SESSION_CLOSE;
    297     Msg.hdr.cParms      = pCtx->uNumParms;
    298 
     269    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_SESSION_CLOSE, pCtx->uNumParms);
    299270    VbglHGCMParmUInt32Set(&Msg.context, pCtx->uContextID);
    300271    VbglHGCMParmUInt32Set(&Msg.flags, fFlags);
    … …  
    317288
    318289    HGCMMsgSessionNotify Msg;
    319 
    320     Msg.hdr.result      = VERR_WRONG_ORDER;
    321     Msg.hdr.u32ClientID = pCtx->uClientID;
    322     Msg.hdr.u32Function = GUEST_SESSION_NOTIFY;
    323     Msg.hdr.cParms      = 3;
    324 
     290    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_SESSION_NOTIFY, 3);
    325291    VbglHGCMParmUInt32Set(&Msg.context, pCtx->uContextID);
    326292    VbglHGCMParmUInt32Set(&Msg.type, uType);
    … …  
    361327
    362328    HGCMMsgSessionOpen Msg;
    363 
    364     Msg.hdr.result      = VERR_WRONG_ORDER;
    365     Msg.hdr.u32ClientID = pCtx->uClientID;
    366     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    367     Msg.hdr.cParms      = pCtx->uNumParms;
    368 
     329    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    369330    VbglHGCMParmUInt32Set(&Msg.context, 0);
    370331    VbglHGCMParmUInt32Set(&Msg.protocol, 0);
    … …  
    411372
    412373    HGCMMsgSessionClose Msg;
    413 
    414     Msg.hdr.result      = VERR_WRONG_ORDER;
    415     Msg.hdr.u32ClientID = pCtx->uClientID;
    416     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    417     Msg.hdr.cParms      = pCtx->uNumParms;
    418 
     374    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    419375    VbglHGCMParmUInt32Set(&Msg.context, 0);
    420376    VbglHGCMParmUInt32Set(&Msg.flags, 0);
    … …  
    457413
    458414    HGCMMsgPathRename Msg;
    459 
    460     Msg.hdr.result      = VERR_WRONG_ORDER;
    461     Msg.hdr.u32ClientID = pCtx->uClientID;
    462     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    463     Msg.hdr.cParms      = pCtx->uNumParms;
    464 
     415    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    465416    VbglHGCMParmUInt32Set(&Msg.context, 0);
    466417    VbglHGCMParmPtrSet(&Msg.source, pszSource, cbSource);
    … …  
    518469
    519470    HGCMMsgProcExec Msg;
    520 
    521     Msg.hdr.result      = VERR_WRONG_ORDER;
    522     Msg.hdr.u32ClientID = pCtx->uClientID;
    523     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    524     Msg.hdr.cParms      = pCtx->uNumParms;
    525 
     471    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    526472    VbglHGCMParmUInt32Set(&Msg.context, 0);
    527473    VbglHGCMParmPtrSet(&Msg.cmd, pszCmd, cbCmd);
    … …  
    604550
    605551    HGCMMsgProcOutput Msg;
    606 
    607     Msg.hdr.result = VERR_WRONG_ORDER;
    608     Msg.hdr.u32ClientID = pCtx->uClientID;
    609     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    610     Msg.hdr.cParms      = pCtx->uNumParms;
    611 
     552    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    612553    VbglHGCMParmUInt32Set(&Msg.context, 0);
    613554    VbglHGCMParmUInt32Set(&Msg.pid, 0);
    … …  
    658599
    659600    HGCMMsgProcInput Msg;
    660 
    661     Msg.hdr.result      = VERR_WRONG_ORDER;
    662     Msg.hdr.u32ClientID = pCtx->uClientID;
    663     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    664     Msg.hdr.cParms      = pCtx->uNumParms;
    665 
     601    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    666602    VbglHGCMParmUInt32Set(&Msg.context, 0);
    667603    VbglHGCMParmUInt32Set(&Msg.pid, 0);
    … …  
    702638
    703639    HGCMMsgDirRemove Msg;
    704 
    705     Msg.hdr.result      = VERR_WRONG_ORDER;
    706     Msg.hdr.u32ClientID = pCtx->uClientID;
    707     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    708     Msg.hdr.cParms      = pCtx->uNumParms;
    709 
     640    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    710641    VbglHGCMParmUInt32Set(&Msg.context, 0);
    711642    VbglHGCMParmPtrSet(&Msg.path, pszPath, cbPath);
    … …  
    753684
    754685    HGCMMsgFileOpen Msg;
    755 
    756     Msg.hdr.result      = VERR_WRONG_ORDER;
    757     Msg.hdr.u32ClientID = pCtx->uClientID;
    758     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    759     Msg.hdr.cParms      = pCtx->uNumParms;
    760 
     686    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    761687    VbglHGCMParmUInt32Set(&Msg.context, 0);
    762688    VbglHGCMParmPtrSet(&Msg.filename, pszFileName, cbFileName);
    … …  
    794720
    795721    HGCMMsgFileClose Msg;
    796 
    797     Msg.hdr.result      = VERR_WRONG_ORDER;
    798     Msg.hdr.u32ClientID = pCtx->uClientID;
    799     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    800     Msg.hdr.cParms      = pCtx->uNumParms;
    801 
     722    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    802723    VbglHGCMParmUInt32Set(&Msg.context, 0);
    803724    VbglHGCMParmUInt32Set(&Msg.handle, 0);
    … …  
    831752
    832753    HGCMMsgFileRead Msg;
    833 
    834     Msg.hdr.result      = VERR_WRONG_ORDER;
    835     Msg.hdr.u32ClientID = pCtx->uClientID;
    836     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    837     Msg.hdr.cParms      = pCtx->uNumParms;
    838 
     754    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    839755    VbglHGCMParmUInt32Set(&Msg.context, 0);
    840756    VbglHGCMParmUInt32Set(&Msg.handle, 0);
    … …  
    870786
    871787    HGCMMsgFileReadAt Msg;
    872 
    873     Msg.hdr.result      = VERR_WRONG_ORDER;
    874     Msg.hdr.u32ClientID = pCtx->uClientID;
    875     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    876     Msg.hdr.cParms      = pCtx->uNumParms;
    877 
     788    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    878789    VbglHGCMParmUInt32Set(&Msg.context, 0);
    879790    VbglHGCMParmUInt32Set(&Msg.handle, 0);
    … …  
    913824
    914825    HGCMMsgFileWrite Msg;
    915 
    916     Msg.hdr.result      = VERR_WRONG_ORDER;
    917     Msg.hdr.u32ClientID = pCtx->uClientID;
    918     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    919     Msg.hdr.cParms      = pCtx->uNumParms;
    920 
     826    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    921827    VbglHGCMParmUInt32Set(&Msg.context, 0);
    922828    VbglHGCMParmUInt32Set(&Msg.handle, 0);
    … …  
    955861
    956862    HGCMMsgFileWriteAt Msg;
    957 
    958     Msg.hdr.result      = VERR_WRONG_ORDER;
    959     Msg.hdr.u32ClientID = pCtx->uClientID;
    960     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    961     Msg.hdr.cParms      = pCtx->uNumParms;
    962 
     863    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    963864    VbglHGCMParmUInt32Set(&Msg.context, 0);
    964865    VbglHGCMParmUInt32Set(&Msg.handle, 0);
    … …  
    998899
    999900    HGCMMsgFileSeek Msg;
    1000 
    1001     Msg.hdr.result      = VERR_WRONG_ORDER;
    1002     Msg.hdr.u32ClientID = pCtx->uClientID;
    1003     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    1004     Msg.hdr.cParms      = pCtx->uNumParms;
    1005 
     901    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    1006902    VbglHGCMParmUInt32Set(&Msg.context, 0);
    1007903    VbglHGCMParmUInt32Set(&Msg.handle, 0);
    … …  
    1037933
    1038934    HGCMMsgFileTell Msg;
    1039 
    1040     Msg.hdr.result      = VERR_WRONG_ORDER;
    1041     Msg.hdr.u32ClientID = pCtx->uClientID;
    1042     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    1043     Msg.hdr.cParms      = pCtx->uNumParms;
    1044 
     935    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    1045936    VbglHGCMParmUInt32Set(&Msg.context, 0);
    1046937    VbglHGCMParmUInt32Set(&Msg.handle, 0);
    … …  
    1072963
    1073964    HGCMMsgProcTerminate Msg;
    1074 
    1075     Msg.hdr.result      = VERR_WRONG_ORDER;
    1076     Msg.hdr.u32ClientID = pCtx->uClientID;
    1077     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    1078     Msg.hdr.cParms      = pCtx->uNumParms;
    1079 
     965    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    1080966    VbglHGCMParmUInt32Set(&Msg.context, 0);
    1081967    VbglHGCMParmUInt32Set(&Msg.pid, 0);
    … …  
    1108994
    1109995    HGCMMsgProcWaitFor Msg;
    1110 
    1111     Msg.hdr.result      = VERR_WRONG_ORDER;
    1112     Msg.hdr.u32ClientID = pCtx->uClientID;
    1113     Msg.hdr.u32Function = GUEST_MSG_WAIT;
    1114     Msg.hdr.cParms      = pCtx->uNumParms;
    1115 
     996    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_MSG_WAIT, pCtx->uNumParms);
    1116997    VbglHGCMParmUInt32Set(&Msg.context, 0);
    1117998    VbglHGCMParmUInt32Set(&Msg.pid, 0);
    … …  
    11451026
    11461027    HGCMReplyFileNotify Msg;
    1147 
    1148     Msg.hdr.result      = VERR_WRONG_ORDER;
    1149     Msg.hdr.u32ClientID = pCtx->uClientID;
    1150     Msg.hdr.u32Function = GUEST_FILE_NOTIFY;
    1151     Msg.hdr.cParms      = 4;
    1152 
     1028    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_FILE_NOTIFY, 4);
    11531029    VbglHGCMParmUInt32Set(&Msg.context, pCtx->uContextID);
    11541030    VbglHGCMParmUInt32Set(&Msg.type, GUEST_FILE_NOTIFYTYPE_OPEN);
    11551031    VbglHGCMParmUInt32Set(&Msg.rc, uRc);
    1156 
    11571032    VbglHGCMParmUInt32Set(&Msg.u.open.handle, uFileHandle);
    11581033
    … …  
    11741049
    11751050    HGCMReplyFileNotify Msg;
    1176 
    1177     Msg.hdr.result      = VERR_WRONG_ORDER;
    1178     Msg.hdr.u32ClientID = pCtx->uClientID;
    1179     Msg.hdr.u32Function = GUEST_FILE_NOTIFY;
    1180     Msg.hdr.cParms      = 3;
    1181 
     1051    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_FILE_NOTIFY, 3);
    11821052    VbglHGCMParmUInt32Set(&Msg.context, pCtx->uContextID);
    11831053    VbglHGCMParmUInt32Set(&Msg.type, GUEST_FILE_NOTIFYTYPE_CLOSE);
    … …  
    12001070
    12011071    HGCMReplyFileNotify Msg;
    1202 
    1203     Msg.hdr.result      = VERR_WRONG_ORDER;
    1204     Msg.hdr.u32ClientID = pCtx->uClientID;
    1205     Msg.hdr.u32Function = GUEST_FILE_NOTIFY;
    1206     Msg.hdr.cParms      = 3;
    1207 
     1072    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_FILE_NOTIFY, 3);
    12081073    VbglHGCMParmUInt32Set(&Msg.context, pCtx->uContextID);
    12091074    VbglHGCMParmUInt32Set(&Msg.type, GUEST_FILE_NOTIFYTYPE_ERROR);
    … …  
    12281093
    12291094    HGCMReplyFileNotify Msg;
    1230 
    1231     Msg.hdr.result      = VERR_WRONG_ORDER;
    1232     Msg.hdr.u32ClientID = pCtx->uClientID;
    1233     Msg.hdr.u32Function = GUEST_FILE_NOTIFY;
    1234     Msg.hdr.cParms      = 4;
    1235 
     1095    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_FILE_NOTIFY, 4);
    12361096    VbglHGCMParmUInt32Set(&Msg.context, pCtx->uContextID);
    12371097    VbglHGCMParmUInt32Set(&Msg.type, GUEST_FILE_NOTIFYTYPE_READ);
    12381098    VbglHGCMParmUInt32Set(&Msg.rc, uRc);
    1239 
    12401099    VbglHGCMParmPtrSet(&Msg.u.read.data, pvData, cbData);
    12411100
    … …  
    12571116
    12581117    HGCMReplyFileNotify Msg;
    1259 
    1260     Msg.hdr.result      = VERR_WRONG_ORDER;
    1261     Msg.hdr.u32ClientID = pCtx->uClientID;
    1262     Msg.hdr.u32Function = GUEST_FILE_NOTIFY;
    1263     Msg.hdr.cParms      = 4;
    1264 
     1118    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_FILE_NOTIFY, 4);
    12651119    VbglHGCMParmUInt32Set(&Msg.context, pCtx->uContextID);
    12661120    VbglHGCMParmUInt32Set(&Msg.type, GUEST_FILE_NOTIFYTYPE_WRITE);
    12671121    VbglHGCMParmUInt32Set(&Msg.rc, uRc);
    1268 
    12691122    VbglHGCMParmUInt32Set(&Msg.u.write.written, uWritten);
    12701123
    … …  
    12861139
    12871140    HGCMReplyFileNotify Msg;
    1288 
    1289     Msg.hdr.result      = VERR_WRONG_ORDER;
    1290     Msg.hdr.u32ClientID = pCtx->uClientID;
    1291     Msg.hdr.u32Function = GUEST_FILE_NOTIFY;
    1292     Msg.hdr.cParms      = 4;
    1293 
     1141    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_FILE_NOTIFY, 4);
    12941142    VbglHGCMParmUInt32Set(&Msg.context, pCtx->uContextID);
    12951143    VbglHGCMParmUInt32Set(&Msg.type, GUEST_FILE_NOTIFYTYPE_SEEK);
    12961144    VbglHGCMParmUInt32Set(&Msg.rc, uRc);
    1297 
    12981145    VbglHGCMParmUInt64Set(&Msg.u.seek.offset, uOffActual);
    12991146
    … …  
    13151162
    13161163    HGCMReplyFileNotify Msg;
    1317 
    1318     Msg.hdr.result      = VERR_WRONG_ORDER;
    1319     Msg.hdr.u32ClientID = pCtx->uClientID;
    1320     Msg.hdr.u32Function = GUEST_FILE_NOTIFY;
    1321     Msg.hdr.cParms      = 4;
    1322 
     1164    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_FILE_NOTIFY, 4);
    13231165    VbglHGCMParmUInt32Set(&Msg.context, pCtx->uContextID);
    13241166    VbglHGCMParmUInt32Set(&Msg.type, GUEST_FILE_NOTIFYTYPE_TELL);
    13251167    VbglHGCMParmUInt32Set(&Msg.rc, uRc);
    1326 
    13271168    VbglHGCMParmUInt64Set(&Msg.u.tell.offset, uOffActual);
    13281169
    … …  
    13511192
    13521193    HGCMMsgProcStatus Msg;
    1353 
    1354     Msg.hdr.result = VERR_WRONG_ORDER;
    1355     Msg.hdr.u32ClientID = pCtx->uClientID;
    1356     Msg.hdr.u32Function = GUEST_EXEC_STATUS;
    1357     Msg.hdr.cParms = 5;
    1358 
     1194    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_EXEC_STATUS, 5);
    13591195    VbglHGCMParmUInt32Set(&Msg.context, pCtx->uContextID);
    13601196    VbglHGCMParmUInt32Set(&Msg.pid, uPID);
    … …  
    13871223
    13881224    HGCMMsgProcOutput Msg;
    1389 
    1390     Msg.hdr.result = VERR_WRONG_ORDER;
    1391     Msg.hdr.u32ClientID = pCtx->uClientID;
    1392     Msg.hdr.u32Function = GUEST_EXEC_OUTPUT;
    1393     Msg.hdr.cParms = 5;
    1394 
     1225    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_EXEC_OUTPUT, 5);
    13951226    VbglHGCMParmUInt32Set(&Msg.context, pCtx->uContextID);
    13961227    VbglHGCMParmUInt32Set(&Msg.pid, uPID);
    … …  
    14231254
    14241255    HGCMMsgProcStatusInput Msg;
    1425 
    1426     Msg.hdr.result = VERR_WRONG_ORDER;
    1427     Msg.hdr.u32ClientID = pCtx->uClientID;
    1428     Msg.hdr.u32Function = GUEST_EXEC_INPUT_STATUS;
    1429     Msg.hdr.cParms = 5;
    1430 
     1256    VBGL_HGCM_HDR_INIT(&Msg.hdr, pCtx->uClientID, GUEST_EXEC_INPUT_STATUS, 5);
    14311257    VbglHGCMParmUInt32Set(&Msg.context, pCtx->uContextID);
    14321258    VbglHGCMParmUInt32Set(&Msg.pid, uPID);

  • trunk/src/VBox/Additions/common/VBoxGuestLib/VBoxGuestR3LibGuestProp.cpp

    r62521 r68458  
    185185    {
    186186        SetProperty Msg;
    187 
    188         Msg.hdr.result = VERR_WRONG_ORDER;
    189         Msg.hdr.u32ClientID = idClient;
    190         Msg.hdr.u32Function = SET_PROP_VALUE;
    191         Msg.hdr.cParms = 3;
     187        VBGL_HGCM_HDR_INIT(&Msg.hdr, idClient, SET_PROP_VALUE, 3);
    192188        VbglHGCMParmPtrSetString(&Msg.name,  pszName);
    193189        VbglHGCMParmPtrSetString(&Msg.value, pszValue);
    … …  
    200196    {
    201197        DelProperty Msg;
    202 
    203         Msg.hdr.result = VERR_WRONG_ORDER;
    204         Msg.hdr.u32ClientID = idClient;
    205         Msg.hdr.u32Function = DEL_PROP;
    206         Msg.hdr.cParms = 1;
     198        VBGL_HGCM_HDR_INIT(&Msg.hdr, idClient, DEL_PROP, 1);
    207199        VbglHGCMParmPtrSetString(&Msg.name, pszName);
    208200        rc = vbglR3DoIOCtl(VBOXGUEST_IOCTL_HGCM_CALL(sizeof(Msg)), &Msg, sizeof(Msg));
    … …  
    234226    {
    235227        SetPropertyValue Msg;
    236 
    237         Msg.hdr.result = VERR_WRONG_ORDER;
    238         Msg.hdr.u32ClientID = idClient;
    239         Msg.hdr.u32Function = SET_PROP_VALUE;
    240         Msg.hdr.cParms = 2;
     228        VBGL_HGCM_HDR_INIT(&Msg.hdr, idClient, SET_PROP_VALUE, 2);
    241229        VbglHGCMParmPtrSetString(&Msg.name, pszName);
    242230        VbglHGCMParmPtrSetString(&Msg.value, pszValue);
    … …  
    248236    {
    249237        DelProperty Msg;
    250 
    251         Msg.hdr.result = VERR_WRONG_ORDER;
    252         Msg.hdr.u32ClientID = idClient;
    253         Msg.hdr.u32Function = DEL_PROP;
    254         Msg.hdr.cParms = 1;
     238        VBGL_HGCM_HDR_INIT(&Msg.hdr, idClient, DEL_PROP, 1);
    255239        VbglHGCMParmPtrSetString(&Msg.name, pszName);
    256240        rc = vbglR3DoIOCtl(VBOXGUEST_IOCTL_HGCM_CALL(sizeof(Msg)), &Msg, sizeof(Msg));
    … …  
    342326     */
    343327    GetProperty Msg;
    344 
    345     Msg.hdr.result = VERR_WRONG_ORDER;
    346     Msg.hdr.u32ClientID = idClient;
    347     Msg.hdr.u32Function = GET_PROP;
    348     Msg.hdr.cParms = 4;
     328    VBGL_HGCM_HDR_INIT(&Msg.hdr, idClient, GET_PROP, 4);
    349329    VbglHGCMParmPtrSetString(&Msg.name, pszName);
    350330    VbglHGCMParmPtrSet(&Msg.buffer, pvBuf, cbBuf);
    … …  
    544524{
    545525    EnumProperties Msg;
    546 
    547     Msg.hdr.result = VERR_WRONG_ORDER;
    548     Msg.hdr.u32ClientID = idClient;
    549     Msg.hdr.u32Function = ENUM_PROPS;
    550     Msg.hdr.cParms = 3;
     526    VBGL_HGCM_HDR_INIT(&Msg.hdr, idClient, ENUM_PROPS, 3);
     527
    551528    /* Get the length of the patterns array... */
    552529    size_t cchPatterns = 0;
    … …  
    814791
    815792    DelProperty Msg;
    816 
    817     Msg.hdr.result = VERR_WRONG_ORDER;
    818     Msg.hdr.u32ClientID = idClient;
    819     Msg.hdr.u32Function = DEL_PROP;
    820     Msg.hdr.cParms = 1;
     793    VBGL_HGCM_HDR_INIT(&Msg.hdr, idClient, DEL_PROP, 1);
    821794    VbglHGCMParmPtrSetString(&Msg.name, pszName);
    822795    int rc = vbglR3DoIOCtl(VBOXGUEST_IOCTL_HGCM_CALL(sizeof(Msg)), &Msg, sizeof(Msg));
    … …  
    924897     */
    925898    GetNotification Msg;
    926 
    927     Msg.hdr.u32Timeout = cMillies;
    928     Msg.hdr.fInterruptible = true;
    929     Msg.hdr.info.result = VERR_WRONG_ORDER;
    930     Msg.hdr.info.u32ClientID = idClient;
    931     Msg.hdr.info.u32Function = GET_NOTIFICATION;
    932     Msg.hdr.info.cParms = 4;
     899    VBGL_HGCM_HDR_INIT_TIMED(&Msg.hdr, idClient, GET_NOTIFICATION, 4, cMillies);
     900
    933901    VbglHGCMParmPtrSetString(&Msg.patterns, pszPatterns);
    934902    Msg.buffer.SetPtr(pvBuf, cbBuf);

  • trunk/src/VBox/Additions/common/VBoxGuestLib/VBoxGuestR3LibHostChannel.cpp

    r62842 r68458  
    8383
    8484    VBoxHostChannelAttach parms;
    85 
    86     parms.hdr.result = VERR_WRONG_ORDER;
    87     parms.hdr.u32ClientID = u32HGCMClientId;
    88     parms.hdr.u32Function = VBOX_HOST_CHANNEL_FN_ATTACH;
    89     parms.hdr.cParms = 3;
    90 
     85    VBGL_HGCM_HDR_INIT(&parms.hdr, u32HGCMClientId, VBOX_HOST_CHANNEL_FN_ATTACH, 3);
    9186    VbglHGCMParmPtrSet(&parms.name, pszCopy, (uint32_t)cbName);
    9287    VbglHGCMParmUInt32Set(&parms.flags, u32Flags);
    … …  
    113108{
    114109    VBoxHostChannelDetach parms;
    115 
    116     parms.hdr.result = VERR_WRONG_ORDER;
    117     parms.hdr.u32ClientID = u32HGCMClientId;
    118     parms.hdr.u32Function = VBOX_HOST_CHANNEL_FN_DETACH;
    119     parms.hdr.cParms = 1;
    120 
     110    VBGL_HGCM_HDR_INIT(&parms.hdr, u32HGCMClientId, VBOX_HOST_CHANNEL_FN_DETACH, 1);
    121111    VbglHGCMParmUInt32Set(&parms.handle, u32ChannelHandle);
    122112
    … …  
    130120{
    131121    VBoxHostChannelSend parms;
    132 
    133     parms.hdr.result = VERR_WRONG_ORDER;
    134     parms.hdr.u32ClientID = u32HGCMClientId;
    135     parms.hdr.u32Function = VBOX_HOST_CHANNEL_FN_SEND;
    136     parms.hdr.cParms = 2;
    137 
     122    VBGL_HGCM_HDR_INIT(&parms.hdr, u32HGCMClientId, VBOX_HOST_CHANNEL_FN_SEND, 2);
    138123    VbglHGCMParmUInt32Set(&parms.handle, u32ChannelHandle);
    139124    VbglHGCMParmPtrSet(&parms.data, pvData, cbData);
    … …  
    157142{
    158143    VBoxHostChannelRecv parms;
    159 
    160     parms.hdr.result = VERR_WRONG_ORDER;
    161     parms.hdr.u32ClientID = u32HGCMClientId;
    162     parms.hdr.u32Function = VBOX_HOST_CHANNEL_FN_RECV;
    163     parms.hdr.cParms = 4;
    164 
     144    VBGL_HGCM_HDR_INIT(&parms.hdr, u32HGCMClientId, VBOX_HOST_CHANNEL_FN_RECV, 4);
    165145    VbglHGCMParmUInt32Set(&parms.handle, u32ChannelHandle);
    166146    VbglHGCMParmPtrSet(&parms.data, pvData, cbData);
    … …  
    194174{
    195175    VBoxHostChannelControl parms;
    196 
    197     parms.hdr.result = VERR_WRONG_ORDER;
    198     parms.hdr.u32ClientID = u32HGCMClientId;
    199     parms.hdr.u32Function = VBOX_HOST_CHANNEL_FN_CONTROL;
    200     parms.hdr.cParms = 5;
    201 
     176    VBGL_HGCM_HDR_INIT(&parms.hdr, u32HGCMClientId, VBOX_HOST_CHANNEL_FN_CONTROL, 5);
    202177    VbglHGCMParmUInt32Set(&parms.handle, u32ChannelHandle);
    203178    VbglHGCMParmUInt32Set(&parms.code, u32Code);
    … …  
    229204{
    230205    VBoxHostChannelEventWait parms;
    231 
    232     parms.hdr.result = VERR_WRONG_ORDER;
    233     parms.hdr.u32ClientID = u32HGCMClientId;
    234     parms.hdr.u32Function = VBOX_HOST_CHANNEL_FN_EVENT_WAIT;
    235     parms.hdr.cParms = 4;
    236 
     206    VBGL_HGCM_HDR_INIT(&parms.hdr, u32HGCMClientId, VBOX_HOST_CHANNEL_FN_EVENT_WAIT, 4);
    237207    VbglHGCMParmUInt32Set(&parms.handle, 0);
    238208    VbglHGCMParmUInt32Set(&parms.id, 0);
    … …  
    260230                                             uint32_t u32HGCMClientId)
    261231{
     232    RT_NOREF1(u32ChannelHandle);
     233
    262234    VBoxHostChannelEventCancel parms;
    263     RT_NOREF1(u32ChannelHandle);
    264 
    265     parms.hdr.result = VERR_WRONG_ORDER;
    266     parms.hdr.u32ClientID = u32HGCMClientId;
    267     parms.hdr.u32Function = VBOX_HOST_CHANNEL_FN_EVENT_CANCEL;
    268     parms.hdr.cParms = 0;
     235    VBGL_HGCM_HDR_INIT(&parms.hdr, u32HGCMClientId, VBOX_HOST_CHANNEL_FN_EVENT_CANCEL, 0);
    269236
    270237    int rc = vbglR3DoIOCtl(VBOXGUEST_IOCTL_HGCM_CALL(sizeof(parms)), &parms, sizeof(parms));
    … …  
    298265
    299266    VBoxHostChannelQuery parms;
    300 
    301     parms.hdr.result = VERR_WRONG_ORDER;
    302     parms.hdr.u32ClientID = u32HGCMClientId;
    303     parms.hdr.u32Function = VBOX_HOST_CHANNEL_FN_QUERY;
    304     parms.hdr.cParms = 5;
    305 
     267    VBGL_HGCM_HDR_INIT(&parms.hdr, u32HGCMClientId, VBOX_HOST_CHANNEL_FN_QUERY, 5);
    306268    VbglHGCMParmPtrSet(&parms.name, pszCopy, (uint32_t)cbName);
    307269    VbglHGCMParmUInt32Set(&parms.code, u32Code);

  • trunk/src/VBox/Additions/common/VBoxGuestLib/VBoxGuestR3LibSharedFolders.cpp

    r62521 r68458  
    144144
    145145    VBoxSFQueryMappings Msg;
    146 
    147     Msg.callInfo.result = VERR_WRONG_ORDER;
    148     Msg.callInfo.u32ClientID = idClient;
    149     Msg.callInfo.u32Function = SHFL_FN_QUERY_MAPPINGS;
    150     Msg.callInfo.cParms = 3;
     146    VBGL_HGCM_HDR_INIT(&Msg.callInfo, idClient, SHFL_FN_QUERY_MAPPINGS, 3);
    151147
    152148    /* Set the mapping flags. */
    … …  
    237233
    238234    VBoxSFQueryMapName Msg;
    239 
    240     Msg.callInfo.result = VERR_WRONG_ORDER;
    241     Msg.callInfo.u32ClientID = idClient;
    242     Msg.callInfo.u32Function = SHFL_FN_QUERY_MAP_NAME;
    243     Msg.callInfo.cParms = 2;
     235    VBGL_HGCM_HDR_INIT(&Msg.callInfo, idClient, SHFL_FN_QUERY_MAP_NAME, 2);
    244236
    245237    int         rc;

  • trunk/src/VBox/Additions/x11/VBoxClient/check3d.cpp

    r64114 r68458  
    3030{
    3131    int rc;
    32     HGCMCLIENTID cClientID;
     32    HGCMCLIENTID idClient;
    3333    CRVBOXHGCMGETCAPS caps;
    3434    LogFlowFunc(("\n"));
    … …  
    4040    if (RT_FAILURE(rc))
    4141        exit(1);
    42     rc = VbglR3HGCMConnect("VBoxSharedCrOpenGL", &cClientID);
     42    rc = VbglR3HGCMConnect("VBoxSharedCrOpenGL", &idClient);
    4343    if (RT_FAILURE(rc))
    4444        exit(1);
    45     caps.hdr.result      = VERR_WRONG_ORDER;
    46     caps.hdr.u32ClientID = cClientID;
    47     caps.hdr.u32Function = SHCRGL_GUEST_FN_GET_CAPS_LEGACY;
    48     caps.hdr.cParms      = SHCRGL_CPARMS_GET_CAPS_LEGACY;
    4945
     46    VBGL_HGCM_HDR_INIT(&caps.hdr, idClient, SHCRGL_GUEST_FN_GET_CAPS_LEGACY, SHCRGL_CPARMS_GET_CAPS_LEGACY);
    5047    caps.Caps.type       = VMMDevHGCMParmType_32bit;
    5148    caps.Caps.u.value32  = 0;
    … …  
    5653    if (caps.Caps.u.value32 & CR_VBOX_CAP_HOST_CAPS_NOT_SUFFICIENT)
    5754        exit(1);
    58     VbglR3HGCMDisconnect(cClientID);
     55    VbglR3HGCMDisconnect(idClient);
    5956    VbglR3Term();
    6057    LogFlowFunc(("returning %Rrc\n", rc));

  • trunk/src/VBox/GuestHost/OpenGL/util/vboxhgcm.c

    r65381 r68458  
    757757        CRVBOXHGCMINJECT parms;
    758758
    759         parms.hdr.result      = VERR_WRONG_ORDER;
    760         parms.hdr.u32ClientID = conn->u32ClientID;
    761         parms.hdr.u32Function = SHCRGL_GUEST_FN_INJECT;
    762         parms.hdr.cParms      = SHCRGL_CPARMS_INJECT;
     759        VBGL_HGCM_HDR_INIT(&parms.hdr, conn->u32ClientID, SHCRGL_GUEST_FN_INJECT, SHCRGL_CPARMS_INJECT);
    763760
    764761        parms.u32ClientID.type       = VMMDevHGCMParmType_32bit;
    … …  
    777774        CRVBOXHGCMWRITE parms;
    778775
    779         parms.hdr.result      = VERR_WRONG_ORDER;
    780         parms.hdr.u32ClientID = conn->u32ClientID;
    781         parms.hdr.u32Function = SHCRGL_GUEST_FN_WRITE;
    782         parms.hdr.cParms      = SHCRGL_CPARMS_WRITE;
     776        VBGL_HGCM_HDR_INIT(&parms.hdr, conn->u32ClientID, SHCRGL_GUEST_FN_WRITE, SHCRGL_CPARMS_WRITE);
    783777
    784778        parms.pBuffer.type                   = VMMDevHGCMParmType_LinAddr_In;
    … …  
    815809    RT_NOREF(buf, len);
    816810
    817     parms.hdr.result      = VERR_WRONG_ORDER;
    818     parms.hdr.u32ClientID = conn->u32ClientID;
    819     parms.hdr.u32Function = SHCRGL_GUEST_FN_READ;
    820     parms.hdr.cParms      = SHCRGL_CPARMS_READ;
     811    VBGL_HGCM_HDR_INIT(&parms.hdr, conn->u32ClientID, SHCRGL_GUEST_FN_READ, SHCRGL_CPARMS_READ);
    821812
    822813    CRASSERT(!conn->pBuffer); /* make sure there's no data to process */
    … …  
    858849    int rc;
    859850
    860     parms.hdr.result      = VERR_WRONG_ORDER;
    861     parms.hdr.u32ClientID = conn->u32ClientID;
    862     parms.hdr.u32Function = SHCRGL_GUEST_FN_WRITE_READ;
    863     parms.hdr.cParms      = SHCRGL_CPARMS_WRITE_READ;
     851    VBGL_HGCM_HDR_INIT(&parms.hdr, conn->u32ClientID, SHCRGL_GUEST_FN_WRITE_READ, SHCRGL_CPARMS_WRITE_READ);
    864852
    865853    parms.pBuffer.type                   = VMMDevHGCMParmType_LinAddr_In;
    … …  
    883871        CRVBOXHGCMWRITEBUFFER wbParms;
    884872
    885         wbParms.hdr.result = VERR_WRONG_ORDER;
    886         wbParms.hdr.u32ClientID = conn->u32ClientID;
    887         wbParms.hdr.u32Function = SHCRGL_GUEST_FN_WRITE_BUFFER;
    888         wbParms.hdr.cParms = SHCRGL_CPARMS_WRITE_BUFFER;
     873        VBGL_HGCM_HDR_INIT(&wbParms.hdr, conn->u32ClientID, SHCRGL_GUEST_FN_WRITE_BUFFER, SHCRGL_CPARMS_WRITE_BUFFER);
    889874
    890875        wbParms.iBufferID.type = VMMDevHGCMParmType_32bit;
    … …  
    927912            CRVBOXHGCMWRITEREADBUFFERED wrbParms;
    928913
    929             wrbParms.hdr.result = VERR_WRONG_ORDER;
    930             wrbParms.hdr.u32ClientID = conn->u32ClientID;
    931             wrbParms.hdr.u32Function = SHCRGL_GUEST_FN_WRITE_READ_BUFFERED;
    932             wrbParms.hdr.cParms = SHCRGL_CPARMS_WRITE_READ_BUFFERED;
     914            VBGL_HGCM_HDR_INIT(&wrbParms.hdr, conn->u32ClientID, SHCRGL_GUEST_FN_WRITE_READ_BUFFERED, SHCRGL_CPARMS_WRITE_READ_BUFFERED);
    933915
    934916            crMemcpy(&wrbParms.iBufferID, &wbParms.iBufferID, sizeof(HGCMFunctionParameter));
    … …  
    10601042    CRASSERT(!conn->pBuffer);
    10611043
    1062     parms.hdr.result      = VERR_WRONG_ORDER;
    1063     parms.hdr.u32ClientID = conn->u32ClientID;
    1064     parms.hdr.u32Function = SHCRGL_GUEST_FN_READ;
    1065     parms.hdr.cParms      = SHCRGL_CPARMS_READ;
     1044    VBGL_HGCM_HDR_INIT(&parms.hdr, conn->u32ClientID, SHCRGL_GUEST_FN_READ, SHCRGL_CPARMS_READ);
    10661045
    10671046    parms.pBuffer.type                   = VMMDevHGCMParmType_LinAddr_Out;
    … …  
    12661245    RT_NOREF(vMajor, vMinor);
    12671246
    1268     parms.hdr.result      = VERR_WRONG_ORDER;
    1269     parms.hdr.u32ClientID = conn->u32ClientID;
    1270     parms.hdr.u32Function = SHCRGL_GUEST_FN_SET_VERSION;
    1271     parms.hdr.cParms      = SHCRGL_CPARMS_SET_VERSION;
     1247    VBGL_HGCM_HDR_INIT(&parms.hdr, conn->u32ClientID, SHCRGL_GUEST_FN_SET_VERSION, SHCRGL_CPARMS_SET_VERSION);
    12721248
    12731249    parms.vMajor.type      = VMMDevHGCMParmType_32bit;
    … …  
    13031279    int rc;
    13041280
    1305     caps.hdr.result      = VERR_WRONG_ORDER;
    1306     caps.hdr.u32ClientID = conn->u32ClientID;
    1307     caps.hdr.u32Function = SHCRGL_GUEST_FN_GET_CAPS_LEGACY;
    1308     caps.hdr.cParms      = SHCRGL_CPARMS_GET_CAPS_LEGACY;
     1281    VBGL_HGCM_HDR_INIT(&caps.hdr, conn->u32ClientID, SHCRGL_GUEST_FN_GET_CAPS_LEGACY, SHCRGL_CPARMS_GET_CAPS_LEGACY);
    13091282
    13101283    caps.Caps.type       = VMMDevHGCMParmType_32bit;
    … …  
    13381311    int rc;
    13391312
    1340     parms.hdr.result      = VERR_WRONG_ORDER;
    1341     parms.hdr.u32ClientID = conn->u32ClientID;
    1342     parms.hdr.u32Function = SHCRGL_GUEST_FN_SET_PID;
    1343     parms.hdr.cParms      = SHCRGL_CPARMS_SET_PID;
     1313    VBGL_HGCM_HDR_INIT(&parms.hdr, conn->u32ClientID, SHCRGL_GUEST_FN_SET_PID, SHCRGL_CPARMS_SET_PID);
    13441314
    13451315    parms.u64PID.type     = VMMDevHGCMParmType_64bit;

  • trunk/src/VBox/GuestHost/OpenGL/util/vboxhgsmi.c

    r65381 r68458  
    11461146    int rc;
    11471147
    1148     parms.hdr.result      = VERR_WRONG_ORDER;
    1149     parms.hdr.u32ClientID = conn->u32ClientID;
    1150     parms.hdr.u32Function = SHCRGL_GUEST_FN_SET_VERSION;
    1151     parms.hdr.cParms      = SHCRGL_CPARMS_SET_VERSION;
     1148    VBGL_HGCM_HDR_INIT(&parms.hdr, conn->u32ClientID, SHCRGL_GUEST_FN_SET_VERSION, SHCRGL_CPARMS_SET_VERSION);
    11521149
    11531150    parms.vMajor.type      = VMMDevHGCMParmType_32bit;
    … …  
    11781175    int rc;
    11791176
    1180     parms.hdr.result      = VERR_WRONG_ORDER;
    1181     parms.hdr.u32ClientID = conn->u32ClientID;
    1182     parms.hdr.u32Function = SHCRGL_GUEST_FN_SET_PID;
    1183     parms.hdr.cParms      = SHCRGL_CPARMS_SET_PID;
     1177    VBGL_HGCM_HDR_INIT(&parms.hdr, conn->u32ClientID, SHCRGL_GUEST_FN_SET_PID, SHCRGL_CPARMS_SET_PID);
    11841178
    11851179    parms.u64PID.type     = VMMDevHGCMParmType_64bit;
Note: See TracChangeset for help on using the changeset viewer.

© 2024 Oracle Support Privacy / Do Not Sell My Info Terms of Use Trademark Policy Automated Access Etiquette