VirtualBox

Changeset 45078 in vbox


Ignore:
Timestamp:
Mar 18, 2013 9:09:20 PM (12 years ago)
Author:
vboxsync
Message:

VBox/err.h + Main/GuestCtrl: Introduced own error range VERR_GSTCTL_*, solves a lot of @todos using VERR_GENERAL_FAILURE before.

Location:
trunk
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • trunk/include/VBox/err.h

    r44897 r45078  
    22282228
    22292229
     2230/** @} */
     2231
     2232/** @name VBox Guest Control Status Codes
     2233 * @{
     2234 */
     2235/** Guest side reported an error. */
     2236#define VERR_GSTCTL_GUEST_ERROR                     (-7100)
     2237/** @} */
     2238
     2239
    22302240/* SED-END */
    22312241

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

    r45010 r45078  
    482482        switch (rc)
    483483        {
    484             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     484            case VERR_GSTCTL_GUEST_ERROR:
    485485                hr = GuestSession::setErrorExternal(this, guestRc);
    486486                break;

  • trunk/src/VBox/Main/src-client/GuestDirectoryImpl.cpp

    r43162 r45078  
    263263        switch (rc)
    264264        {
    265             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     265            case VERR_GSTCTL_GUEST_ERROR:
    266266                hr = GuestProcess::setErrorExternal(this, guestRc);
    267267                break;

  • trunk/src/VBox/Main/src-client/GuestProcessImpl.cpp

    r45077 r45078  
    939939            }
    940940            else
    941                 vrc = VERR_GENERAL_FAILURE; /** @todo Special guest control rc needed! */
     941                vrc = VERR_GSTCTL_GUEST_ERROR;
    942942
    943943            if (pGuestRc)
    … …  
    11641164                }
    11651165                else
    1166                     vrc = VERR_GENERAL_FAILURE; /** @todo Special guest control rc needed! */
     1166                    vrc = VERR_GSTCTL_GUEST_ERROR;
    11671167
    11681168                if (pGuestRc)
    … …  
    12951295            }
    12961296            else
    1297                 vrc = VERR_GENERAL_FAILURE; /** @todo Special guest control rc needed! */
     1297                vrc = VERR_GSTCTL_GUEST_ERROR;
    12981298
    12991299            if (pGuestRc)
    … …  
    13321332        if (pGuestRc)
    13331333            *pGuestRc = mData.mRC; /* Return last set error. */
    1334         return VERR_GENERAL_FAILURE; /** @todo Special guest control rc needed! */
     1334        return VERR_GSTCTL_GUEST_ERROR;
    13351335    }
    13361336
    … …  
    14411441        if (pGuestRc)
    14421442            *pGuestRc = mData.mRC; /* Return last set error (if any). */
    1443         return RT_SUCCESS(mData.mRC) ? VINF_SUCCESS : VERR_GENERAL_FAILURE; /** @todo Special guest control rc needed! */
     1443        return RT_SUCCESS(mData.mRC) ? VINF_SUCCESS : VERR_GSTCTL_GUEST_ERROR;
    14441444    }
    14451445
    … …  
    14711471        {
    14721472            waitResult = pEvent->GetWaitResult();
    1473             int waitRc = pEvent->GetWaitRc();
    1474 
    1475             LogFlowThisFunc(("Waiting event returned rc=%Rrc\n", waitRc));
     1473            int guestRc = pEvent->GetWaitRc();
     1474            if (RT_FAILURE(guestRc))
     1475                vrc = VERR_GSTCTL_GUEST_ERROR;
     1476
     1477            LogFlowThisFunc(("Waiting event returned rc=%Rrc\n", guestRc));
    14761478
    14771479            if (pGuestRc)
    1478                 *pGuestRc = waitRc;
    1479 
    1480             vrc = RT_SUCCESS(waitRc) ? VINF_SUCCESS : VERR_GENERAL_FAILURE; /** @todo Special guest control rc needed! */
     1480                *pGuestRc = guestRc;
    14811481        }
    14821482
    … …  
    16031603            }
    16041604            else
    1605                 vrc = VERR_GENERAL_FAILURE; /** @todo Special guest control rc needed! */
     1605                vrc = VERR_GSTCTL_GUEST_ERROR;
    16061606
    16071607            if (pGuestRc)
    … …  
    16531653        switch (vrc)
    16541654        {
    1655             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     1655            case VERR_GSTCTL_GUEST_ERROR:
    16561656                hr = GuestProcess::setErrorExternal(this, guestRc);
    16571657                break;
    … …  
    16901690        switch (vrc)
    16911691        {
    1692            case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     1692           case VERR_GSTCTL_GUEST_ERROR:
    16931693                hr = GuestProcess::setErrorExternal(this, guestRc);
    16941694                break;
    … …  
    17501750        switch (vrc)
    17511751        {
    1752             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     1752            case VERR_GSTCTL_GUEST_ERROR:
    17531753                hr = GuestProcess::setErrorExternal(this, guestRc);
    17541754                break;
    … …  
    18181818        switch (vrc)
    18191819        {
    1820             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     1820            case VERR_GSTCTL_GUEST_ERROR:
    18211821                hr = GuestProcess::setErrorExternal(this, guestRc);
    18221822                break;
    … …  
    18991899       )
    19001900    {
    1901         vrc = VERR_GENERAL_FAILURE; /** @todo Special guest control rc needed! */
     1901        vrc = VERR_GSTCTL_GUEST_ERROR;
    19021902    }
    19031903
    … …  
    20492049
    20502050            case ProcessWaitResult_Error:
    2051                 vrc = VERR_GENERAL_FAILURE; /** @todo Special guest control rc needed! */
     2051                vrc = VERR_GSTCTL_GUEST_ERROR;
    20522052                break;
    20532053

  • trunk/src/VBox/Main/src-client/GuestSessionImpl.cpp

    r45010 r45078  
    554554            }
    555555            else
    556                 vrc = VERR_GENERAL_FAILURE; /** @todo Special guest control rc needed! */
     556                vrc = VERR_GSTCTL_GUEST_ERROR;
    557557
    558558            if (pGuestRc)
    … …  
    597597int GuestSession::directoryCreateInternal(const Utf8Str &strPath, uint32_t uMode, uint32_t uFlags, int *pGuestRc)
    598598{
     599    /* pGuestRc is optional. */
     600
    599601    LogFlowThisFunc(("strPath=%s, uMode=%x, uFlags=%x\n",
    600602                     strPath.c_str(), uMode, uFlags));
    … …  
    640642        }
    641643
    642         if (vrc == VERR_GENERAL_FAILURE) /** @todo Special guest control rc needed! */
     644        if (   vrc == VERR_GSTCTL_GUEST_ERROR
     645            && pGuestRc)
     646        {
    643647            *pGuestRc = guestRc;
     648        }
    644649    }
    645650
    … …  
    693698    }
    694699
    695     if (   vrc == VERR_GENERAL_FAILURE /** @todo Special guest control rc needed! */
     700    if (   vrc == VERR_GSTCTL_GUEST_ERROR
    696701        && pGuestRc)
     702    {
    697703        *pGuestRc = guestRc;
     704    }
    698705
    699706    LogFlowFuncLeaveRC(vrc);
    … …  
    900907    }
    901908
    902     if (   vrc == VERR_GENERAL_FAILURE /** @todo Special guest control rc needed! */
     909    if (   vrc == VERR_GSTCTL_GUEST_ERROR
    903910        && pGuestRc)
     911    {
    904912        *pGuestRc = guestRc;
     913    }
    905914
    906915    LogFlowFuncLeaveRC(vrc);
    … …  
    10281037    }
    10291038
    1030     if (   vrc == VERR_GENERAL_FAILURE /** @todo Special guest control rc needed! */
     1039    if (   vrc == VERR_GSTCTL_GUEST_ERROR
    10311040        && pGuestRc)
     1041    {
    10321042        *pGuestRc = guestRc;
     1043    }
    10331044
    10341045    LogFlowFuncLeaveRC(vrc);
    … …  
    12531264            }
    12541265            else
    1255                 vrc = VERR_GENERAL_FAILURE; /** @todo Special guest control rc needed! */
     1266                vrc = VERR_GSTCTL_GUEST_ERROR;
    12561267
    12571268            if (pGuestRc)
    … …  
    16401651        if (pGuestRc)
    16411652            *pGuestRc = mData.mRC; /* Return last set error. */
    1642         return VERR_GENERAL_FAILURE; /** @todo Special guest control rc needed! */
     1653        return VERR_GSTCTL_GUEST_ERROR;
    16431654    }
    16441655
    … …  
    17151726        if (pGuestRc)
    17161727            *pGuestRc = mData.mRC; /* Return last set error (if any). */
    1717         return RT_SUCCESS(mData.mRC) ? VINF_SUCCESS : VERR_GENERAL_FAILURE; /** @todo Special guest control rc needed! */
     1728        return RT_SUCCESS(mData.mRC) ? VINF_SUCCESS : VERR_GSTCTL_GUEST_ERROR;
    17181729    }
    17191730
    … …  
    17451756        {
    17461757            waitResult = pEvent->GetWaitResult();
    1747             int waitRc = pEvent->GetWaitRc();
    1748 
    1749             LogFlowThisFunc(("Waiting event returned rc=%Rrc\n", waitRc));
     1758            int guestRc = pEvent->GetWaitRc();
     1759            if (RT_FAILURE(guestRc))
     1760                vrc = VERR_GSTCTL_GUEST_ERROR;
     1761
     1762            LogFlowThisFunc(("Waiting event returned rc=%Rrc\n", guestRc));
    17501763
    17511764            if (pGuestRc)
    1752                 *pGuestRc = waitRc;
    1753 
    1754             vrc = RT_SUCCESS(waitRc) ? VINF_SUCCESS : VERR_GENERAL_FAILURE; /** @todo Special guest control rc needed! */
     1765                *pGuestRc = guestRc;
    17551766        }
    17561767
    … …  
    19631974        switch (rc)
    19641975        {
    1965             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     1976            case VERR_GSTCTL_GUEST_ERROR:
    19661977                hr = GuestProcess::setErrorExternal(this, guestRc);
    19671978                break;
    … …  
    20192030        switch (rc)
    20202031        {
    2021             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     2032            case VERR_GSTCTL_GUEST_ERROR:
    20222033                hr = GuestProcess::setErrorExternal(this, guestRc);
    20232034                break;
    … …  
    20602071        switch (rc)
    20612072        {
    2062             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     2073            case VERR_GSTCTL_GUEST_ERROR:
    20632074                hr = GuestProcess::setErrorExternal(this, guestRc);
    20642075                break;
    … …  
    21702181        switch (vrc)
    21712182        {
    2172             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     2183            case VERR_GSTCTL_GUEST_ERROR:
    21732184                hr = GuestProcess::setErrorExternal(this, guestRc);
    21742185                break;
    … …  
    23722383    switch (vrc)
    23732384    {
    2374         case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     2385        case VERR_GSTCTL_GUEST_ERROR:
    23752386            hr = GuestProcess::setErrorExternal(this, guestRc);
    23762387            break;
    … …  
    24112422        switch (vrc)
    24122423        {
    2413             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     2424            case VERR_GSTCTL_GUEST_ERROR:
    24142425                hr = GuestProcess::setErrorExternal(this, guestRc);
    24152426                break;
    … …  
    24712482        switch (vrc)
    24722483        {
    2473             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     2484            case VERR_GSTCTL_GUEST_ERROR:
    24742485                hr = GuestProcess::setErrorExternal(this, guestRc);
    24752486                break;
    … …  
    25222533        switch (vrc)
    25232534        {
    2524             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     2535            case VERR_GSTCTL_GUEST_ERROR:
    25252536                hr = GuestProcess::setErrorExternal(this, guestRc);
    25262537                break;
    … …  
    25662577        switch (vrc)
    25672578        {
    2568             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     2579            case VERR_GSTCTL_GUEST_ERROR:
    25692580                hr = GuestProcess::setErrorExternal(this, guestRc);
    25702581                break;
    … …  
    28582869        switch (vrc)
    28592870        {
    2860             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     2871            case VERR_GSTCTL_GUEST_ERROR:
    28612872                hr = GuestSession::setErrorExternal(this, guestRc);
    28622873                break;

  • trunk/src/VBox/Main/src-client/GuestSessionImplTasks.cpp

    r45010 r45078  
    324324        switch (rc)
    325325        {
    326             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     326            case VERR_GSTCTL_GUEST_ERROR:
    327327                setProgressErrorMsg(VBOX_E_IPRT_ERROR,
    328328                                    GuestProcess::guestErrorToString(guestRc));
    … …  
    419419                switch (rc)
    420420                {
    421                     case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     421                    case VERR_GSTCTL_GUEST_ERROR:
    422422                        setProgressErrorMsg(VBOX_E_IPRT_ERROR,
    423423                                            GuestProcess::guestErrorToString(guestRc));
    … …  
    643643                switch (rc)
    644644                {
    645                     case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     645                    case VERR_GSTCTL_GUEST_ERROR:
    646646                        setProgressErrorMsg(VBOX_E_IPRT_ERROR,
    647647                                            GuestProcess::guestErrorToString(guestRc));
    … …  
    672672                        switch (rc)
    673673                        {
    674                             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     674                            case VERR_GSTCTL_GUEST_ERROR:
    675675                                setProgressErrorMsg(VBOX_E_IPRT_ERROR,
    676676                                                    GuestProcess::guestErrorToString(guestRc));
    … …  
    703703                            switch (rc)
    704704                            {
    705                                 case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     705                                case VERR_GSTCTL_GUEST_ERROR:
    706706                                    setProgressErrorMsg(VBOX_E_IPRT_ERROR,
    707707                                                        GuestProcess::guestErrorToString(guestRc));
    … …  
    941941                switch (rc)
    942942                {
    943                     case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     943                    case VERR_GSTCTL_GUEST_ERROR:
    944944                        setProgressErrorMsg(VBOX_E_IPRT_ERROR,
    945945                                            GuestProcess::guestErrorToString(guestRc));
    … …  
    992992                break;
    993993
    994             case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     994            case VERR_GSTCTL_GUEST_ERROR:
    995995                setProgressErrorMsg(VBOX_E_IPRT_ERROR,
    996996                                    GuestProcess::guestErrorToString(guestRc));
    … …  
    12311231                switch (rc)
    12321232                {
    1233                     case VERR_GENERAL_FAILURE: /** @todo Special guest control rc needed! */
     1233                    case VERR_GSTCTL_GUEST_ERROR:
    12341234                        hr = setProgressErrorMsg(VBOX_E_IPRT_ERROR,
    12351235                                                 GuestProcess::guestErrorToString(guestRc));
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