VirtualBox

Changeset 26922 in vbox for trunk/src/VBox/Additions/WINNT


Ignore:
Timestamp:
Mar 1, 2010 4:22:05 PM (15 years ago)
Author:
vboxsync
Message:

Additions/win: no need to check the rc of the request header after VbglGRPerform()

Location:
trunk/src/VBox/Additions/WINNT
Files:
8 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Additions/WINNT/Graphics/Miniport/Helper.cpp

    r26179 r26922  
    8484        rc = VbglGRPerform (&req->header);
    8585
    86         if (RT_SUCCESS(rc) && RT_SUCCESS(req->header.rc))
     86        if (RT_SUCCESS(rc))
    8787        {
    8888            if (xres)
    … …  
    9898        else
    9999        {
    100             dprintf(("VBoxVideo::vboxQueryDisplayRequest: ERROR querying display request from VMMDev."
    101                      "rc = %Rrc, VMMDev rc = %Rrc\n", rc, req->header.rc));
     100            dprintf(("VBoxVideo::vboxQueryDisplayRequest: ERROR querying display request from VMMDev. "
     101                     "rc = %Rrc\n", rc));
    102102        }
    103103
    … …  
    129129        req->bpp    = bpp;
    130130        rc = VbglGRPerform(&req->header);
    131         if (RT_SUCCESS(rc) && RT_SUCCESS(req->header.rc))
     131        if (RT_SUCCESS(rc))
    132132        {
    133133            bRC = req->fSupported;
    … …  
    136136        {
    137137            dprintf(("VBoxVideo::vboxLikesVideoMode: ERROR querying video mode supported status from VMMDev."
    138                      "rc = %Rrc, VMMDev rc = %Rrc\n", rc, req->header.rc));
     138                     "rc = %Rrc\n", rc));
    139139        }
    140140        VbglGRFree(&req->header);
    … …  
    161161    {
    162162        rc = VbglGRPerform(&req->header);
    163         if (RT_SUCCESS(rc) && RT_SUCCESS(req->header.rc))
     163        if (RT_SUCCESS(rc))
    164164        {
    165165            retHeight = (ULONG)req->heightReduction;
    … …  
    167167        else
    168168        {
    169             dprintf(("VBoxVideo::vboxGetHeightReduction: ERROR querying height reduction value from VMMDev."
    170                      "rc = %Rrc, VMMDev rc = %Rrc\n", rc, req->header.rc));
     169            dprintf(("VBoxVideo::vboxGetHeightReduction: ERROR querying height reduction value from VMMDev. "
     170                     "rc = %Rrc\n", rc));
    171171        }
    172172        VbglGRFree(&req->header);
    … …  
    196196        rc = VbglGRPerform (&req->header);
    197197
    198         if (RT_SUCCESS(rc) && RT_SUCCESS(req->header.rc))
     198        if (RT_SUCCESS(rc))
    199199        {
    200200            if (req->mouseFeatures & VMMDEV_MOUSE_HOST_CAN_ABSOLUTE)
    … …  
    215215        else
    216216        {
    217             dprintf(("VBoxVideo::vboxQueryPointerPosInternal: ERROR querying mouse capabilities from VMMDev."
    218                      "rc = %Rrc, VMMDev rc = %Rrc\n", rc, req->header.rc));
     217            dprintf(("VBoxVideo::vboxQueryPointerPosInternal: ERROR querying mouse capabilities from VMMDev. "
     218                     "rc = %Rrc\n", rc));
    219219        }
    220220
    … …  
    359359        else
    360360        {
    361             dprintf(("VBoxVideo::vboxUpdatePointerShape: ERROR querying mouse capabilities from VMMDev."
    362                      "rc = %Rrc, VMMDev rc = %Rrc\n", rc, req->header.rc));
     361            dprintf(("VBoxVideo::vboxUpdatePointerShape: ERROR querying mouse capabilities from VMMDev. "
     362                     "rc = %Rrc\n", rc));
    363363        }
    364364

  • trunk/src/VBox/Additions/WINNT/Graphics/Miniport/VBoxVideo.cpp

    r26767 r26922  
    22082208                rc = VbglGRPerform (&req->header);
    22092209
    2210                 if (RT_SUCCESS(rc) && RT_SUCCESS(req->header.rc))
     2210                if (RT_SUCCESS(rc))
    22112211                {
    22122212                    Result = TRUE;
    … …  
    22142214                }
    22152215            }
    2216             dprintf(("VBoxVideo::VBoxVideoStartIO: Failed with rc=%x (hdr.rc=%x)\n", rc, (req) ? req->header.rc : -1));
     2216
     2217            dprintf(("VBoxVideo::VBoxVideoStartIO: Failed with rc=%x\n", rc));
    22172218            RequestPacket->StatusBlock->Status = ERROR_INVALID_FUNCTION;
    22182219            return FALSE;
    … …  
    24542455
    24552456        rc = VbglGRPerform (&req->header);
    2456         if (!RT_SUCCESS(rc) || !RT_SUCCESS(req->header.rc))
    2457             dprintf(("VBoxVideoSetGraphicsCap: request failed, rc = %Rrc, VMMDev rc = %Rrc\n", rc, req->header.rc));
    2458         if (RT_SUCCESS(rc))
    2459             rc = req->header.rc;
     2457        if (RT_FAILURE)
     2458            dprintf(("VBoxVideoSetGraphicsCap: request failed, rc = %Rrc\n", rc));
    24602459    }
    24612460    if (req != NULL)
    … …  
    27662765            int rc = VbglGRPerform (&req->header);
    27672766
    2768             if (RT_FAILURE(rc) || RT_FAILURE(req->header.rc))
    2769             {
    2770                 dprintf(("VBoxVideo::vbvaFlush: rc = %Rrc, VMMDev rc = %Rrc!!!\n", rc, req->header.rc));
     2767            if (RT_FAILURE(rc))
     2768            {
     2769                dprintf(("VBoxVideo::vbvaFlush: rc = %Rrc!\n", rc));
    27712770            }
    27722771        }
    … …  
    28602859            rc = VbglGRPerform (&req->header);
    28612860
    2862             if (RT_SUCCESS(rc) && RT_SUCCESS(req->header.rc))
     2861            if (RT_SUCCESS(rc))
    28632862            {
    28642863                if (req->fu32Status & VBVA_F_STATUS_ACCEPTED)
    … …  
    28972896            else
    28982897            {
    2899                 dprintf(("VBoxVideo::vboxVbvaEnable: rc = %Rrc, VMMDev rc = %Rrc!!!\n", rc, req->header.rc));
     2898                dprintf(("VBoxVideo::vboxVbvaEnable: rc = %Rrc!\n", rc));
    29002899
    29012900                if (RT_SUCCESS(rc))
    … …  
    29092908        else
    29102909        {
    2911             dprintf(("VBoxVideo::vboxVbvaEnable: VbglGRAlloc rc = %Rrc!!!\n", rc));
     2910            dprintf(("VBoxVideo::vboxVbvaEnable: VbglGRAlloc rc = %Rrc!\n", rc));
    29122911        }
    29132912

  • trunk/src/VBox/Additions/WINNT/MouseFilter/VBoxMouse.cpp

    r23846 r26922  
    278278            vboxRC = VbglGRPerform (&req->header);
    279279
    280             if (RT_FAILURE(vboxRC) || RT_FAILURE(req->header.rc))
     280            if (RT_FAILURE(vboxRC))
    281281            {
    282282                dprintf(("VBoxMouse::vboxDeviceRemoved: ERROR communicating new mouse capabilities to VMMDev.\n"
    283                          "rc = %d, VMMDev rc = %Rrc\n", vboxRC, req->header.rc));
     283                         "rc = %Rrc\n", vboxRC));
    284284            }
    285285
    … …  
    338338                vboxRC = VbglGRPerform (&req->header);
    339339
    340                 if (RT_FAILURE(vboxRC) || RT_FAILURE(req->header.rc))
     340                if (RT_FAILURE(vboxRC))
    341341                {
    342                     dprintf(("VBoxMouse::vboxInformHost: ERROR communicating new mouse capabilities to VMMDev."
    343                               "rc = %d, VMMDev rc = %Rrc\n", vboxRC, req->header.rc));
     342                    dprintf(("VBoxMouse::vboxInformHost: ERROR communicating new mouse capabilities to VMMDev. "
     343                             "rc = %Rrc\n", vboxRC));
    344344                }
    345345                else
    … …  
    11161116        int rc = VbglGRPerform (&req->header);
    11171117
    1118         if (RT_SUCCESS(rc) && RT_SUCCESS(req->header.rc))
     1118        if (RT_SUCCESS(rc))
    11191119        {
    11201120            if (req->mouseFeatures & VMMDEV_MOUSE_HOST_CAN_ABSOLUTE)
    … …  
    11331133        else
    11341134        {
    1135             dprintf(("VBoxMouse::VBoxMouse_ServiceCallback: ERROR querying mouse capabilities from VMMDev."
    1136                      "rc = %Rrc, VMMDev rc = %Rrc\n", rc, req->header.rc));
     1135            dprintf(("VBoxMouse::VBoxMouse_ServiceCallback: ERROR querying mouse capabilities from VMMDev. "
     1136                     "rc = %Rrc\n", rc));
    11371137        }
    11381138    }

  • trunk/src/VBox/Additions/WINNT/VBoxGuest/Helper.cpp

    r25021 r26922  
    236236        /** @todo registry lookup for additional information */
    237237
    238 
    239238        rc = VbglGRPerform (&req->header);
    240239
    241         if (RT_FAILURE(rc) || RT_FAILURE(req->header.rc))
     240        if (RT_FAILURE(rc))
    242241        {
    243             dprintf(("VBoxGuest::hlpVBoxReportGuestInfo: error reporting guest info to VMMDev."
    244                       "rc = %d, VMMDev rc = %Rrc\n", rc, req->header.rc));
     242            dprintf(("VBoxGuest::hlpVBoxReportGuestInfo: error reporting guest info to VMMDev. "
     243                     "rc = %Rrc\n", rc));
    245244        }
    246245

  • trunk/src/VBox/Additions/WINNT/VBoxGuest/VBoxGuest.cpp

    r26918 r26922  
    542542
    543543        rc = VbglGRPerform (&req->header);
    544         if (RT_FAILURE (rc) || RT_FAILURE (req->header.rc))
     544        if (RT_FAILURE (rc))
    545545        {
    546546            dprintf (("VBoxGuest::VBoxGuestDeviceControl: error issuing request to VMMDev! "
    547                       "rc = %d, VMMDev rc = %Rrc\n", rc, req->header.rc));
     547                      "rc = %Rrc\n", rc));
    548548        }
    549549        else
    … …  
    647647
    648648            rc = VbglGRPerform(&req->header);
    649             if (RT_FAILURE(rc) || RT_FAILURE(req->header.rc))
    650             {
    651                 dprintf(("VBoxGuest::VBoxGuestSetBalloonSize: error issuing request to VMMDev!"
    652                          "rc = %d, VMMDev rc = %Rrc\n", rc, req->header.rc));
     649            if (RT_FAILURE(rc))
     650            {
     651                dprintf(("VBoxGuest::VBoxGuestSetBalloonSize: error issuing request to VMMDev! "
     652                         "rc = %Rrc\n", rc));
    653653
    654654#ifndef TARGET_NT4
    … …  
    699699
    700700                rc = VbglGRPerform(&req->header);
    701                 if (RT_FAILURE(rc) || RT_FAILURE(req->header.rc))
     701                if (RT_FAILURE(rc))
    702702                {
    703                     AssertMsgFailed(("VBoxGuest::VBoxGuestSetBalloonSize: error issuing request to VMMDev! rc = %d, VMMDev rc = %Rrc\n", rc, req->header.rc));
     703                    AssertMsgFailed(("VBoxGuest::VBoxGuestSetBalloonSize: error issuing request to VMMDev! rc = %Rrc\n", rc));
    704704                    break;
    705705                }
    … …  
    743743        rc = VbglGRPerform(&req->header);
    744744
    745         if (RT_FAILURE(rc) || RT_FAILURE(req->header.rc))
    746         {
    747             dprintf(("VBoxGuest::VBoxGuestDeviceControl VBOXGUEST_IOCTL_CTL_CHECK_BALLOON: error issuing request to VMMDev!"
    748                      "rc = %d, VMMDev rc = %Rrc\n", rc, req->header.rc));
     745        if (RT_FAILURE(rc))
     746        {
     747            dprintf(("VBoxGuest::VBoxGuestDeviceControl VBOXGUEST_IOCTL_CTL_CHECK_BALLOON: error issuing request to VMMDev! "
     748                     "rc = %Rrc\n", rc));
    749749        }
    750750        else
    … …  
    995995                rc = VbglGRPerform(req);
    996996
    997                 if (RT_FAILURE(rc) || RT_FAILURE(req->rc))
     997                if (RT_FAILURE(rc))
    998998                {
    999999                    dprintf(("VBoxGuest::VBoxGuestDeviceControl VBOXGUEST_IOCTL_VMMREQUEST: Error issuing request to VMMDev! "
    1000                              "rc = %d, VMMDev rc = %Rrc\n", rc, req->rc));
     1000                             "rc = %Rrc\n", rc));
    10011001                    Status = STATUS_UNSUCCESSFUL;
    10021002                }
    … …  
    14731473        int rc = VbglGRPerform (&req->header);
    14741474
    1475         if (RT_FAILURE(rc) || RT_FAILURE(req->header.rc))
    1476         {
    1477             dprintf(("VBoxGuest::PowerStateRequest: error performing request to VMMDev."
    1478                       "rc = %d, VMMDev rc = %Rrc\n", rc, req->header.rc));
     1475        if (RT_FAILURE(rc))
     1476        {
     1477            dprintf(("VBoxGuest::PowerStateRequest: error performing request to VMMDev! "
     1478                     "rc = %Rrc\n", rc));
    14791479        }
    14801480    }
    … …  
    15721572
    15731573        rc = VbglGRPerform (&req->header);
    1574         if (RT_SUCCESS(rc) && RT_SUCCESS(req->header.rc))
     1574        if (RT_SUCCESS(rc))
    15751575        {
    15761576            dprintf(("VBoxGuest::VBoxGuestIsrHandler: acknowledge events succeeded %#x\n",
    … …  
    15841584            /* This can't be actually. This is sign of a serious problem. */
    15851585            dprintf(("VBoxGuest::VBoxGuestIsrHandler: "
    1586                      "acknowledge events failed rc = %d, header rc = %d\n",
    1587                      rc, req->header.rc));
     1586                     "acknowledge events failed rc = %Rrc\n", rc));
    15881587        }
    15891588
    … …  
    17381737        rc = VbglGRPerform (&req->header);
    17391738
    1740         if (RT_SUCCESS(rc) && RT_SUCCESS(req->header.rc))
     1739        if (RT_SUCCESS(rc))
    17411740        {
    17421741            hypervisorSize = req->hypervisorSize;
    … …  
    17811780            rc = VbglGRPerform (&req->header);
    17821781
    1783             if (RT_FAILURE(rc) || RT_FAILURE(req->header.rc))
    1784             {
    1785                 dprintf(("VBoxGuest::reserveHypervisorMemory: error communicating physical address to VMMDev!"
    1786                          "rc = %d, VMMDev rc = %Rrc\n", rc, req->header.rc));
     1782            if (RT_FAILURE(rc))
     1783            {
     1784                dprintf(("VBoxGuest::reserveHypervisorMemory: error communicating physical address to VMMDev! "
     1785                         "rc = %Rrc\n", rc));
    17871786            }
    17881787        }
    17891788        else
    17901789        {
    1791             dprintf(("VBoxGuest::reserveHypervisorMemory: request failed with rc %d, VMMDev rc = %Rrc\n", rc, req->header.rc));
     1790            dprintf(("VBoxGuest::reserveHypervisorMemory: request failed with rc = %Rrc\n", rc));
    17921791        }
    17931792        VbglGRFree (&req->header);
    … …  
    18091808
    18101809                rc = VbglGRPerform (&req->header);
    1811                 if (RT_FAILURE(rc) || RT_FAILURE(req->header.rc))
     1810                if (RT_FAILURE(rc))
    18121811                {
    1813                     dprintf(("VBoxGuest::reserveHypervisorMemory: VMMDevReq_RegisterPatchMemory error!"
    1814                                 "rc = %d, VMMDev rc = %Rrc\n", rc, req->header.rc));
     1812                    dprintf(("VBoxGuest::reserveHypervisorMemory: VMMDevReq_RegisterPatchMemory error! "
     1813                             "rc = %Rrc\n", rc));
    18151814                    RTR0MemObjFree(pDevExt->PatchMemObj, true);
    18161815                    pDevExt->PatchMemObj = NULL;
    … …  
    18191818            else
    18201819            {
    1821                 dprintf(("VBoxGuest::reserveHypervisorMemory: RTR0MemObjAllocPage failed with rc %d\n", rc));
     1820                dprintf(("VBoxGuest::reserveHypervisorMemory: RTR0MemObjAllocPage failed with rc = %Rrc\n", rc));
    18221821            }
    18231822            VbglGRFree (&req->header);
    … …  
    18471846
    18481847            rc = VbglGRPerform (&req->header);
    1849             if (RT_FAILURE(rc) || RT_FAILURE(req->header.rc))
    1850             {
    1851                 dprintf(("VBoxGuest::reserveHypervisorMemory: VMMDevReq_DeregisterPatchMemory error!"
    1852                             "rc = %d, VMMDev rc = %Rrc\n", rc, req->header.rc));
     1848            if (RT_FAILURE(rc))
     1849            {
     1850                dprintf(("VBoxGuest::reserveHypervisorMemory: VMMDevReq_DeregisterPatchMemory error! "
     1851                         "rc = %Rrc\n", rc));
    18531852                /* We intentially leak the memory object here as there still could
    18541853                 * be references to it!!!
    … …  
    18761875        rc = VbglGRPerform (&req->header);
    18771876
    1878         if (RT_FAILURE(rc) || RT_FAILURE(req->header.rc))
    1879         {
    1880             dprintf(("VBoxGuest::unreserveHypervisorMemory: error communicating physical address to VMMDev!"
    1881                      "rc = %d, VMMDev rc = %Rrc\n", rc, req->header.rc));
     1877        if (RT_FAILURE(rc))
     1878        {
     1879            dprintf(("VBoxGuest::unreserveHypervisorMemory: error communicating physical address to VMMDev! "
     1880                     "rc = %Rrc\n", rc));
    18821881        }
    18831882

  • trunk/src/VBox/Additions/WINNT/VBoxGuest/VBoxGuestPnP.cpp

    r17163 r26922  
    559559                                rc = VbglGRPerform (&req->header);
    560560
    561                                 if (RT_FAILURE(rc) || RT_FAILURE(req->header.rc))
     561                                if (RT_FAILURE(rc))
    562562                                {
    563                                     dprintf(("VBoxGuest::PowerStateRequest: error communicating new power status to VMMDev."
    564                                              "rc = %d, VMMDev rc = %Rrc\n", rc, req->header.rc));
     563                                    dprintf(("VBoxGuest::PowerStateRequest: error communicating new power status to VMMDev. "
     564                                             "rc = %Rrc\n", rc));
    565565                                }
    566566
    … …  
    587587                                    int rc = VbglGRPerform (&req->header);
    588588
    589                                     if (RT_FAILURE(rc) || RT_FAILURE(req->header.rc))
     589                                    if (RT_FAILURE(rc))
    590590                                    {
    591                                         dprintf(("VBoxGuest::PowerStateRequest: error communicating new power status to VMMDev."
    592                                                  "rc = %d, VMMDev rc = %Rrc\n", rc, req->header.rc));
     591                                        dprintf(("VBoxGuest::PowerStateRequest: error communicating new power status to VMMDev. "
     592                                                 "rc = %Rrc\n", rc));
    593593                                    }
    594594                                }

  • trunk/src/VBox/Additions/WINNT/i8042prt/i8042dep.c

    r21226 r26922  
    13581358            vboxRC = VbglGRPerform (&req->header);
    13591359
    1360             if (RT_FAILURE(vboxRC) || RT_FAILURE(req->header.rc))
     1360            if (RT_FAILURE(vboxRC))
    13611361            {
    1362                 dprintf(("i8042prt::DriverEntry: ERROR communicating new mouse capabilities to VMMDev."
    1363                          "rc = %d, VMMDev rc = %Rrc\n", vboxRC, req->header.rc));
     1362                dprintf(("i8042prt::DriverEntry: ERROR communicating new mouse capabilities to VMMDev. "
     1363                         "rc = %Rrc\n", vboxRC));
    13641364            }
    13651365            else

  • trunk/src/VBox/Additions/WINNT/i8042prt/moudep.c

    r21226 r26922  
    541541                    int rc = VbglGRPerform (&req->header);
    542542
    543                     if (RT_SUCCESS(rc) && RT_SUCCESS(req->header.rc))
     543                    if (RT_SUCCESS(rc))
    544544                    {
    545545                        if (req->mouseFeatures & VMMDEV_MOUSE_HOST_CAN_ABSOLUTE)
    … …  
    553553                    else
    554554                    {
    555                         dprintf(("i8042prt: ERROR querying mouse capabilities from VMMDev."
    556                                  "rc = %Rrc, VMMDev rc = %Rrc\n", rc, req->header.rc));
     555                        dprintf(("i8042prt: ERROR querying mouse capabilities from VMMDev. "
     556                                 "rc = %Rrc\n", rc));
    557557                    }
    558558                }
    … …  
    607607                    int rc = VbglGRPerform (&req->header);
    608608
    609                     if (RT_SUCCESS(rc) && RT_SUCCESS(req->header.rc))
     609                    if (RT_SUCCESS(rc))
    610610                    {
    611611                        if (req->mouseFeatures & VMMDEV_MOUSE_HOST_CAN_ABSOLUTE)
    … …  
    619619                    else
    620620                    {
    621                         dprintf(("i8042prt: ERROR querying mouse capabilities from VMMDev."
    622                                  "rc = %Rrc, VMMDev rc = %Rrc\n", rc, req->header.rc));
     621                        dprintf(("i8042prt: ERROR querying mouse capabilities from VMMDev. "
     622                                 "rc = %Rrc\n", rc));
    623623                    }
    624624                }
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