VirtualBox

Ignore:
Timestamp:
Apr 9, 2010 9:46:42 AM (15 years ago)
Author:
vboxsync
Message:

Main/ConsoleImpl2.cpp: some missing RC_CHECK; cosmetical fixes

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Main/ConsoleImpl2.cpp

    r28128 r28130  
    7272
    7373#ifdef VBOX_WITH_NETFLT
    74 # ifdef RT_OS_SOLARIS
     74# if defined(RT_OS_SOLARIS)
    7575#  include <zone.h>
    76 # endif
    77 # ifdef RT_OS_LINUX
     76# elif defined(RT_OS_LINUX)
    7877#  include <unistd.h>
    7978#  include <sys/ioctl.h>
    … …  
    8281#  include <linux/if.h>
    8382#  include <linux/wireless.h>
    84 # endif
    85 # ifdef RT_OS_FREEBSD
     83# elif defined(RT_OS_FREEBSD)
    8684#  include <unistd.h>
    8785#  include <sys/types.h>
    … …  
    9189#  include <net80211/ieee80211_ioctl.h>
    9290# endif
    93 # ifdef RT_OS_WINDOWS
     91# if defined(RT_OS_WINDOWS)
    9492#  include <VBox/WinNetConfig.h>
    9593#  include <Ntddndis.h>
    … …  
    114112     *   http://osxbook.com/book/bonus/chapter7/tpmdrmmyth/
    115113     */
    116     typedef struct {
     114    typedef struct
     115    {
    117116        uint32_t   key;
    118117        uint8_t    pad0[22];
    … …  
    153152                                       &outputStructCnt);
    154153        if (kr != kIOReturnSuccess)
     154        {
     155            IOServiceClose(port);
    155156            return VERR_INTERNAL_ERROR;
     157        }
    156158
    157159        for (int j=0; j<32; j++)
    … …  
    166168}
    167169
    168 #endif
    169 
     170#endif /* RT_OS_DARWIN */
     171
     172/* Darwin compile cludge */
    170173#undef PVM
    171174
    … …  
    304307#define STR_FREE()  do { if (str) { SysFreeString(str); str = NULL; } } while (0)
    305308#define RC_CHECK()  do { if (RT_FAILURE(rc)) { AssertMsgFailed(("rc=%Rrc\n", rc));  STR_FREE(); return rc;                   } } while (0)
    306 #define H()         do { if (FAILED(hrc))    { AssertMsgFailed(("hrc=%#x\n", hrc)); STR_FREE(); return VERR_GENERAL_FAILURE; } } while (0)
     309#define H()         do { if (FAILED(hrc))    { AssertMsgFailed(("hrc=%Rhrc\n", hrc)); STR_FREE(); return VERR_GENERAL_FAILURE; } } while (0)
    307310
    308311    /*
    … …  
    348351
    349352    Bstr guestTypeFamilyId;
    350     hrc = guestOSType->COMGETTER(FamilyId)(guestTypeFamilyId.asOutParam());       H();
     353    hrc = guestOSType->COMGETTER(FamilyId)(guestTypeFamilyId.asOutParam());         H();
    351354    BOOL fOsXGuest = guestTypeFamilyId == Bstr("MacOS");
    352355
    … …  
    375378    rc = CFGMR3InsertInteger(pRoot, "CSAMEnabled",          1);     /* boolean */   RC_CHECK();
    376379#endif
     380    STR_FREE();
    377381
    378382    /* cpuid leaf overrides. */
    … …  
    493497    /* HWVirtEx exclusive mode */
    494498    BOOL fHWVirtExExclusive = true;
    495     hrc = pMachine->GetHWVirtExProperty(HWVirtExPropertyType_Exclusive, &fHWVirtExExclusive);  H();
    496     rc = CFGMR3InsertInteger(pHWVirtExt, "Exclusive", fHWVirtExExclusive);                     RC_CHECK();
     499    hrc = pMachine->GetHWVirtExProperty(HWVirtExPropertyType_Exclusive, &fHWVirtExExclusive); H();
     500    rc = CFGMR3InsertInteger(pHWVirtExt, "Exclusive", fHWVirtExExclusive);          RC_CHECK();
    497501
    498502    /* Nested paging (VT-x/AMD-V) */
    499503    BOOL fEnableNestedPaging = false;
    500     hrc = pMachine->GetHWVirtExProperty(HWVirtExPropertyType_NestedPaging, &fEnableNestedPaging);   H();
    501     rc = CFGMR3InsertInteger(pHWVirtExt, "EnableNestedPaging", fEnableNestedPaging);     RC_CHECK();
     504    hrc = pMachine->GetHWVirtExProperty(HWVirtExPropertyType_NestedPaging, &fEnableNestedPaging); H();
     505    rc = CFGMR3InsertInteger(pHWVirtExt, "EnableNestedPaging", fEnableNestedPaging); RC_CHECK();
    502506
    503507    /* Large pages; requires nested paging */
    504508    BOOL fEnableLargePages = false;
    505     hrc = pMachine->GetHWVirtExProperty(HWVirtExPropertyType_LargePages, &fEnableLargePages);   H();
    506     rc = CFGMR3InsertInteger(pHWVirtExt, "EnableLargePages", fEnableLargePages);     RC_CHECK();
     509    hrc = pMachine->GetHWVirtExProperty(HWVirtExPropertyType_LargePages, &fEnableLargePages); H();
     510    rc = CFGMR3InsertInteger(pHWVirtExt, "EnableLargePages", fEnableLargePages);    RC_CHECK();
    507511
    508512    /* VPID (VT-x) */
    509513    BOOL fEnableVPID = false;
    510     hrc = pMachine->GetHWVirtExProperty(HWVirtExPropertyType_VPID, &fEnableVPID);        H();
    511     rc = CFGMR3InsertInteger(pHWVirtExt, "EnableVPID", fEnableVPID);                     RC_CHECK();
     514    hrc = pMachine->GetHWVirtExProperty(HWVirtExPropertyType_VPID, &fEnableVPID);   H();
     515    rc = CFGMR3InsertInteger(pHWVirtExt, "EnableVPID", fEnableVPID);                RC_CHECK();
    512516
    513517    /* Physical Address Extension (PAE) */
    … …  
    522526
    523527    BOOL fPXEDebug;
    524     hrc = biosSettings->COMGETTER(PXEDebugEnabled)(&fPXEDebug);                      H();
     528    hrc = biosSettings->COMGETTER(PXEDebugEnabled)(&fPXEDebug);                     H();
    525529
    526530    /*
    … …  
    531535    PCFGMNODE pDrivers;
    532536    PCFGMNODE pMod;
    533     rc = CFGMR3InsertNode(pRoot,    "PDM", &pPDM);                                     RC_CHECK();
    534     rc = CFGMR3InsertNode(pPDM,     "Drivers", &pDrivers);                             RC_CHECK();
    535     rc = CFGMR3InsertNode(pDrivers, "VBoxC", &pMod);                                   RC_CHECK();
     537    rc = CFGMR3InsertNode(pRoot,    "PDM", &pPDM);                                  RC_CHECK();
     538    rc = CFGMR3InsertNode(pPDM,     "Drivers", &pDrivers);                          RC_CHECK();
     539    rc = CFGMR3InsertNode(pDrivers, "VBoxC", &pMod);                                RC_CHECK();
    536540#ifdef VBOX_WITH_XPCOM
    537541    // VBoxC is located in the components subdirectory
    … …  
    539543    rc = RTPathAppPrivateArch(szPathVBoxC, RTPATH_MAX - sizeof("/components/VBoxC")); AssertRC(rc);
    540544    strcat(szPathVBoxC, "/components/VBoxC");
    541     rc = CFGMR3InsertString(pMod,   "Path",  szPathVBoxC);                             RC_CHECK();
     545    rc = CFGMR3InsertString(pMod,   "Path",  szPathVBoxC);                          RC_CHECK();
    542546#else
    543     rc = CFGMR3InsertString(pMod,   "Path",  "VBoxC");                                 RC_CHECK();
     547    rc = CFGMR3InsertString(pMod,   "Path",  "VBoxC");                              RC_CHECK();
    544548#endif
    545549
    … …  
    549553    PCFGMNODE pPDMAc;
    550554    PCFGMNODE pPDMAcFile;
    551     rc = CFGMR3InsertNode(pPDM, "AsyncCompletion", &pPDMAc);                           RC_CHECK();
    552     rc = CFGMR3InsertNode(pPDMAc, "File", &pPDMAcFile);                                RC_CHECK();
     555    rc = CFGMR3InsertNode(pPDM, "AsyncCompletion", &pPDMAc);                        RC_CHECK();
     556    rc = CFGMR3InsertNode(pPDMAc, "File", &pPDMAcFile);                             RC_CHECK();
    553557
    554558    /* I/O manager type */
    555559    IoMgrType_T ioMgrType;
    556     hrc = pMachine->COMGETTER(IoMgr)(&ioMgrType);                                      H();
     560    hrc = pMachine->COMGETTER(IoMgr)(&ioMgrType);                                   H();
    557561    if (ioMgrType == IoMgrType_Async)
    558562        rc = CFGMR3InsertString(pPDMAcFile, "IoMgr", "Async");
    … …  
    565569    /* I/O backend type */
    566570    IoBackendType_T ioBackendType;
    567     hrc = pMachine->COMGETTER(IoBackend)(&ioBackendType);                              H();
     571    hrc = pMachine->COMGETTER(IoBackend)(&ioBackendType);                           H();
    568572    if (ioBackendType == IoBackendType_Buffered)
    569573        rc = CFGMR3InsertString(pPDMAcFile, "FileBackend", "Buffered");
    … …  
    576580    /* Builtin I/O cache */
    577581    BOOL fIoCache = true;
    578     hrc = pMachine->COMGETTER(IoCacheEnabled)(&fIoCache);                              H();
    579     rc = CFGMR3InsertInteger(pPDMAcFile, "CacheEnabled", fIoCache);                    RC_CHECK();
     582    hrc = pMachine->COMGETTER(IoCacheEnabled)(&fIoCache);                           H();
     583    rc = CFGMR3InsertInteger(pPDMAcFile, "CacheEnabled", fIoCache);                 RC_CHECK();
    580584
    581585    /* I/O cache size */
    582586    ULONG ioCacheSize = 5;
    583     hrc = pMachine->COMGETTER(IoCacheSize)(&ioCacheSize);                              H();
    584     rc = CFGMR3InsertInteger(pPDMAcFile, "CacheSize", ioCacheSize * _1M);              RC_CHECK();
     587    hrc = pMachine->COMGETTER(IoCacheSize)(&ioCacheSize);                           H();
     588    rc = CFGMR3InsertInteger(pPDMAcFile, "CacheSize", ioCacheSize * _1M);           RC_CHECK();
    585589
    586590    /* Maximum I/O bandwidth */
    587591    ULONG ioBandwidthMax = 0;
    588     hrc = pMachine->COMGETTER(IoBandwidthMax)(&ioBandwidthMax);                        H();
     592    hrc = pMachine->COMGETTER(IoBandwidthMax)(&ioBandwidthMax);                     H();
    589593    if (ioBandwidthMax != 0)
    590594    {
    … …  
    683687    if (fHpetEnabled)
    684688    {
    685         rc = CFGMR3InsertNode(pDevices, "hpet", &pDev);                      RC_CHECK();
    686         rc = CFGMR3InsertNode(pDev,     "0", &pInst);                        RC_CHECK();
    687         rc = CFGMR3InsertInteger(pInst, "Trusted",   1);     /* boolean */   RC_CHECK();
     689        rc = CFGMR3InsertNode(pDevices, "hpet", &pDev);                             RC_CHECK();
     690        rc = CFGMR3InsertNode(pDev,     "0", &pInst);                               RC_CHECK();
     691        rc = CFGMR3InsertInteger(pInst, "Trusted",   1);     /* boolean */          RC_CHECK();
    688692    }
    689693
    … …  
    700704    {
    701705        Bstr tmpStr2;
    702         rc = CFGMR3InsertNode(pDevices, "smc", &pDev);                       RC_CHECK();
    703         rc = CFGMR3InsertNode(pDev,     "0", &pInst);                        RC_CHECK();
    704         rc = CFGMR3InsertInteger(pInst, "Trusted",   1);     /* boolean */   RC_CHECK();
    705         rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                    RC_CHECK();
    706         rc = getSmcDeviceKey(pMachine,   tmpStr2.asOutParam());              RC_CHECK();
    707         rc = CFGMR3InsertString(pCfg,   "DeviceKey", Utf8Str(tmpStr2).raw());RC_CHECK();
     706        rc = CFGMR3InsertNode(pDevices, "smc", &pDev);                              RC_CHECK();
     707        rc = CFGMR3InsertNode(pDev,     "0", &pInst);                               RC_CHECK();
     708        rc = CFGMR3InsertInteger(pInst, "Trusted",   1);     /* boolean */          RC_CHECK();
     709        rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                           RC_CHECK();
     710        rc = getSmcDeviceKey(pMachine,   tmpStr2.asOutParam());                     RC_CHECK();
     711        rc = CFGMR3InsertString(pCfg,   "DeviceKey", Utf8Str(tmpStr2).raw());       RC_CHECK();
    708712    }
    709713
    … …  
    720724    if (fLpcEnabled)
    721725    {
    722         rc = CFGMR3InsertNode(pDevices, "lpc", &pDev);                       RC_CHECK();
    723         rc = CFGMR3InsertNode(pDev,     "0", &pInst);                        RC_CHECK();
    724         rc = CFGMR3InsertInteger(pInst, "Trusted",   1);     /* boolean */   RC_CHECK();
     726        rc = CFGMR3InsertNode(pDevices, "lpc", &pDev);                              RC_CHECK();
     727        rc = CFGMR3InsertNode(pDev,     "0", &pInst);                               RC_CHECK();
     728        rc = CFGMR3InsertInteger(pInst, "Trusted",   1);     /* boolean */          RC_CHECK();
    725729    }
    726730
    … …  
    804808    BOOL fRTCUseUTC;
    805809    hrc = pMachine->COMGETTER(RTCUseUTC)(&fRTCUseUTC);                              H();
    806     rc = CFGMR3InsertInteger(pCfg,  "UseUTC", fRTCUseUTC ? 1 : 0);
     810    rc = CFGMR3InsertInteger(pCfg,  "UseUTC", fRTCUseUTC ? 1 : 0);                  RC_CHECK();
    807811
    808812    /*
    … …  
    871875    }
    872876    STR_FREE();
    873     rc = CFGMR3InsertInteger(pCfg,  "CustomVideoModes", cModes);
     877    rc = CFGMR3InsertInteger(pCfg,  "CustomVideoModes", cModes);                    RC_CHECK();
    874878
    875879    /* VESA height reduction */
    … …  
    10271031        rc = CFGMR3InsertInteger(pCfg,  "NumCPUs",          cCpus);                 RC_CHECK();
    10281032        rc = CFGMR3InsertString(pCfg,   "EfiRom",           efiRomFile.raw());      RC_CHECK();
    1029         rc = CFGMR3InsertString(pCfg,   "BootArgs",         Utf8Str(bootArgs).raw());RC_CHECK();
    1030         rc = CFGMR3InsertString(pCfg,   "DeviceProps",      Utf8Str(deviceProps).raw());RC_CHECK();
     1033        rc = CFGMR3InsertString(pCfg,   "BootArgs",         Utf8Str(bootArgs).raw()); RC_CHECK();
     1034        rc = CFGMR3InsertString(pCfg,   "DeviceProps",      Utf8Str(deviceProps).raw()); RC_CHECK();
    10311035        rc = CFGMR3InsertInteger(pCfg,  "IOAPIC",           fIOAPIC);           RC_CHECK();
    1032         rc = CFGMR3InsertBytes(pCfg,    "UUID", &HardwareUuid,sizeof(HardwareUuid));RC_CHECK();
     1036        rc = CFGMR3InsertBytes(pCfg,    "UUID", &HardwareUuid,sizeof(HardwareUuid)); RC_CHECK();
    10331037        rc = CFGMR3InsertInteger(pCfg,  "64BitEntry", f64BitEntry); /* boolean */   RC_CHECK();
    10341038        rc = CFGMR3InsertInteger(pCfg,  "GopMode", u32GopMode);    RC_CHECK();
    1035         rc = CFGMR3InsertInteger(pCfg,  "UgaHorizontalResolution", u32UgaHorisontal);    RC_CHECK();
    1036         rc = CFGMR3InsertInteger(pCfg,  "UgaVerticalResolution", u32UgaVertical);    RC_CHECK();
     1039        rc = CFGMR3InsertInteger(pCfg,  "UgaHorizontalResolution", u32UgaHorisontal); RC_CHECK();
     1040        rc = CFGMR3InsertInteger(pCfg,  "UgaVerticalResolution", u32UgaVertical);   RC_CHECK();
    10371041
    10381042        /* For OS X guests we'll force passing host's DMI info to the guest */
    … …  
    16531657            PCFGMNODE pNetBtDevCfg;
    16541658            achBootIdx[0] = '0' + uBootIdx++;   /* Boot device order. */
    1655             rc = CFGMR3InsertNode(pNetBootCfg, achBootIdx, &pNetBtDevCfg);      RC_CHECK();
    1656             rc = CFGMR3InsertInteger(pNetBtDevCfg, "NIC", it->mInstance);       RC_CHECK();
    1657             rc = CFGMR3InsertInteger(pNetBtDevCfg, "PCIDeviceNo", it->mPciDev); RC_CHECK();
    1658             rc = CFGMR3InsertInteger(pNetBtDevCfg, "PCIFunctionNo", it->mPciFn);RC_CHECK();
     1659            rc = CFGMR3InsertNode(pNetBootCfg, achBootIdx, &pNetBtDevCfg);          RC_CHECK();
     1660            rc = CFGMR3InsertInteger(pNetBtDevCfg, "NIC", it->mInstance);           RC_CHECK();
     1661            rc = CFGMR3InsertInteger(pNetBtDevCfg, "PCIDeviceNo", it->mPciDev);     RC_CHECK();
     1662            rc = CFGMR3InsertInteger(pNetBtDevCfg, "PCIFunctionNo", it->mPciFn);    RC_CHECK();
    16591663        }
    16601664    }
    … …  
    24812485#define STR_FREE()  do { if (str) { SysFreeString(str); str = NULL; } } while (0)
    24822486#define RC_CHECK()  do { if (RT_FAILURE(rc)) { AssertMsgFailed(("rc=%Rrc\n", rc));  STR_FREE(); return rc;                   } } while (0)
    2483 #define H()         do { if (FAILED(hrc))    { AssertMsgFailed(("hrc=%#x\n", hrc)); STR_FREE(); return VERR_GENERAL_FAILURE; } } while (0)
     2487#define H()         do { if (FAILED(hrc))    { AssertMsgFailed(("hrc=%Rhrc\n", hrc)); STR_FREE(); return VERR_GENERAL_FAILURE; } } while (0)
    24842488
    24852489    HRESULT hrc;
    … …  
    25932597            hrc = natDriver->COMGETTER(Network)(&str);                  H();
    25942598            if (str)
     2599            {
    25952600                rc = CFGMR3InsertStringW(pCfg, "Network", str);         RC_CHECK();
     2601            }
    25962602            STR_FREE();
    25972603            hrc = natDriver->COMGETTER(HostIP)(&str);                   H();
    … …  
    26932699                uint32_t proto = strProto.toUInt32();
    26942700                bool fValid = true;
    2695                 switch(proto)
     2701                switch (proto)
    26962702                {
    26972703                    case NATProtocol_UDP:
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