VirtualBox

Changeset 29324 in vbox


Ignore:
Timestamp:
May 11, 2010 10:06:23 AM (15 years ago)
Author:
vboxsync
Message:

ConsoleImpl2.cpp: Use AssertMsg[Stmt]Return for H and RC_CHECK. Put all invocations in column 89.

File:
1 edited

Legend:

Unmodified
Added
Removed

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

    r29292 r29324  
    303303    Bstr            bstr;
    304304
    305 #define RC_CHECK()  do { if (RT_FAILURE(rc)) { AssertMsgFailed(("rc=%Rrc\n", rc));  return rc;                   } } while (0)
    306 #define H()         do { if (FAILED(hrc))    { AssertMsgFailed(("hrc=%Rhrc\n", hrc)); return VERR_GENERAL_FAILURE; } } while (0)
     305#define RC_CHECK()  AssertMsgReturn(RT_SUCCESS(rc), ("rc=%Rrc\n", rc), rc)
     306#define H()         AssertMsgReturn(!FAILED(hrc), ("hrc=%Rhrc\n", hrc), VERR_GENERAL_FAILURE)
    307307
    308308    /*
    … …  
    310310     */
    311311    ComPtr<IVirtualBox> virtualBox;
    312     hrc = pMachine->COMGETTER(Parent)(virtualBox.asOutParam());                     H();
     312    hrc = pMachine->COMGETTER(Parent)(virtualBox.asOutParam());                         H();
    313313
    314314    ComPtr<IHost> host;
    315     hrc = virtualBox->COMGETTER(Host)(host.asOutParam());                           H();
     315    hrc = virtualBox->COMGETTER(Host)(host.asOutParam());                               H();
    316316
    317317    ComPtr<ISystemProperties> systemProperties;
    318     hrc = virtualBox->COMGETTER(SystemProperties)(systemProperties.asOutParam());   H();
     318    hrc = virtualBox->COMGETTER(SystemProperties)(systemProperties.asOutParam());       H();
    319319
    320320    ComPtr<IBIOSSettings> biosSettings;
    321     hrc = pMachine->COMGETTER(BIOSSettings)(biosSettings.asOutParam());             H();
    322 
    323     hrc = pMachine->COMGETTER(HardwareUUID)(bstr.asOutParam());                                  H();
     321    hrc = pMachine->COMGETTER(BIOSSettings)(biosSettings.asOutParam());                 H();
     322
     323    hrc = pMachine->COMGETTER(HardwareUUID)(bstr.asOutParam());                         H();
    324324    RTUUID HardwareUuid;
    325     rc = RTUuidFromUtf16(&HardwareUuid, bstr.raw());                                       RC_CHECK();
     325    rc = RTUuidFromUtf16(&HardwareUuid, bstr.raw());                                    RC_CHECK();
    326326
    327327    ULONG cRamMBs;
    328     hrc = pMachine->COMGETTER(MemorySize)(&cRamMBs);                                H();
     328    hrc = pMachine->COMGETTER(MemorySize)(&cRamMBs);                                    H();
    329329#if 0 /* enable to play with lots of memory. */
    330330    if (RTEnvExist("VBOX_RAM_SIZE"))
    … …  
    335335
    336336    ULONG cCpus = 1;
    337     hrc = pMachine->COMGETTER(CPUCount)(&cCpus);                                    H();
     337    hrc = pMachine->COMGETTER(CPUCount)(&cCpus);                                        H();
    338338
    339339    Bstr osTypeId;
    340     hrc = pMachine->COMGETTER(OSTypeId)(osTypeId.asOutParam());                     H();
     340    hrc = pMachine->COMGETTER(OSTypeId)(osTypeId.asOutParam());                         H();
    341341
    342342    BOOL fIOAPIC;
    343     hrc = biosSettings->COMGETTER(IOAPICEnabled)(&fIOAPIC);                         H();
     343    hrc = biosSettings->COMGETTER(IOAPICEnabled)(&fIOAPIC);                             H();
    344344
    345345    ComPtr<IGuestOSType> guestOSType;
    346     hrc = virtualBox->GetGuestOSType(osTypeId, guestOSType.asOutParam());           H();
     346    hrc = virtualBox->GetGuestOSType(osTypeId, guestOSType.asOutParam());               H();
    347347
    348348    Bstr guestTypeFamilyId;
    349     hrc = guestOSType->COMGETTER(FamilyId)(guestTypeFamilyId.asOutParam());         H();
     349    hrc = guestOSType->COMGETTER(FamilyId)(guestTypeFamilyId.asOutParam());             H();
    350350    BOOL fOsXGuest = guestTypeFamilyId == Bstr("MacOS");
    351351
    … …  
    360360     * Set the root (and VMM) level values.
    361361     */
    362     hrc = pMachine->COMGETTER(Name)(bstr.asOutParam());                             H();
    363     rc = CFGMR3InsertStringW(pRoot, "Name",                 bstr.raw());            RC_CHECK();
    364     rc = CFGMR3InsertBytes(pRoot,   "UUID", &HardwareUuid, sizeof(HardwareUuid));   RC_CHECK();
    365     rc = CFGMR3InsertInteger(pRoot, "RamSize",              cbRam);                 RC_CHECK();
    366     rc = CFGMR3InsertInteger(pRoot, "RamHoleSize",          cbRamHole);             RC_CHECK();
    367     rc = CFGMR3InsertInteger(pRoot, "NumCPUs",              cCpus);                 RC_CHECK();
    368     rc = CFGMR3InsertInteger(pRoot, "TimerMillies",         10);                    RC_CHECK();
     362    hrc = pMachine->COMGETTER(Name)(bstr.asOutParam());                                 H();
     363    rc = CFGMR3InsertStringW(pRoot, "Name",                 bstr.raw());                RC_CHECK();
     364    rc = CFGMR3InsertBytes(pRoot,   "UUID", &HardwareUuid, sizeof(HardwareUuid));       RC_CHECK();
     365    rc = CFGMR3InsertInteger(pRoot, "RamSize",              cbRam);                     RC_CHECK();
     366    rc = CFGMR3InsertInteger(pRoot, "RamHoleSize",          cbRamHole);                 RC_CHECK();
     367    rc = CFGMR3InsertInteger(pRoot, "NumCPUs",              cCpus);                     RC_CHECK();
     368    rc = CFGMR3InsertInteger(pRoot, "TimerMillies",         10);                        RC_CHECK();
    369369#ifdef VBOX_WITH_RAW_MODE
    370     rc = CFGMR3InsertInteger(pRoot, "RawR3Enabled",         1);     /* boolean */   RC_CHECK();
    371     rc = CFGMR3InsertInteger(pRoot, "RawR0Enabled",         1);     /* boolean */   RC_CHECK();
     370    rc = CFGMR3InsertInteger(pRoot, "RawR3Enabled",         1);     /* boolean */       RC_CHECK();
     371    rc = CFGMR3InsertInteger(pRoot, "RawR0Enabled",         1);     /* boolean */       RC_CHECK();
    372372    /** @todo Config: RawR0, PATMEnabled and CSAMEnabled needs attention later. */
    373     rc = CFGMR3InsertInteger(pRoot, "PATMEnabled",          1);     /* boolean */   RC_CHECK();
    374     rc = CFGMR3InsertInteger(pRoot, "CSAMEnabled",          1);     /* boolean */   RC_CHECK();
     373    rc = CFGMR3InsertInteger(pRoot, "PATMEnabled",          1);     /* boolean */       RC_CHECK();
     374    rc = CFGMR3InsertInteger(pRoot, "CSAMEnabled",          1);     /* boolean */       RC_CHECK();
    375375#endif
    376376
    … …  
    407407        LogRel(("Limiting CPUID leaf count for NT4 guests\n"));
    408408        PCFGMNODE pCPUM;
    409         rc = CFGMR3InsertNode(pRoot, "CPUM", &pCPUM);                               RC_CHECK();
    410         rc = CFGMR3InsertInteger(pCPUM, "NT4LeafLimit", true);                      RC_CHECK();
     409        rc = CFGMR3InsertNode(pRoot, "CPUM", &pCPUM);                                   RC_CHECK();
     410        rc = CFGMR3InsertInteger(pCPUM, "NT4LeafLimit", true);                          RC_CHECK();
    411411    }
    412412
    … …  
    418418        LogRel(("Using MWAIT extensions\n"));
    419419        PCFGMNODE pCPUM;
    420         rc = CFGMR3InsertNode(pRoot, "CPUM", &pCPUM);                               RC_CHECK();
    421         rc = CFGMR3InsertInteger(pCPUM, "MWaitExtensions", true);                   RC_CHECK();
     420        rc = CFGMR3InsertNode(pRoot, "CPUM", &pCPUM);                                   RC_CHECK();
     421        rc = CFGMR3InsertInteger(pCPUM, "MWaitExtensions", true);                       RC_CHECK();
    422422    }
    423423
    … …  
    442442    fHWVirtExEnabled = fHwVirtExtForced = TRUE;
    443443#endif /* !VBOX_WITH_RAW_MODE */
    444     rc = CFGMR3InsertInteger(pRoot, "HwVirtExtForced",      fHwVirtExtForced);      RC_CHECK();
     444    rc = CFGMR3InsertInteger(pRoot, "HwVirtExtForced",      fHwVirtExtForced);          RC_CHECK();
    445445
    446446    PCFGMNODE pHWVirtExt;
    447     rc = CFGMR3InsertNode(pRoot, "HWVirtExt", &pHWVirtExt);                         RC_CHECK();
     447    rc = CFGMR3InsertNode(pRoot, "HWVirtExt", &pHWVirtExt);                             RC_CHECK();
    448448    if (fHWVirtExEnabled)
    449449    {
    450         rc = CFGMR3InsertInteger(pHWVirtExt, "Enabled",     1);                     RC_CHECK();
     450        rc = CFGMR3InsertInteger(pHWVirtExt, "Enabled",     1);                         RC_CHECK();
    451451
    452452        /* Indicate whether 64-bit guests are supported or not. */
    … …  
    456456        BOOL fSupportsLongMode = false;
    457457        hrc = host->GetProcessorFeature(ProcessorFeature_LongMode,
    458                                         &fSupportsLongMode);                        H();
    459         hrc = guestOSType->COMGETTER(Is64Bit)(&fIs64BitGuest);                      H();
     458                                        &fSupportsLongMode);                            H();
     459        hrc = guestOSType->COMGETTER(Is64Bit)(&fIs64BitGuest);                          H();
    460460
    461461        if (fSupportsLongMode && fIs64BitGuest)
    462462        {
    463             rc = CFGMR3InsertInteger(pHWVirtExt, "64bitEnabled", 1);                RC_CHECK();
     463            rc = CFGMR3InsertInteger(pHWVirtExt, "64bitEnabled", 1);                    RC_CHECK();
    464464#if ARCH_BITS == 32 /* The recompiler must use VBoxREM64 (32-bit host only). */
    465465            PCFGMNODE pREM;
    466             rc = CFGMR3InsertNode(pRoot, "REM", &pREM);                             RC_CHECK();
    467             rc = CFGMR3InsertInteger(pREM, "64bitEnabled", 1);                      RC_CHECK();
     466            rc = CFGMR3InsertNode(pRoot, "REM", &pREM);                                 RC_CHECK();
     467            rc = CFGMR3InsertInteger(pREM, "64bitEnabled", 1);                          RC_CHECK();
    468468#endif
    469469        }
    … …  
    471471        else
    472472        {
    473             rc = CFGMR3InsertInteger(pHWVirtExt, "64bitEnabled", 0);                RC_CHECK();
     473            rc = CFGMR3InsertInteger(pHWVirtExt, "64bitEnabled", 0);                    RC_CHECK();
    474474        }
    475475#endif
    … …  
    486486             * We may want to consider adding more guest OSes (Solaris) later on.
    487487             */
    488             rc = CFGMR3InsertInteger(pHWVirtExt, "TPRPatchingEnabled", 1);          RC_CHECK();
     488            rc = CFGMR3InsertInteger(pHWVirtExt, "TPRPatchingEnabled", 1);              RC_CHECK();
    489489        }
    490490    }
    … …  
    493493    BOOL fHWVirtExExclusive = true;
    494494    hrc = pMachine->GetHWVirtExProperty(HWVirtExPropertyType_Exclusive, &fHWVirtExExclusive); H();
    495     rc = CFGMR3InsertInteger(pHWVirtExt, "Exclusive", fHWVirtExExclusive);          RC_CHECK();
     495    rc = CFGMR3InsertInteger(pHWVirtExt, "Exclusive", fHWVirtExExclusive);              RC_CHECK();
    496496
    497497    /* Nested paging (VT-x/AMD-V) */
    498498    BOOL fEnableNestedPaging = false;
    499499    hrc = pMachine->GetHWVirtExProperty(HWVirtExPropertyType_NestedPaging, &fEnableNestedPaging); H();
    500     rc = CFGMR3InsertInteger(pHWVirtExt, "EnableNestedPaging", fEnableNestedPaging); RC_CHECK();
     500    rc = CFGMR3InsertInteger(pHWVirtExt, "EnableNestedPaging", fEnableNestedPaging);    RC_CHECK();
    501501
    502502    /* Large pages; requires nested paging */
    503503    BOOL fEnableLargePages = false;
    504504    hrc = pMachine->GetHWVirtExProperty(HWVirtExPropertyType_LargePages, &fEnableLargePages); H();
    505     rc = CFGMR3InsertInteger(pHWVirtExt, "EnableLargePages", fEnableLargePages);    RC_CHECK();
     505    rc = CFGMR3InsertInteger(pHWVirtExt, "EnableLargePages", fEnableLargePages);        RC_CHECK();
    506506
    507507    /* VPID (VT-x) */
    508508    BOOL fEnableVPID = false;
    509     hrc = pMachine->GetHWVirtExProperty(HWVirtExPropertyType_VPID, &fEnableVPID);   H();
    510     rc = CFGMR3InsertInteger(pHWVirtExt, "EnableVPID", fEnableVPID);                RC_CHECK();
     509    hrc = pMachine->GetHWVirtExProperty(HWVirtExPropertyType_VPID, &fEnableVPID);       H();
     510    rc = CFGMR3InsertInteger(pHWVirtExt, "EnableVPID", fEnableVPID);                    RC_CHECK();
    511511
    512512    /* Physical Address Extension (PAE) */
    513513    BOOL fEnablePAE = false;
    514     hrc = pMachine->GetCPUProperty(CPUPropertyType_PAE, &fEnablePAE);               H();
    515     rc = CFGMR3InsertInteger(pRoot, "EnablePAE", fEnablePAE);                       RC_CHECK();
     514    hrc = pMachine->GetCPUProperty(CPUPropertyType_PAE, &fEnablePAE);                   H();
     515    rc = CFGMR3InsertInteger(pRoot, "EnablePAE", fEnablePAE);                           RC_CHECK();
    516516
    517517    /* Synthetic CPU */
    518518    BOOL fSyntheticCpu = false;
    519     hrc = pMachine->GetCPUProperty(CPUPropertyType_Synthetic, &fSyntheticCpu);      H();
    520     rc = CFGMR3InsertInteger(pRoot, "SyntheticCpu", fSyntheticCpu);                 RC_CHECK();
     519    hrc = pMachine->GetCPUProperty(CPUPropertyType_Synthetic, &fSyntheticCpu);          H();
     520    rc = CFGMR3InsertInteger(pRoot, "SyntheticCpu", fSyntheticCpu);                     RC_CHECK();
    521521
    522522    BOOL fPXEDebug;
    523     hrc = biosSettings->COMGETTER(PXEDebugEnabled)(&fPXEDebug);                     H();
     523    hrc = biosSettings->COMGETTER(PXEDebugEnabled)(&fPXEDebug);                         H();
    524524
    525525    /*
    … …  
    530530    PCFGMNODE pDrivers;
    531531    PCFGMNODE pMod;
    532     rc = CFGMR3InsertNode(pRoot,    "PDM", &pPDM);                                  RC_CHECK();
    533     rc = CFGMR3InsertNode(pPDM,     "Drivers", &pDrivers);                          RC_CHECK();
    534     rc = CFGMR3InsertNode(pDrivers, "VBoxC", &pMod);                                RC_CHECK();
     532    rc = CFGMR3InsertNode(pRoot,    "PDM", &pPDM);                                      RC_CHECK();
     533    rc = CFGMR3InsertNode(pPDM,     "Drivers", &pDrivers);                              RC_CHECK();
     534    rc = CFGMR3InsertNode(pDrivers, "VBoxC", &pMod);                                    RC_CHECK();
    535535#ifdef VBOX_WITH_XPCOM
    536536    // VBoxC is located in the components subdirectory
    537537    char szPathVBoxC[RTPATH_MAX];
    538     rc = RTPathAppPrivateArch(szPathVBoxC, RTPATH_MAX - sizeof("/components/VBoxC")); AssertRC(rc);
     538    rc = RTPathAppPrivateArch(szPathVBoxC, RTPATH_MAX - sizeof("/components/VBoxC"));   AssertRC(rc);
    539539    strcat(szPathVBoxC, "/components/VBoxC");
    540     rc = CFGMR3InsertString(pMod,   "Path",  szPathVBoxC);                          RC_CHECK();
     540    rc = CFGMR3InsertString(pMod,   "Path",  szPathVBoxC);                              RC_CHECK();
    541541#else
    542     rc = CFGMR3InsertString(pMod,   "Path",  "VBoxC");                              RC_CHECK();
     542    rc = CFGMR3InsertString(pMod,   "Path",  "VBoxC");                                  RC_CHECK();
    543543#endif
    544544
    … …  
    548548    PCFGMNODE pPDMAc;
    549549    PCFGMNODE pPDMAcFile;
    550     rc = CFGMR3InsertNode(pPDM, "AsyncCompletion", &pPDMAc);                        RC_CHECK();
    551     rc = CFGMR3InsertNode(pPDMAc, "File", &pPDMAcFile);                             RC_CHECK();
     550    rc = CFGMR3InsertNode(pPDM, "AsyncCompletion", &pPDMAc);                            RC_CHECK();
     551    rc = CFGMR3InsertNode(pPDMAc, "File", &pPDMAcFile);                                 RC_CHECK();
    552552
    553553    /* Builtin I/O cache */
    554554    BOOL fIoCache = true;
    555     hrc = pMachine->COMGETTER(IoCacheEnabled)(&fIoCache);                           H();
    556     rc = CFGMR3InsertInteger(pPDMAcFile, "CacheEnabled", fIoCache);                 RC_CHECK();
     555    hrc = pMachine->COMGETTER(IoCacheEnabled)(&fIoCache);                               H();
     556    rc = CFGMR3InsertInteger(pPDMAcFile, "CacheEnabled", fIoCache);                     RC_CHECK();
    557557
    558558    /* I/O cache size */
    559559    ULONG ioCacheSize = 5;
    560     hrc = pMachine->COMGETTER(IoCacheSize)(&ioCacheSize);                           H();
    561     rc = CFGMR3InsertInteger(pPDMAcFile, "CacheSize", ioCacheSize * _1M);           RC_CHECK();
     560    hrc = pMachine->COMGETTER(IoCacheSize)(&ioCacheSize);                               H();
     561    rc = CFGMR3InsertInteger(pPDMAcFile, "CacheSize", ioCacheSize * _1M);               RC_CHECK();
    562562
    563563    /* Maximum I/O bandwidth */
    564564    ULONG ioBandwidthMax = 0;
    565     hrc = pMachine->COMGETTER(IoBandwidthMax)(&ioBandwidthMax);                     H();
     565    hrc = pMachine->COMGETTER(IoBandwidthMax)(&ioBandwidthMax);                         H();
    566566    if (ioBandwidthMax != 0)
    567567    {
    … …  
    582582    PCFGMNODE pNetBootCfg = NULL;   /* /Devices/pcbios/0/Config/NetBoot/ */
    583583
    584     rc = CFGMR3InsertNode(pRoot, "Devices", &pDevices);                             RC_CHECK();
     584    rc = CFGMR3InsertNode(pRoot, "Devices", &pDevices);                                 RC_CHECK();
    585585
    586586    /*
    587587     * PC Arch.
    588588     */
    589     rc = CFGMR3InsertNode(pDevices, "pcarch", &pDev);                               RC_CHECK();
    590     rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
    591     rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */   RC_CHECK();
    592     rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
     589    rc = CFGMR3InsertNode(pDevices, "pcarch", &pDev);                                   RC_CHECK();
     590    rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                       RC_CHECK();
     591    rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */       RC_CHECK();
     592    rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                                   RC_CHECK();
    593593
    594594    /*
    … …  
    596596     */
    597597    LONG64 timeOffset;
    598     hrc = biosSettings->COMGETTER(TimeOffset)(&timeOffset);                         H();
     598    hrc = biosSettings->COMGETTER(TimeOffset)(&timeOffset);                             H();
    599599    PCFGMNODE pTMNode;
    600     rc = CFGMR3InsertNode(pRoot, "TM", &pTMNode);                                   RC_CHECK();
    601     rc = CFGMR3InsertInteger(pTMNode, "UTCOffset", timeOffset * 1000000);           RC_CHECK();
     600    rc = CFGMR3InsertNode(pRoot, "TM", &pTMNode);                                       RC_CHECK();
     601    rc = CFGMR3InsertInteger(pTMNode, "UTCOffset", timeOffset * 1000000);               RC_CHECK();
    602602
    603603    /*
    604604     * DMA
    605605     */
    606     rc = CFGMR3InsertNode(pDevices, "8237A", &pDev);                                RC_CHECK();
    607     rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
    608     rc = CFGMR3InsertInteger(pInst, "Trusted", 1);                  /* boolean */   RC_CHECK();
     606    rc = CFGMR3InsertNode(pDevices, "8237A", &pDev);                                    RC_CHECK();
     607    rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                       RC_CHECK();
     608    rc = CFGMR3InsertInteger(pInst, "Trusted", 1);                  /* boolean */       RC_CHECK();
    609609
    610610    /*
    611611     * PCI buses.
    612612     */
    613     rc = CFGMR3InsertNode(pDevices, "pci", &pDev); /* piix3 */                      RC_CHECK();
    614     rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
    615     rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */   RC_CHECK();
    616     rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
    617     rc = CFGMR3InsertInteger(pCfg,  "IOAPIC", fIOAPIC);                             RC_CHECK();
     613    rc = CFGMR3InsertNode(pDevices, "pci", &pDev); /* piix3 */                          RC_CHECK();
     614    rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                       RC_CHECK();
     615    rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */       RC_CHECK();
     616    rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                                   RC_CHECK();
     617    rc = CFGMR3InsertInteger(pCfg,  "IOAPIC", fIOAPIC);                                 RC_CHECK();
    618618
    619619#if 0 /* enable this to test PCI bridging */
    620     rc = CFGMR3InsertNode(pDevices, "pcibridge", &pDev);                            RC_CHECK();
    621     rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
    622     rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */   RC_CHECK();
    623     rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
    624     rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo",         14);                     RC_CHECK();
    625     rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",        0);                     RC_CHECK();
    626     rc = CFGMR3InsertInteger(pInst, "PCIBusNo",             0);/* -> pci[0] */      RC_CHECK();
    627 
    628     rc = CFGMR3InsertNode(pDev,     "1", &pInst);                                   RC_CHECK();
    629     rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */   RC_CHECK();
    630     rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
    631     rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo",          1);                     RC_CHECK();
    632     rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",        0);                     RC_CHECK();
    633     rc = CFGMR3InsertInteger(pInst, "PCIBusNo",             1);/* ->pcibridge[0] */ RC_CHECK();
    634 
    635     rc = CFGMR3InsertNode(pDev,     "2", &pInst);                                   RC_CHECK();
    636     rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */   RC_CHECK();
    637     rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
    638     rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo",          3);                     RC_CHECK();
    639     rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",        0);                     RC_CHECK();
    640     rc = CFGMR3InsertInteger(pInst, "PCIBusNo",             1);/* ->pcibridge[0] */ RC_CHECK();
     620    rc = CFGMR3InsertNode(pDevices, "pcibridge", &pDev);                                RC_CHECK();
     621    rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                       RC_CHECK();
     622    rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */       RC_CHECK();
     623    rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                                   RC_CHECK();
     624    rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo",         14);                         RC_CHECK();
     625    rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",        0);                         RC_CHECK();
     626    rc = CFGMR3InsertInteger(pInst, "PCIBusNo",             0);/* -> pci[0] */          RC_CHECK();
     627
     628    rc = CFGMR3InsertNode(pDev,     "1", &pInst);                                       RC_CHECK();
     629    rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */       RC_CHECK();
     630    rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                                   RC_CHECK();
     631    rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo",          1);                         RC_CHECK();
     632    rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",        0);                         RC_CHECK();
     633    rc = CFGMR3InsertInteger(pInst, "PCIBusNo",             1);/* ->pcibridge[0] */     RC_CHECK();
     634
     635    rc = CFGMR3InsertNode(pDev,     "2", &pInst);                                       RC_CHECK();
     636    rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */       RC_CHECK();
     637    rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                                   RC_CHECK();
     638    rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo",          3);                         RC_CHECK();
     639    rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",        0);                         RC_CHECK();
     640    rc = CFGMR3InsertInteger(pInst, "PCIBusNo",             1);/* ->pcibridge[0] */     RC_CHECK();
    641641#endif
    642642
    … …  
    650650#ifdef VBOX_WITH_HPET
    651651    /* Other guests may wish to use HPET too, but MacOS X not functional without it */
    652     hrc = pMachine->COMGETTER(HpetEnabled)(&fHpetEnabled);                          H();
     652    hrc = pMachine->COMGETTER(HpetEnabled)(&fHpetEnabled);                              H();
    653653    /* so always enable HPET in extended profile */
    654654    fHpetEnabled |= fOsXGuest;
    … …  
    658658    if (fHpetEnabled)
    659659    {
    660         rc = CFGMR3InsertNode(pDevices, "hpet", &pDev);                             RC_CHECK();
    661         rc = CFGMR3InsertNode(pDev,     "0", &pInst);                               RC_CHECK();
    662         rc = CFGMR3InsertInteger(pInst, "Trusted",   1);     /* boolean */          RC_CHECK();
     660        rc = CFGMR3InsertNode(pDevices, "hpet", &pDev);                                 RC_CHECK();
     661        rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
     662        rc = CFGMR3InsertInteger(pInst, "Trusted",   1);     /* boolean */              RC_CHECK();
    663663    }
    664664
    … …  
    675675    {
    676676        Bstr tmpStr2;
    677         rc = CFGMR3InsertNode(pDevices, "smc", &pDev);                              RC_CHECK();
    678         rc = CFGMR3InsertNode(pDev,     "0", &pInst);                               RC_CHECK();
    679         rc = CFGMR3InsertInteger(pInst, "Trusted",   1);     /* boolean */          RC_CHECK();
    680         rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                           RC_CHECK();
    681         rc = getSmcDeviceKey(pMachine,   tmpStr2.asOutParam());                     RC_CHECK();
    682         rc = CFGMR3InsertString(pCfg,   "DeviceKey", Utf8Str(tmpStr2).raw());       RC_CHECK();
     677        rc = CFGMR3InsertNode(pDevices, "smc", &pDev);                                  RC_CHECK();
     678        rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
     679        rc = CFGMR3InsertInteger(pInst, "Trusted",   1);     /* boolean */              RC_CHECK();
     680        rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
     681        rc = getSmcDeviceKey(pMachine,   tmpStr2.asOutParam());                         RC_CHECK();
     682        rc = CFGMR3InsertString(pCfg,   "DeviceKey", Utf8Str(tmpStr2).raw());           RC_CHECK();
    683683    }
    684684
    … …  
    695695    if (fLpcEnabled)
    696696    {
    697         rc = CFGMR3InsertNode(pDevices, "lpc", &pDev);                              RC_CHECK();
    698         rc = CFGMR3InsertNode(pDev,     "0", &pInst);                               RC_CHECK();
    699         rc = CFGMR3InsertInteger(pInst, "Trusted",   1);     /* boolean */          RC_CHECK();
     697        rc = CFGMR3InsertNode(pDevices, "lpc", &pDev);                                  RC_CHECK();
     698        rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
     699        rc = CFGMR3InsertInteger(pInst, "Trusted",   1);     /* boolean */              RC_CHECK();
    700700    }
    701701
    … …  
    703703     * PS/2 keyboard & mouse.
    704704     */
    705     rc = CFGMR3InsertNode(pDevices, "pckbd", &pDev);                                RC_CHECK();
    706     rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
    707     rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */   RC_CHECK();
    708     rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
    709 
    710     rc = CFGMR3InsertNode(pInst,    "LUN#0", &pLunL0);                              RC_CHECK();
    711     rc = CFGMR3InsertString(pLunL0, "Driver",               "KeyboardQueue");       RC_CHECK();
    712     rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
    713     rc = CFGMR3InsertInteger(pCfg,  "QueueSize",            64);                    RC_CHECK();
    714 
    715     rc = CFGMR3InsertNode(pLunL0,   "AttachedDriver", &pLunL1);                     RC_CHECK();
    716     rc = CFGMR3InsertString(pLunL1, "Driver",               "MainKeyboard");        RC_CHECK();
    717     rc = CFGMR3InsertNode(pLunL1,   "Config", &pCfg);                               RC_CHECK();
     705    rc = CFGMR3InsertNode(pDevices, "pckbd", &pDev);                                    RC_CHECK();
     706    rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                       RC_CHECK();
     707    rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */       RC_CHECK();
     708    rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                                   RC_CHECK();
     709
     710    rc = CFGMR3InsertNode(pInst,    "LUN#0", &pLunL0);                                  RC_CHECK();
     711    rc = CFGMR3InsertString(pLunL0, "Driver",               "KeyboardQueue");           RC_CHECK();
     712    rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                                   RC_CHECK();
     713    rc = CFGMR3InsertInteger(pCfg,  "QueueSize",            64);                        RC_CHECK();
     714
     715    rc = CFGMR3InsertNode(pLunL0,   "AttachedDriver", &pLunL1);                         RC_CHECK();
     716    rc = CFGMR3InsertString(pLunL1, "Driver",               "MainKeyboard");            RC_CHECK();
     717    rc = CFGMR3InsertNode(pLunL1,   "Config", &pCfg);                                   RC_CHECK();
    718718    Keyboard *pKeyboard = pConsole->mKeyboard;
    719     rc = CFGMR3InsertInteger(pCfg,  "Object",     (uintptr_t)pKeyboard);            RC_CHECK();
    720 
    721     rc = CFGMR3InsertNode(pInst,    "LUN#1", &pLunL0);                              RC_CHECK();
    722     rc = CFGMR3InsertString(pLunL0, "Driver",               "MouseQueue");          RC_CHECK();
    723     rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
    724     rc = CFGMR3InsertInteger(pCfg,  "QueueSize",            128);                   RC_CHECK();
    725 
    726     rc = CFGMR3InsertNode(pLunL0,   "AttachedDriver", &pLunL1);                     RC_CHECK();
    727     rc = CFGMR3InsertString(pLunL1, "Driver",               "MainMouse");           RC_CHECK();
    728     rc = CFGMR3InsertNode(pLunL1,   "Config", &pCfg);                               RC_CHECK();
     719    rc = CFGMR3InsertInteger(pCfg,  "Object",     (uintptr_t)pKeyboard);                RC_CHECK();
     720
     721    rc = CFGMR3InsertNode(pInst,    "LUN#1", &pLunL0);                                  RC_CHECK();
     722    rc = CFGMR3InsertString(pLunL0, "Driver",               "MouseQueue");              RC_CHECK();
     723    rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                                   RC_CHECK();
     724    rc = CFGMR3InsertInteger(pCfg,  "QueueSize",            128);                       RC_CHECK();
     725
     726    rc = CFGMR3InsertNode(pLunL0,   "AttachedDriver", &pLunL1);                         RC_CHECK();
     727    rc = CFGMR3InsertString(pLunL1, "Driver",               "MainMouse");               RC_CHECK();
     728    rc = CFGMR3InsertNode(pLunL1,   "Config", &pCfg);                                   RC_CHECK();
    729729    Mouse *pMouse = pConsole->mMouse;
    730     rc = CFGMR3InsertInteger(pCfg,  "Object",     (uintptr_t)pMouse);               RC_CHECK();
     730    rc = CFGMR3InsertInteger(pCfg,  "Object",     (uintptr_t)pMouse);                   RC_CHECK();
    731731
    732732    /*
    733733     * i8254 Programmable Interval Timer And Dummy Speaker
    734734     */
    735     rc = CFGMR3InsertNode(pDevices, "i8254", &pDev);                                RC_CHECK();
    736     rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
    737     rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
     735    rc = CFGMR3InsertNode(pDevices, "i8254", &pDev);                                    RC_CHECK();
     736    rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                       RC_CHECK();
     737    rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                                   RC_CHECK();
    738738#ifdef DEBUG
    739     rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */   RC_CHECK();
     739    rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */       RC_CHECK();
    740740#endif
    741741
    … …  
    743743     * i8259 Programmable Interrupt Controller.
    744744     */
    745     rc = CFGMR3InsertNode(pDevices, "i8259", &pDev);                                RC_CHECK();
    746     rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
    747     rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */   RC_CHECK();
    748     rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
     745    rc = CFGMR3InsertNode(pDevices, "i8259", &pDev);                                    RC_CHECK();
     746    rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                       RC_CHECK();
     747    rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */       RC_CHECK();
     748    rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                                   RC_CHECK();
    749749
    750750    /*
    … …  
    753753     *      thus only single insert
    754754     */
    755     rc = CFGMR3InsertNode(pDevices, "apic", &pDev);                                 RC_CHECK();
    756     rc = CFGMR3InsertNode(pDev, "0", &pInst);                                       RC_CHECK();
    757     rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */   RC_CHECK();
    758     rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
    759     rc = CFGMR3InsertInteger(pCfg,  "IOAPIC", fIOAPIC);                             RC_CHECK();
    760     rc = CFGMR3InsertInteger(pCfg,  "NumCPUs", cCpus);                              RC_CHECK();
     755    rc = CFGMR3InsertNode(pDevices, "apic", &pDev);                                     RC_CHECK();
     756    rc = CFGMR3InsertNode(pDev, "0", &pInst);                                           RC_CHECK();
     757    rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */       RC_CHECK();
     758    rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                                   RC_CHECK();
     759    rc = CFGMR3InsertInteger(pCfg,  "IOAPIC", fIOAPIC);                                 RC_CHECK();
     760    rc = CFGMR3InsertInteger(pCfg,  "NumCPUs", cCpus);                                  RC_CHECK();
    761761
    762762    if (fIOAPIC)
    … …  
    765765         * I/O Advanced Programmable Interrupt Controller.
    766766         */
    767         rc = CFGMR3InsertNode(pDevices, "ioapic", &pDev);                           RC_CHECK();
    768         rc = CFGMR3InsertNode(pDev,     "0", &pInst);                               RC_CHECK();
    769         rc = CFGMR3InsertInteger(pInst, "Trusted",          1);     /* boolean */   RC_CHECK();
    770         rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                           RC_CHECK();
     767        rc = CFGMR3InsertNode(pDevices, "ioapic", &pDev);                               RC_CHECK();
     768        rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
     769        rc = CFGMR3InsertInteger(pInst, "Trusted",          1);     /* boolean */       RC_CHECK();
     770        rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
    771771    }
    772772
    … …  
    774774     * RTC MC146818.
    775775     */
    776     rc = CFGMR3InsertNode(pDevices, "mc146818", &pDev);                             RC_CHECK();
    777     rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
    778     rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
     776    rc = CFGMR3InsertNode(pDevices, "mc146818", &pDev);                                 RC_CHECK();
     777    rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                       RC_CHECK();
     778    rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                                   RC_CHECK();
    779779    BOOL fRTCUseUTC;
    780     hrc = pMachine->COMGETTER(RTCUseUTC)(&fRTCUseUTC);                              H();
    781     rc = CFGMR3InsertInteger(pCfg,  "UseUTC", fRTCUseUTC ? 1 : 0);                  RC_CHECK();
     780    hrc = pMachine->COMGETTER(RTCUseUTC)(&fRTCUseUTC);                                  H();
     781    rc = CFGMR3InsertInteger(pCfg,  "UseUTC", fRTCUseUTC ? 1 : 0);                      RC_CHECK();
    782782
    783783    /*
    784784     * VGA.
    785785     */
    786     rc = CFGMR3InsertNode(pDevices, "vga", &pDev);                                  RC_CHECK();
    787     rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
    788     rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */   RC_CHECK();
    789     rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo",          2);                     RC_CHECK();
     786    rc = CFGMR3InsertNode(pDevices, "vga", &pDev);                                      RC_CHECK();
     787    rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                       RC_CHECK();
     788    rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */       RC_CHECK();
     789    rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo",          2);                         RC_CHECK();
    790790    Assert(!afPciDeviceNo[2]);
    791791    afPciDeviceNo[2] = true;
    792     rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",        0);                     RC_CHECK();
    793     rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
     792    rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",        0);                         RC_CHECK();
     793    rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                                   RC_CHECK();
    794794    ULONG cVRamMBs;
    795     hrc = pMachine->COMGETTER(VRAMSize)(&cVRamMBs);                                 H();
    796     rc = CFGMR3InsertInteger(pCfg,  "VRamSize",             cVRamMBs * _1M);        RC_CHECK();
     795    hrc = pMachine->COMGETTER(VRAMSize)(&cVRamMBs);                                     H();
     796    rc = CFGMR3InsertInteger(pCfg,  "VRamSize",             cVRamMBs * _1M);            RC_CHECK();
    797797    ULONG cMonitorCount;
    798     hrc = pMachine->COMGETTER(MonitorCount)(&cMonitorCount);                        H();
    799     rc = CFGMR3InsertInteger(pCfg,  "MonitorCount",         cMonitorCount);         RC_CHECK();
    800 #ifdef VBOX_WITH_2X_4GB_ADDR_SPACE /* not safe here yet. */ /** @todo this needs fixing !!! No wonder VGA is slooooooooow on 32-bit darwin! */
    801     rc = CFGMR3InsertInteger(pCfg,  "R0Enabled",            fHWVirtExEnabled);      RC_CHECK();
     798    hrc = pMachine->COMGETTER(MonitorCount)(&cMonitorCount);                            H();
     799    rc = CFGMR3InsertInteger(pCfg,  "MonitorCount",         cMonitorCount);             RC_CHECK();
     800#ifdef VBOX_WITH_2X_4GB_ADDR_SPACE
     801    rc = CFGMR3InsertInteger(pCfg,  "R0Enabled",            fHWVirtExEnabled);          RC_CHECK();
    802802#endif
    803803
    … …  
    806806     */
    807807    BOOL fFadeIn;
    808     hrc = biosSettings->COMGETTER(LogoFadeIn)(&fFadeIn);                            H();
    809     rc = CFGMR3InsertInteger(pCfg,  "FadeIn",  fFadeIn ? 1 : 0);                    RC_CHECK();
     808    hrc = biosSettings->COMGETTER(LogoFadeIn)(&fFadeIn);                                H();
     809    rc = CFGMR3InsertInteger(pCfg,  "FadeIn",  fFadeIn ? 1 : 0);                        RC_CHECK();
    810810    BOOL fFadeOut;
    811     hrc = biosSettings->COMGETTER(LogoFadeOut)(&fFadeOut);                          H();
    812     rc = CFGMR3InsertInteger(pCfg,  "FadeOut", fFadeOut ? 1: 0);                    RC_CHECK();
     811    hrc = biosSettings->COMGETTER(LogoFadeOut)(&fFadeOut);                              H();
     812    rc = CFGMR3InsertInteger(pCfg,  "FadeOut", fFadeOut ? 1: 0);                        RC_CHECK();
    813813    ULONG logoDisplayTime;
    814     hrc = biosSettings->COMGETTER(LogoDisplayTime)(&logoDisplayTime);               H();
    815     rc = CFGMR3InsertInteger(pCfg,  "LogoTime", logoDisplayTime);                   RC_CHECK();
     814    hrc = biosSettings->COMGETTER(LogoDisplayTime)(&logoDisplayTime);                   H();
     815    rc = CFGMR3InsertInteger(pCfg,  "LogoTime", logoDisplayTime);                       RC_CHECK();
    816816    Bstr logoImagePath;
    817     hrc = biosSettings->COMGETTER(LogoImagePath)(logoImagePath.asOutParam());       H();
     817    hrc = biosSettings->COMGETTER(LogoImagePath)(logoImagePath.asOutParam());           H();
    818818    rc = CFGMR3InsertString(pCfg,   "LogoFile", logoImagePath ? Utf8Str(logoImagePath).c_str() : ""); RC_CHECK();
    819819
    … …  
    830830        default:                        iShowBootMenu = 2;  break;
    831831    }
    832     rc = CFGMR3InsertInteger(pCfg, "ShowBootMenu", iShowBootMenu);                  RC_CHECK();
     832    rc = CFGMR3InsertInteger(pCfg, "ShowBootMenu", iShowBootMenu);                      RC_CHECK();
    833833
    834834    /* Custom VESA mode list */
    … …  
    838838        char szExtraDataKey[sizeof("CustomVideoModeXX")];
    839839        RTStrPrintf(szExtraDataKey, sizeof(szExtraDataKey), "CustomVideoMode%u", iMode);
    840         hrc = pMachine->GetExtraData(Bstr(szExtraDataKey), bstr.asOutParam());     H();
     840        hrc = pMachine->GetExtraData(Bstr(szExtraDataKey), bstr.asOutParam());          H();
    841841        if (bstr.isEmpty())
    842842            break;
    843         rc = CFGMR3InsertStringW(pCfg, szExtraDataKey, bstr.raw());                RC_CHECK();
     843        rc = CFGMR3InsertStringW(pCfg, szExtraDataKey, bstr.raw());                     RC_CHECK();
    844844        ++cModes;
    845845    }
    846     rc = CFGMR3InsertInteger(pCfg,  "CustomVideoModes", cModes);                    RC_CHECK();
     846    rc = CFGMR3InsertInteger(pCfg,  "CustomVideoModes", cModes);                        RC_CHECK();
    847847
    848848    /* VESA height reduction */
    … …  
    851851    if (pFramebuffer)
    852852    {
    853         hrc = pFramebuffer->COMGETTER(HeightReduction)(&ulHeightReduction);         H();
     853        hrc = pFramebuffer->COMGETTER(HeightReduction)(&ulHeightReduction);             H();
    854854    }
    855855    else
    … …  
    858858        ulHeightReduction = 0;
    859859    }
    860     rc = CFGMR3InsertInteger(pCfg,  "HeightReduction", ulHeightReduction);          RC_CHECK();
     860    rc = CFGMR3InsertInteger(pCfg,  "HeightReduction", ulHeightReduction);              RC_CHECK();
    861861
    862862    /* Attach the display. */
    863     rc = CFGMR3InsertNode(pInst,    "LUN#0", &pLunL0);                              RC_CHECK();
    864     rc = CFGMR3InsertString(pLunL0, "Driver",               "MainDisplay");         RC_CHECK();
    865     rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
     863    rc = CFGMR3InsertNode(pInst,    "LUN#0", &pLunL0);                                  RC_CHECK();
     864    rc = CFGMR3InsertString(pLunL0, "Driver",               "MainDisplay");             RC_CHECK();
     865    rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                                   RC_CHECK();
    866866    Display *pDisplay = pConsole->mDisplay;
    867     rc = CFGMR3InsertInteger(pCfg,  "Object", (uintptr_t)pDisplay);                 RC_CHECK();
     867    rc = CFGMR3InsertInteger(pCfg,  "Object", (uintptr_t)pDisplay);                     RC_CHECK();
    868868
    869869
    … …  
    872872     */
    873873    FirmwareType_T eFwType =  FirmwareType_BIOS;
    874     hrc = pMachine->COMGETTER(FirmwareType)(&eFwType);                                H();
     874    hrc = pMachine->COMGETTER(FirmwareType)(&eFwType);                                  H();
    875875
    876876#ifdef VBOX_WITH_EFI
    … …  
    983983         * EFI subtree.
    984984         */
    985         rc = CFGMR3InsertNode(pDevices, "efi", &pDev);                                   RC_CHECK();
    986         rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                    RC_CHECK();
    987         rc = CFGMR3InsertInteger(pInst, "Trusted", 1);              /* boolean */        RC_CHECK();
    988         rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                                RC_CHECK();
    989         rc = CFGMR3InsertInteger(pCfg,  "RamSize",          cbRam);                      RC_CHECK();
    990         rc = CFGMR3InsertInteger(pCfg,  "RamHoleSize",      cbRamHole);                  RC_CHECK();
    991         rc = CFGMR3InsertInteger(pCfg,  "NumCPUs",          cCpus);                      RC_CHECK();
    992         rc = CFGMR3InsertString(pCfg,   "EfiRom",           efiRomFile.raw());           RC_CHECK();
    993         rc = CFGMR3InsertString(pCfg,   "BootArgs",         Utf8Str(bootArgs).raw());    RC_CHECK();
    994         rc = CFGMR3InsertString(pCfg,   "DeviceProps",      Utf8Str(deviceProps).raw()); RC_CHECK();
    995         rc = CFGMR3InsertInteger(pCfg,  "IOAPIC",           fIOAPIC);                    RC_CHECK();
    996         rc = CFGMR3InsertBytes(pCfg,    "UUID", &HardwareUuid,sizeof(HardwareUuid));     RC_CHECK();
    997         rc = CFGMR3InsertInteger(pCfg,  "64BitEntry", f64BitEntry); /* boolean */        RC_CHECK();
    998         rc = CFGMR3InsertInteger(pCfg,  "GopMode", u32GopMode);                          RC_CHECK();
    999         rc = CFGMR3InsertInteger(pCfg,  "UgaHorizontalResolution", u32UgaHorisontal);    RC_CHECK();
    1000         rc = CFGMR3InsertInteger(pCfg,  "UgaVerticalResolution", u32UgaVertical);        RC_CHECK();
     985        rc = CFGMR3InsertNode(pDevices, "efi", &pDev);                                  RC_CHECK();
     986        rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
     987        rc = CFGMR3InsertInteger(pInst, "Trusted", 1);              /* boolean */       RC_CHECK();
     988        rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
     989        rc = CFGMR3InsertInteger(pCfg,  "RamSize",          cbRam);                     RC_CHECK();
     990        rc = CFGMR3InsertInteger(pCfg,  "RamHoleSize",      cbRamHole);                 RC_CHECK();
     991        rc = CFGMR3InsertInteger(pCfg,  "NumCPUs",          cCpus);                     RC_CHECK();
     992        rc = CFGMR3InsertString(pCfg,   "EfiRom",           efiRomFile.raw());          RC_CHECK();
     993        rc = CFGMR3InsertString(pCfg,   "BootArgs",         Utf8Str(bootArgs).raw());   RC_CHECK();
     994        rc = CFGMR3InsertString(pCfg,   "DeviceProps",      Utf8Str(deviceProps).raw());RC_CHECK();
     995        rc = CFGMR3InsertInteger(pCfg,  "IOAPIC",           fIOAPIC);                   RC_CHECK();
     996        rc = CFGMR3InsertBytes(pCfg,    "UUID", &HardwareUuid,sizeof(HardwareUuid));    RC_CHECK();
     997        rc = CFGMR3InsertInteger(pCfg,  "64BitEntry", f64BitEntry); /* boolean */       RC_CHECK();
     998        rc = CFGMR3InsertInteger(pCfg,  "GopMode", u32GopMode);                         RC_CHECK();
     999        rc = CFGMR3InsertInteger(pCfg,  "UgaHorizontalResolution", u32UgaHorisontal);   RC_CHECK();
     1000        rc = CFGMR3InsertInteger(pCfg,  "UgaVerticalResolution", u32UgaVertical);       RC_CHECK();
    10011001
    10021002        /* For OS X guests we'll force passing host's DMI info to the guest */
    10031003        if (fOsXGuest)
    10041004        {
    1005             rc = CFGMR3InsertInteger(pCfg,  "DmiUseHostInfo", 1);                        RC_CHECK();
    1006             rc = CFGMR3InsertInteger(pCfg,  "DmiExposeMemoryTable", 1);                  RC_CHECK();
     1005            rc = CFGMR3InsertInteger(pCfg,  "DmiUseHostInfo", 1);                       RC_CHECK();
     1006            rc = CFGMR3InsertInteger(pCfg,  "DmiExposeMemoryTable", 1);                 RC_CHECK();
    10071007        }
    10081008    }
    … …  
    10131013    com::SafeIfaceArray<IStorageController> ctrls;
    10141014    PCFGMNODE aCtrlNodes[StorageControllerType_LsiLogicSas + 1] = {};
    1015     hrc = pMachine->COMGETTER(StorageControllers)(ComSafeArrayAsOutParam(ctrls));               H();
     1015    hrc = pMachine->COMGETTER(StorageControllers)(ComSafeArrayAsOutParam(ctrls));       H();
    10161016
    10171017    for (size_t i = 0; i < ctrls.size(); ++i)
    … …  
    10201020
    10211021        StorageControllerType_T enmCtrlType;
    1022         rc = ctrls[i]->COMGETTER(ControllerType)(&enmCtrlType);                                 H();
     1022        rc = ctrls[i]->COMGETTER(ControllerType)(&enmCtrlType);                         H();
    10231023        AssertRelease((unsigned)enmCtrlType < RT_ELEMENTS(aCtrlNodes));
    10241024
    10251025        StorageBus_T enmBus;
    1026         rc = ctrls[i]->COMGETTER(Bus)(&enmBus);                                                 H();
     1026        rc = ctrls[i]->COMGETTER(Bus)(&enmBus);                                         H();
    10271027
    10281028        Bstr controllerName;
    1029         rc = ctrls[i]->COMGETTER(Name)(controllerName.asOutParam());                            H();
     1029        rc = ctrls[i]->COMGETTER(Name)(controllerName.asOutParam());                    H();
    10301030
    10311031        ULONG ulInstance = 999;
    1032         rc = ctrls[i]->COMGETTER(Instance)(&ulInstance);                                        H();
     1032        rc = ctrls[i]->COMGETTER(Instance)(&ulInstance);                                H();
    10331033
    10341034        IoBackendType_T enmIoBackend;
    1035         rc = ctrls[i]->COMGETTER(IoBackend)(&enmIoBackend);                                     H();
     1035        rc = ctrls[i]->COMGETTER(IoBackend)(&enmIoBackend);                             H();
    10361036
    10371037        /* /Devices/<ctrldev>/ */
    … …  
    10401040        if (!pDev)
    10411041        {
    1042             rc = CFGMR3InsertNode(pDevices, pszCtrlDev, &pDev);                                 RC_CHECK();
     1042            rc = CFGMR3InsertNode(pDevices, pszCtrlDev, &pDev);                         RC_CHECK();
    10431043            aCtrlNodes[enmCtrlType] = pDev; /* IDE variants are handled in the switch */
    10441044        }
    … …  
    10461046        /* /Devices/<ctrldev>/<instance>/ */
    10471047        PCFGMNODE pCtlInst = NULL;
    1048         rc = CFGMR3InsertNodeF(pDev, &pCtlInst, "%u", ulInstance);                              RC_CHECK();
     1048        rc = CFGMR3InsertNodeF(pDev, &pCtlInst, "%u", ulInstance);                      RC_CHECK();
    10491049
    10501050        /* Device config: /Devices/<ctrldev>/<instance>/<values> & /ditto/Config/<values> */
    1051         rc = CFGMR3InsertInteger(pCtlInst, "Trusted",   1);                                     RC_CHECK();
    1052         rc = CFGMR3InsertNode(pCtlInst,    "Config",    &pCfg);                                 RC_CHECK();
     1051        rc = CFGMR3InsertInteger(pCtlInst, "Trusted",   1);                             RC_CHECK();
     1052        rc = CFGMR3InsertNode(pCtlInst,    "Config",    &pCfg);                         RC_CHECK();
    10531053
    10541054        switch (enmCtrlType)
    … …  
    10561056            case StorageControllerType_LsiLogic:
    10571057            {
    1058                 rc = CFGMR3InsertInteger(pCtlInst, "PCIDeviceNo",          20);                 RC_CHECK();
     1058                rc = CFGMR3InsertInteger(pCtlInst, "PCIDeviceNo",          20);         RC_CHECK();
    10591059                Assert(!afPciDeviceNo[20]);
    10601060                afPciDeviceNo[20] = true;
    1061                 rc = CFGMR3InsertInteger(pCtlInst, "PCIFunctionNo",        0);                  RC_CHECK();
     1061                rc = CFGMR3InsertInteger(pCtlInst, "PCIFunctionNo",        0);          RC_CHECK();
    10621062
    10631063                /* Attach the status driver */
    1064                 rc = CFGMR3InsertNode(pCtlInst, "LUN#999", &pLunL0);                            RC_CHECK();
    1065                 rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");          RC_CHECK();
    1066                 rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
     1064                rc = CFGMR3InsertNode(pCtlInst, "LUN#999", &pLunL0);                    RC_CHECK();
     1065                rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");  RC_CHECK();
     1066                rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                       RC_CHECK();
    10671067                rc = CFGMR3InsertInteger(pCfg,  "papLeds", (uintptr_t)&pConsole->mapStorageLeds[iLedScsi]); RC_CHECK();
    1068                 rc = CFGMR3InsertInteger(pCfg,  "First",    0);                                 RC_CHECK();
     1068                rc = CFGMR3InsertInteger(pCfg,  "First",    0);                         RC_CHECK();
    10691069                Assert(cLedScsi >= 16);
    1070                 rc = CFGMR3InsertInteger(pCfg,  "Last",     15);                                RC_CHECK();
     1070                rc = CFGMR3InsertInteger(pCfg,  "Last",     15);                        RC_CHECK();
    10711071                paLedDevType = &pConsole->maStorageDevType[iLedScsi];
    10721072                break;
    … …  
    10751075            case StorageControllerType_BusLogic:
    10761076            {
    1077                 rc = CFGMR3InsertInteger(pCtlInst, "PCIDeviceNo",          21);                 RC_CHECK();
     1077                rc = CFGMR3InsertInteger(pCtlInst, "PCIDeviceNo",          21);         RC_CHECK();
    10781078                Assert(!afPciDeviceNo[21]);
    10791079                afPciDeviceNo[21] = true;
    1080                 rc = CFGMR3InsertInteger(pCtlInst, "PCIFunctionNo",        0);                  RC_CHECK();
     1080                rc = CFGMR3InsertInteger(pCtlInst, "PCIFunctionNo",        0);          RC_CHECK();
    10811081
    10821082                /* Attach the status driver */
    1083                 rc = CFGMR3InsertNode(pCtlInst, "LUN#999", &pLunL0);                            RC_CHECK();
    1084                 rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");          RC_CHECK();
    1085                 rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
     1083                rc = CFGMR3InsertNode(pCtlInst, "LUN#999", &pLunL0);                    RC_CHECK();
     1084                rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");  RC_CHECK();
     1085                rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                       RC_CHECK();
    10861086                rc = CFGMR3InsertInteger(pCfg,  "papLeds", (uintptr_t)&pConsole->mapStorageLeds[iLedScsi]); RC_CHECK();
    1087                 rc = CFGMR3InsertInteger(pCfg,  "First",    0);                                 RC_CHECK();
     1087                rc = CFGMR3InsertInteger(pCfg,  "First",    0);                         RC_CHECK();
    10881088                Assert(cLedScsi >= 16);
    1089                 rc = CFGMR3InsertInteger(pCfg,  "Last",     15);                                RC_CHECK();
     1089                rc = CFGMR3InsertInteger(pCfg,  "Last",     15);                        RC_CHECK();
    10901090                paLedDevType = &pConsole->maStorageDevType[iLedScsi];
    10911091                break;
    … …  
    10941094            case StorageControllerType_IntelAhci:
    10951095            {
    1096                 rc = CFGMR3InsertInteger(pCtlInst, "PCIDeviceNo",          13);                 RC_CHECK();
     1096                rc = CFGMR3InsertInteger(pCtlInst, "PCIDeviceNo",          13);         RC_CHECK();
    10971097                Assert(!afPciDeviceNo[13]);
    10981098                afPciDeviceNo[13] = true;
    1099                 rc = CFGMR3InsertInteger(pCtlInst, "PCIFunctionNo",        0);                  RC_CHECK();
     1099                rc = CFGMR3InsertInteger(pCtlInst, "PCIFunctionNo",        0);          RC_CHECK();
    11001100
    11011101                ULONG cPorts = 0;
    1102                 hrc = ctrls[i]->COMGETTER(PortCount)(&cPorts);                                  H();
    1103                 rc = CFGMR3InsertInteger(pCfg, "PortCount", cPorts);                            RC_CHECK();
     1102                hrc = ctrls[i]->COMGETTER(PortCount)(&cPorts);                          H();
     1103                rc = CFGMR3InsertInteger(pCfg, "PortCount", cPorts);                    RC_CHECK();
    11041104
    11051105                /* Needed configuration values for the bios. */
    11061106                if (pBiosCfg)
    11071107                {
    1108                     rc = CFGMR3InsertString(pBiosCfg, "SataHardDiskDevice", "ahci");            RC_CHECK();
     1108                    rc = CFGMR3InsertString(pBiosCfg, "SataHardDiskDevice", "ahci");    RC_CHECK();
    11091109                }
    11101110
    … …  
    11171117
    11181118                    LONG lPortNumber = -1;
    1119                     hrc = ctrls[i]->GetIDEEmulationPort(j, &lPortNumber);                       H();
    1120                     rc = CFGMR3InsertInteger(pCfg, s_apszConfig[j], lPortNumber);               RC_CHECK();
     1119                    hrc = ctrls[i]->GetIDEEmulationPort(j, &lPortNumber);               H();
     1120                    rc = CFGMR3InsertInteger(pCfg, s_apszConfig[j], lPortNumber);       RC_CHECK();
    11211121                    if (pBiosCfg)
    11221122                    {
    1123                         rc = CFGMR3InsertInteger(pBiosCfg, s_apszBiosConfig[j], lPortNumber);   RC_CHECK();
     1123                        rc = CFGMR3InsertInteger(pBiosCfg, s_apszBiosConfig[j], lPortNumber); RC_CHECK();
    11241124                    }
    11251125                }
    11261126
    11271127                /* Attach the status driver */
    1128                 rc = CFGMR3InsertNode(pCtlInst, "LUN#999", &pLunL0);                            RC_CHECK();
    1129                 rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");          RC_CHECK();
    1130                 rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
     1128                rc = CFGMR3InsertNode(pCtlInst, "LUN#999", &pLunL0);                    RC_CHECK();
     1129                rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");  RC_CHECK();
     1130                rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                       RC_CHECK();
    11311131                AssertRelease(cPorts <= cLedSata);
    11321132                rc = CFGMR3InsertInteger(pCfg,  "papLeds", (uintptr_t)&pConsole->mapStorageLeds[iLedSata]); RC_CHECK();
    1133                 rc = CFGMR3InsertInteger(pCfg,  "First",    0);                                 RC_CHECK();
    1134                 rc = CFGMR3InsertInteger(pCfg,  "Last",     cPorts - 1);                        RC_CHECK();
     1133                rc = CFGMR3InsertInteger(pCfg,  "First",    0);                         RC_CHECK();
     1134                rc = CFGMR3InsertInteger(pCfg,  "Last",     cPorts - 1);                RC_CHECK();
    11351135                paLedDevType = &pConsole->maStorageDevType[iLedSata];
    11361136                break;
    … …  
    11441144                 * IDE (update this when the main interface changes)
    11451145                 */
    1146                 rc = CFGMR3InsertInteger(pCtlInst, "PCIDeviceNo",          1);                  RC_CHECK();
     1146                rc = CFGMR3InsertInteger(pCtlInst, "PCIDeviceNo",          1);          RC_CHECK();
    11471147                Assert(!afPciDeviceNo[1]);
    11481148                afPciDeviceNo[1] = true;
    1149                 rc = CFGMR3InsertInteger(pCtlInst, "PCIFunctionNo",        1);                  RC_CHECK();
    1150                 rc = CFGMR3InsertString(pCfg,  "Type", controllerString(enmCtrlType));          RC_CHECK();
     1149                rc = CFGMR3InsertInteger(pCtlInst, "PCIFunctionNo",        1);          RC_CHECK();
     1150                rc = CFGMR3InsertString(pCfg,  "Type", controllerString(enmCtrlType));  RC_CHECK();
    11511151
    11521152                /* Attach the status driver */
    1153                 rc = CFGMR3InsertNode(pCtlInst,    "LUN#999", &pLunL0);                         RC_CHECK();
    1154                 rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");          RC_CHECK();
    1155                 rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
     1153                rc = CFGMR3InsertNode(pCtlInst,    "LUN#999", &pLunL0);                 RC_CHECK();
     1154                rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");  RC_CHECK();
     1155                rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                       RC_CHECK();
    11561156                rc = CFGMR3InsertInteger(pCfg,  "papLeds", (uintptr_t)&pConsole->mapStorageLeds[iLedIde]); RC_CHECK();
    1157                 rc = CFGMR3InsertInteger(pCfg,  "First",    0);                                 RC_CHECK();
     1157                rc = CFGMR3InsertInteger(pCfg,  "First",    0);                         RC_CHECK();
    11581158                Assert(cLedIde >= 4);
    1159                 rc = CFGMR3InsertInteger(pCfg,  "Last",     3);                                 RC_CHECK();
     1159                rc = CFGMR3InsertInteger(pCfg,  "Last",     3);                         RC_CHECK();
    11601160                paLedDevType = &pConsole->maStorageDevType[iLedIde];
    11611161
    … …  
    11731173                 */
    11741174                fFdcEnabled = true;
    1175                 rc = CFGMR3InsertInteger(pCfg,  "IRQ",       6);                                RC_CHECK();
    1176                 rc = CFGMR3InsertInteger(pCfg,  "DMA",       2);                                RC_CHECK();
    1177                 rc = CFGMR3InsertInteger(pCfg,  "MemMapped", 0 );                               RC_CHECK();
    1178                 rc = CFGMR3InsertInteger(pCfg,  "IOBase",    0x3f0);                            RC_CHECK();
     1175                rc = CFGMR3InsertInteger(pCfg,  "IRQ",       6);                        RC_CHECK();
     1176                rc = CFGMR3InsertInteger(pCfg,  "DMA",       2);                        RC_CHECK();
     1177                rc = CFGMR3InsertInteger(pCfg,  "MemMapped", 0 );                       RC_CHECK();
     1178                rc = CFGMR3InsertInteger(pCfg,  "IOBase",    0x3f0);                    RC_CHECK();
    11791179
    11801180                /* Attach the status driver */
    1181                 rc = CFGMR3InsertNode(pCtlInst, "LUN#999", &pLunL0);                            RC_CHECK();
    1182                 rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");          RC_CHECK();
    1183                 rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
     1181                rc = CFGMR3InsertNode(pCtlInst, "LUN#999", &pLunL0);                    RC_CHECK();
     1182                rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");  RC_CHECK();
     1183                rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                       RC_CHECK();
    11841184                rc = CFGMR3InsertInteger(pCfg,  "papLeds", (uintptr_t)&pConsole->mapStorageLeds[iLedFloppy]); RC_CHECK();
    1185                 rc = CFGMR3InsertInteger(pCfg,  "First",    0);                                 RC_CHECK();
     1185                rc = CFGMR3InsertInteger(pCfg,  "First",    0);                         RC_CHECK();
    11861186                Assert(cLedFloppy >= 1);
    1187                 rc = CFGMR3InsertInteger(pCfg,  "Last",     0);                                 RC_CHECK();
     1187                rc = CFGMR3InsertInteger(pCfg,  "Last",     0);                         RC_CHECK();
    11881188                paLedDevType = &pConsole->maStorageDevType[iLedFloppy];
    11891189                break;
    … …  
    11921192            case StorageControllerType_LsiLogicSas:
    11931193            {
    1194                 rc = CFGMR3InsertInteger(pCtlInst, "PCIDeviceNo",          22);                 RC_CHECK();
     1194                rc = CFGMR3InsertInteger(pCtlInst, "PCIDeviceNo",          22);         RC_CHECK();
    11951195                Assert(!afPciDeviceNo[22]);
    11961196                afPciDeviceNo[22] = true;
    1197                 rc = CFGMR3InsertInteger(pCtlInst, "PCIFunctionNo",        0);                  RC_CHECK();
    1198 
    1199                 rc = CFGMR3InsertString(pCfg,  "ControllerType", "SAS1068");                    RC_CHECK();
     1197                rc = CFGMR3InsertInteger(pCtlInst, "PCIFunctionNo",        0);          RC_CHECK();
     1198
     1199                rc = CFGMR3InsertString(pCfg,  "ControllerType", "SAS1068");            RC_CHECK();
    12001200
    12011201                /* Attach the status driver */
    1202                 rc = CFGMR3InsertNode(pCtlInst, "LUN#999", &pLunL0);                            RC_CHECK();
    1203                 rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");          RC_CHECK();
    1204                 rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
     1202                rc = CFGMR3InsertNode(pCtlInst, "LUN#999", &pLunL0);                    RC_CHECK();
     1203                rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");  RC_CHECK();
     1204                rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                       RC_CHECK();
    12051205                rc = CFGMR3InsertInteger(pCfg,  "papLeds", (uintptr_t)&pConsole->mapStorageLeds[iLedScsi]); RC_CHECK();
    1206                 rc = CFGMR3InsertInteger(pCfg,  "First",    0);                                 RC_CHECK();
     1206                rc = CFGMR3InsertInteger(pCfg,  "First",    0);                         RC_CHECK();
    12071207                Assert(cLedScsi >= 16);
    1208                 rc = CFGMR3InsertInteger(pCfg,  "Last",     15)        ;                        RC_CHECK();
     1208                rc = CFGMR3InsertInteger(pCfg,  "Last",     15)        ;                RC_CHECK();
    12091209                paLedDevType = &pConsole->maStorageDevType[iLedScsi];
    12101210                break;
    … …  
    12181218        com::SafeIfaceArray<IMediumAttachment> atts;
    12191219        hrc = pMachine->GetMediumAttachmentsOfController(controllerName,
    1220                                                          ComSafeArrayAsOutParam(atts));         H();
     1220                                                         ComSafeArrayAsOutParam(atts)); H();
    12211221
    12221222        for (size_t j = 0; j < atts.size(); ++j)
    … …  
    12361236                                                  false /* fForceUnmount */,
    12371237                                                  NULL /* pVM */,
    1238                                                   paLedDevType);                                 RC_CHECK();
     1238                                                  paLedDevType);                        RC_CHECK();
    12391239        }
    12401240        H();
    … …  
    12491249#endif
    12501250    PCFGMNODE pDevPCNet = NULL;          /* PCNet-type devices */
    1251     rc = CFGMR3InsertNode(pDevices, "pcnet", &pDevPCNet);                           RC_CHECK();
     1251    rc = CFGMR3InsertNode(pDevices, "pcnet", &pDevPCNet);                               RC_CHECK();
    12521252#ifdef VBOX_WITH_E1000
    12531253    PCFGMNODE pDevE1000 = NULL;          /* E1000-type devices */
    1254     rc = CFGMR3InsertNode(pDevices, "e1000", &pDevE1000);                           RC_CHECK();
     1254    rc = CFGMR3InsertNode(pDevices, "e1000", &pDevE1000);                               RC_CHECK();
    12551255#endif
    12561256#ifdef VBOX_WITH_VIRTIO
    12571257    PCFGMNODE pDevVirtioNet = NULL;          /* Virtio network devices */
    1258     rc = CFGMR3InsertNode(pDevices, "virtio-net", &pDevVirtioNet);                  RC_CHECK();
     1258    rc = CFGMR3InsertNode(pDevices, "virtio-net", &pDevVirtioNet);                      RC_CHECK();
    12591259#endif /* VBOX_WITH_VIRTIO */
    12601260    std::list<BootNic> llBootNics;
    … …  
    12621262    {
    12631263        ComPtr<INetworkAdapter> networkAdapter;
    1264         hrc = pMachine->GetNetworkAdapter(ulInstance, networkAdapter.asOutParam()); H();
     1264        hrc = pMachine->GetNetworkAdapter(ulInstance, networkAdapter.asOutParam());     H();
    12651265        BOOL fEnabled = FALSE;
    1266         hrc = networkAdapter->COMGETTER(Enabled)(&fEnabled);                        H();
     1266        hrc = networkAdapter->COMGETTER(Enabled)(&fEnabled);                            H();
    12671267        if (!fEnabled)
    12681268            continue;
    … …  
    12731273        const char *pszAdapterName = "pcnet";
    12741274        NetworkAdapterType_T adapterType;
    1275         hrc = networkAdapter->COMGETTER(AdapterType)(&adapterType);                 H();
     1275        hrc = networkAdapter->COMGETTER(AdapterType)(&adapterType);                     H();
    12761276        switch (adapterType)
    12771277        {
    … …  
    13021302        }
    13031303
    1304         rc = CFGMR3InsertNodeF(pDev, &pInst, "%u", ulInstance);                     RC_CHECK();
    1305         rc = CFGMR3InsertInteger(pInst, "Trusted",              1); /* boolean */   RC_CHECK();
     1304        rc = CFGMR3InsertNodeF(pDev, &pInst, "%u", ulInstance);                         RC_CHECK();
     1305        rc = CFGMR3InsertInteger(pInst, "Trusted",              1); /* boolean */       RC_CHECK();
    13061306        /* the first network card gets the PCI ID 3, the next 3 gets 8..10,
    13071307         * next 4 get 16..19. */
    … …  
    13271327            iPciDeviceNo = 3;
    13281328#endif
    1329         rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo", iPciDeviceNo);               RC_CHECK();
     1329        rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo", iPciDeviceNo);                   RC_CHECK();
    13301330        Assert(!afPciDeviceNo[iPciDeviceNo]);
    13311331        afPciDeviceNo[iPciDeviceNo] = true;
    1332         rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",        0);                 RC_CHECK();
    1333         rc = CFGMR3InsertNode(pInst, "Config", &pCfg);                              RC_CHECK();
     1332        rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",        0);                     RC_CHECK();
     1333        rc = CFGMR3InsertNode(pInst, "Config", &pCfg);                                  RC_CHECK();
    13341334#ifdef VBOX_WITH_2X_4GB_ADDR_SPACE /* not safe here yet. */
    13351335        if (pDev == pDevPCNet)
    13361336        {
    1337             rc = CFGMR3InsertInteger(pCfg,  "R0Enabled",    false);                 RC_CHECK();
     1337            rc = CFGMR3InsertInteger(pCfg,  "R0Enabled",    false);                     RC_CHECK();
    13381338        }
    13391339#endif
    … …  
    13471347        nic.mPciFn    = 0;
    13481348
    1349         hrc = networkAdapter->COMGETTER(BootPriority)(&nic.mBootPrio);              H();
     1349        hrc = networkAdapter->COMGETTER(BootPriority)(&nic.mBootPrio);                  H();
    13501350
    13511351        llBootNics.push_back(nic);
    … …  
    13571357        {
    13581358            case NetworkAdapterType_Am79C970A:
    1359                 rc = CFGMR3InsertInteger(pCfg, "Am79C973", 0);                      RC_CHECK();
     1359                rc = CFGMR3InsertInteger(pCfg, "Am79C973", 0);                          RC_CHECK();
    13601360                break;
    13611361            case NetworkAdapterType_Am79C973:
    1362                 rc = CFGMR3InsertInteger(pCfg, "Am79C973", 1);                      RC_CHECK();
     1362                rc = CFGMR3InsertInteger(pCfg, "Am79C973", 1);                          RC_CHECK();
    13631363                break;
    13641364            case NetworkAdapterType_I82540EM:
    1365                 rc = CFGMR3InsertInteger(pCfg, "AdapterType", 0);                   RC_CHECK();
     1365                rc = CFGMR3InsertInteger(pCfg, "AdapterType", 0);                       RC_CHECK();
    13661366                break;
    13671367            case NetworkAdapterType_I82543GC:
    1368                 rc = CFGMR3InsertInteger(pCfg, "AdapterType", 1);                   RC_CHECK();
     1368                rc = CFGMR3InsertInteger(pCfg, "AdapterType", 1);                       RC_CHECK();
    13691369                break;
    13701370            case NetworkAdapterType_I82545EM:
    1371                 rc = CFGMR3InsertInteger(pCfg, "AdapterType", 2);                   RC_CHECK();
     1371                rc = CFGMR3InsertInteger(pCfg, "AdapterType", 2);                       RC_CHECK();
    13721372                break;
    13731373        }
    … …  
    13771377         */
    13781378        Bstr macAddr;
    1379         hrc = networkAdapter->COMGETTER(MACAddress)(macAddr.asOutParam());          H();
     1379        hrc = networkAdapter->COMGETTER(MACAddress)(macAddr.asOutParam());              H();
    13801380        Assert(macAddr);
    13811381        Utf8Str macAddrUtf8 = macAddr;
    … …  
    13951395            *pMac++ = ((c1 & 0x0f) << 4) | (c2 & 0x0f);
    13961396        }
    1397         rc = CFGMR3InsertBytes(pCfg, "MAC", &Mac, sizeof(Mac));                     RC_CHECK();
     1397        rc = CFGMR3InsertBytes(pCfg, "MAC", &Mac, sizeof(Mac));                         RC_CHECK();
    13981398
    13991399        /*
    … …  
    14011401         */
    14021402        BOOL fCableConnected;
    1403         hrc = networkAdapter->COMGETTER(CableConnected)(&fCableConnected);          H();
    1404         rc = CFGMR3InsertInteger(pCfg, "CableConnected", fCableConnected ? 1 : 0);  RC_CHECK();
     1403        hrc = networkAdapter->COMGETTER(CableConnected)(&fCableConnected);              H();
     1404        rc = CFGMR3InsertInteger(pCfg, "CableConnected", fCableConnected ? 1 : 0);      RC_CHECK();
    14051405
    14061406        /*
    … …  
    14081408         */
    14091409        ULONG ulLineSpeed;
    1410         hrc = networkAdapter->COMGETTER(LineSpeed)(&ulLineSpeed);                   H();
    1411         rc = CFGMR3InsertInteger(pCfg, "LineSpeed", ulLineSpeed);                   RC_CHECK();
     1410        hrc = networkAdapter->COMGETTER(LineSpeed)(&ulLineSpeed);                       H();
     1411        rc = CFGMR3InsertInteger(pCfg, "LineSpeed", ulLineSpeed);                       RC_CHECK();
    14121412
    14131413        /*
    14141414         * Attach the status driver.
    14151415         */
    1416         rc = CFGMR3InsertNode(pInst,    "LUN#999", &pLunL0);                        RC_CHECK();
    1417         rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");      RC_CHECK();
    1418         rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                           RC_CHECK();
     1416        rc = CFGMR3InsertNode(pInst,    "LUN#999", &pLunL0);                            RC_CHECK();
     1417        rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");          RC_CHECK();
     1418        rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
    14191419        rc = CFGMR3InsertInteger(pCfg,  "papLeds", (uintptr_t)&pConsole->mapNetworkLeds[ulInstance]); RC_CHECK();
    14201420
    … …  
    14241424        rc = pConsole->configNetwork(pszAdapterName, ulInstance, 0,
    14251425                                     networkAdapter, pCfg, pLunL0, pInst,
    1426                                      false /*fAttachDetach*/);                      RC_CHECK();
     1426                                     false /*fAttachDetach*/);                          RC_CHECK();
    14271427    }
    14281428
    … …  
    14451445            PCFGMNODE pNetBtDevCfg;
    14461446            achBootIdx[0] = '0' + uBootIdx++;   /* Boot device order. */
    1447             rc = CFGMR3InsertNode(pNetBootCfg, achBootIdx, &pNetBtDevCfg);          RC_CHECK();
    1448             rc = CFGMR3InsertInteger(pNetBtDevCfg, "NIC", it->mInstance);           RC_CHECK();
    1449             rc = CFGMR3InsertInteger(pNetBtDevCfg, "PCIDeviceNo", it->mPciDev);     RC_CHECK();
    1450             rc = CFGMR3InsertInteger(pNetBtDevCfg, "PCIFunctionNo", it->mPciFn);    RC_CHECK();
     1447            rc = CFGMR3InsertNode(pNetBootCfg, achBootIdx, &pNetBtDevCfg);              RC_CHECK();
     1448            rc = CFGMR3InsertInteger(pNetBtDevCfg, "NIC", it->mInstance);               RC_CHECK();
     1449            rc = CFGMR3InsertInteger(pNetBtDevCfg, "PCIDeviceNo", it->mPciDev);         RC_CHECK();
     1450            rc = CFGMR3InsertInteger(pNetBtDevCfg, "PCIFunctionNo", it->mPciFn);        RC_CHECK();
    14511451        }
    14521452    }
    … …  
    14551455     * Serial (UART) Ports
    14561456     */
    1457     rc = CFGMR3InsertNode(pDevices, "serial", &pDev);                               RC_CHECK();
     1457    rc = CFGMR3InsertNode(pDevices, "serial", &pDev);                                   RC_CHECK();
    14581458    for (ULONG ulInstance = 0; ulInstance < SchemaDefs::SerialPortCount; ++ulInstance)
    14591459    {
    14601460        ComPtr<ISerialPort> serialPort;
    1461         hrc = pMachine->GetSerialPort(ulInstance, serialPort.asOutParam());        H();
     1461        hrc = pMachine->GetSerialPort(ulInstance, serialPort.asOutParam());             H();
    14621462        BOOL fEnabled = FALSE;
    14631463        if (serialPort)
    1464             hrc = serialPort->COMGETTER(Enabled)(&fEnabled);                        H();
     1464            hrc = serialPort->COMGETTER(Enabled)(&fEnabled);                            H();
    14651465        if (!fEnabled)
    14661466            continue;
    14671467
    1468         rc = CFGMR3InsertNodeF(pDev, &pInst, "%u", ulInstance);                     RC_CHECK();
    1469         rc = CFGMR3InsertNode(pInst, "Config", &pCfg);                              RC_CHECK();
     1468        rc = CFGMR3InsertNodeF(pDev, &pInst, "%u", ulInstance);                         RC_CHECK();
     1469        rc = CFGMR3InsertNode(pInst, "Config", &pCfg);                                  RC_CHECK();
    14701470
    14711471        ULONG ulIRQ;
    1472         hrc = serialPort->COMGETTER(IRQ)(&ulIRQ);                                   H();
    1473         rc = CFGMR3InsertInteger(pCfg,   "IRQ", ulIRQ);                             RC_CHECK();
     1472        hrc = serialPort->COMGETTER(IRQ)(&ulIRQ);                                       H();
     1473        rc = CFGMR3InsertInteger(pCfg,   "IRQ", ulIRQ);                                 RC_CHECK();
    14741474        ULONG ulIOBase;
    1475         hrc = serialPort->COMGETTER(IOBase)(&ulIOBase);                             H();
    1476         rc = CFGMR3InsertInteger(pCfg,   "IOBase", ulIOBase);                       RC_CHECK();
     1475        hrc = serialPort->COMGETTER(IOBase)(&ulIOBase);                                 H();
     1476        rc = CFGMR3InsertInteger(pCfg,   "IOBase", ulIOBase);                           RC_CHECK();
    14771477        BOOL  fServer;
    1478         hrc = serialPort->COMGETTER(Server)(&fServer);                              H();
    1479         hrc = serialPort->COMGETTER(Path)(bstr.asOutParam());                       H();
     1478        hrc = serialPort->COMGETTER(Server)(&fServer);                                  H();
     1479        hrc = serialPort->COMGETTER(Path)(bstr.asOutParam());                           H();
    14801480        PortMode_T eHostMode;
    1481         hrc = serialPort->COMGETTER(HostMode)(&eHostMode);                          H();
     1481        hrc = serialPort->COMGETTER(HostMode)(&eHostMode);                              H();
    14821482        if (eHostMode != PortMode_Disconnected)
    14831483        {
    1484             rc = CFGMR3InsertNode(pInst,     "LUN#0", &pLunL0);                     RC_CHECK();
     1484            rc = CFGMR3InsertNode(pInst,     "LUN#0", &pLunL0);                         RC_CHECK();
    14851485            if (eHostMode == PortMode_HostPipe)
    14861486            {
    1487                 rc = CFGMR3InsertString(pLunL0,  "Driver", "Char");                 RC_CHECK();
    1488                 rc = CFGMR3InsertNode(pLunL0,    "AttachedDriver", &pLunL1);        RC_CHECK();
    1489                 rc = CFGMR3InsertString(pLunL1,  "Driver", "NamedPipe");            RC_CHECK();
    1490                 rc = CFGMR3InsertNode(pLunL1,    "Config", &pLunL2);                RC_CHECK();
    1491                 rc = CFGMR3InsertStringW(pLunL2, "Location", bstr.raw());           RC_CHECK();
    1492                 rc = CFGMR3InsertInteger(pLunL2, "IsServer", fServer);              RC_CHECK();
     1487                rc = CFGMR3InsertString(pLunL0,  "Driver", "Char");                     RC_CHECK();
     1488                rc = CFGMR3InsertNode(pLunL0,    "AttachedDriver", &pLunL1);            RC_CHECK();
     1489                rc = CFGMR3InsertString(pLunL1,  "Driver", "NamedPipe");                RC_CHECK();
     1490                rc = CFGMR3InsertNode(pLunL1,    "Config", &pLunL2);                    RC_CHECK();
     1491                rc = CFGMR3InsertStringW(pLunL2, "Location", bstr.raw());               RC_CHECK();
     1492                rc = CFGMR3InsertInteger(pLunL2, "IsServer", fServer);                  RC_CHECK();
    14931493            }
    14941494            else if (eHostMode == PortMode_HostDevice)
    14951495            {
    1496                 rc = CFGMR3InsertString(pLunL0,  "Driver", "Host Serial");          RC_CHECK();
    1497                 rc = CFGMR3InsertNode(pLunL0,    "Config", &pLunL1);                RC_CHECK();
    1498                 rc = CFGMR3InsertStringW(pLunL1, "DevicePath", bstr.raw());         RC_CHECK();
     1496                rc = CFGMR3InsertString(pLunL0,  "Driver", "Host Serial");              RC_CHECK();
     1497                rc = CFGMR3InsertNode(pLunL0,    "Config", &pLunL1);                    RC_CHECK();
     1498                rc = CFGMR3InsertStringW(pLunL1, "DevicePath", bstr.raw());             RC_CHECK();
    14991499            }
    15001500            else if (eHostMode == PortMode_RawFile)
    15011501            {
    1502                 rc = CFGMR3InsertString(pLunL0,  "Driver", "Char");                 RC_CHECK();
    1503                 rc = CFGMR3InsertNode(pLunL0,    "AttachedDriver", &pLunL1);        RC_CHECK();
    1504                 rc = CFGMR3InsertString(pLunL1,  "Driver", "RawFile");              RC_CHECK();
    1505                 rc = CFGMR3InsertNode(pLunL1,    "Config", &pLunL2);                RC_CHECK();
    1506                 rc = CFGMR3InsertStringW(pLunL2, "Location", bstr.raw());           RC_CHECK();
     1502                rc = CFGMR3InsertString(pLunL0,  "Driver", "Char");                     RC_CHECK();
     1503                rc = CFGMR3InsertNode(pLunL0,    "AttachedDriver", &pLunL1);            RC_CHECK();
     1504                rc = CFGMR3InsertString(pLunL1,  "Driver", "RawFile");                  RC_CHECK();
     1505                rc = CFGMR3InsertNode(pLunL1,    "Config", &pLunL2);                    RC_CHECK();
     1506                rc = CFGMR3InsertStringW(pLunL2, "Location", bstr.raw());               RC_CHECK();
    15071507            }
    15081508        }
    … …  
    15121512     * Parallel (LPT) Ports
    15131513     */
    1514     rc = CFGMR3InsertNode(pDevices, "parallel", &pDev);                             RC_CHECK();
     1514    rc = CFGMR3InsertNode(pDevices, "parallel", &pDev);                                 RC_CHECK();
    15151515    for (ULONG ulInstance = 0; ulInstance < SchemaDefs::ParallelPortCount; ++ulInstance)
    15161516    {
    15171517        ComPtr<IParallelPort> parallelPort;
    1518         hrc = pMachine->GetParallelPort(ulInstance, parallelPort.asOutParam());     H();
     1518        hrc = pMachine->GetParallelPort(ulInstance, parallelPort.asOutParam());         H();
    15191519        BOOL fEnabled = FALSE;
    15201520        if (parallelPort)
    15211521        {
    1522             hrc = parallelPort->COMGETTER(Enabled)(&fEnabled);                      H();
     1522            hrc = parallelPort->COMGETTER(Enabled)(&fEnabled);                          H();
    15231523        }
    15241524        if (!fEnabled)
    15251525            continue;
    15261526
    1527         rc = CFGMR3InsertNodeF(pDev, &pInst, "%u", ulInstance);                     RC_CHECK();
    1528         rc = CFGMR3InsertNode(pInst, "Config", &pCfg);                              RC_CHECK();
     1527        rc = CFGMR3InsertNodeF(pDev, &pInst, "%u", ulInstance);                         RC_CHECK();
     1528        rc = CFGMR3InsertNode(pInst, "Config", &pCfg);                                  RC_CHECK();
    15291529
    15301530        ULONG ulIRQ;
    1531         hrc = parallelPort->COMGETTER(IRQ)(&ulIRQ);                                 H();
    1532         rc = CFGMR3InsertInteger(pCfg,   "IRQ", ulIRQ);                             RC_CHECK();
     1531        hrc = parallelPort->COMGETTER(IRQ)(&ulIRQ);                                     H();
     1532        rc = CFGMR3InsertInteger(pCfg,   "IRQ", ulIRQ);                                 RC_CHECK();
    15331533        ULONG ulIOBase;
    1534         hrc = parallelPort->COMGETTER(IOBase)(&ulIOBase);                           H();
    1535         rc = CFGMR3InsertInteger(pCfg,   "IOBase", ulIOBase);                       RC_CHECK();
    1536         rc = CFGMR3InsertNode(pInst,     "LUN#0", &pLunL0);                         RC_CHECK();
    1537         rc = CFGMR3InsertString(pLunL0,  "Driver", "HostParallel");                 RC_CHECK();
    1538         rc = CFGMR3InsertNode(pLunL0,    "AttachedDriver", &pLunL1);                RC_CHECK();
    1539         hrc = parallelPort->COMGETTER(Path)(bstr.asOutParam());                     H();
    1540         rc = CFGMR3InsertStringW(pLunL1,  "DevicePath", bstr.raw());                RC_CHECK();
     1534        hrc = parallelPort->COMGETTER(IOBase)(&ulIOBase);                               H();
     1535        rc = CFGMR3InsertInteger(pCfg,   "IOBase", ulIOBase);                           RC_CHECK();
     1536        rc = CFGMR3InsertNode(pInst,     "LUN#0", &pLunL0);                             RC_CHECK();
     1537        rc = CFGMR3InsertString(pLunL0,  "Driver", "HostParallel");                     RC_CHECK();
     1538        rc = CFGMR3InsertNode(pLunL0,    "AttachedDriver", &pLunL1);                    RC_CHECK();
     1539        hrc = parallelPort->COMGETTER(Path)(bstr.asOutParam());                         H();
     1540        rc = CFGMR3InsertStringW(pLunL1,  "DevicePath", bstr.raw());                    RC_CHECK();
    15411541    }
    15421542
    … …  
    15441544     * VMM Device
    15451545     */
    1546     rc = CFGMR3InsertNode(pDevices, "VMMDev", &pDev);                               RC_CHECK();
    1547     rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
    1548     rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
    1549     rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */   RC_CHECK();
    1550     rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo",          4);                     RC_CHECK();
     1546    rc = CFGMR3InsertNode(pDevices, "VMMDev", &pDev);                                   RC_CHECK();
     1547    rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                       RC_CHECK();
     1548    rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                                   RC_CHECK();
     1549    rc = CFGMR3InsertInteger(pInst, "Trusted",              1);     /* boolean */       RC_CHECK();
     1550    rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo",          4);                         RC_CHECK();
    15511551    Assert(!afPciDeviceNo[4]);
    15521552    afPciDeviceNo[4] = true;
    1553     rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",        0);                     RC_CHECK();
     1553    rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",        0);                         RC_CHECK();
    15541554    Bstr hwVersion;
    1555     hrc = pMachine->COMGETTER(HardwareVersion)(hwVersion.asOutParam());             H();
     1555    hrc = pMachine->COMGETTER(HardwareVersion)(hwVersion.asOutParam());                 H();
    15561556    if (hwVersion.compare(Bstr("1")) == 0) /* <= 2.0.x */
    15571557    {
    1558         CFGMR3InsertInteger(pCfg, "HeapEnabled", 0);                                RC_CHECK();
     1558        CFGMR3InsertInteger(pCfg, "HeapEnabled", 0);                                    RC_CHECK();
    15591559    }
    15601560
    15611561    /* the VMM device's Main driver */
    1562     rc = CFGMR3InsertNode(pInst,    "LUN#0", &pLunL0);                              RC_CHECK();
    1563     rc = CFGMR3InsertString(pLunL0, "Driver",               "HGCM");                RC_CHECK();
    1564     rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
     1562    rc = CFGMR3InsertNode(pInst,    "LUN#0", &pLunL0);                                  RC_CHECK();
     1563    rc = CFGMR3InsertString(pLunL0, "Driver",               "HGCM");                    RC_CHECK();
     1564    rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                                   RC_CHECK();
    15651565    VMMDev *pVMMDev = pConsole->mVMMDev;
    1566     rc = CFGMR3InsertInteger(pCfg,  "Object", (uintptr_t)pVMMDev);                  RC_CHECK();
     1566    rc = CFGMR3InsertInteger(pCfg,  "Object", (uintptr_t)pVMMDev);                      RC_CHECK();
    15671567
    15681568    /*
    15691569     * Attach the status driver.
    15701570     */
    1571     rc = CFGMR3InsertNode(pInst,    "LUN#999", &pLunL0);                            RC_CHECK();
    1572     rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");          RC_CHECK();
    1573     rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
     1571    rc = CFGMR3InsertNode(pInst,    "LUN#999", &pLunL0);                                RC_CHECK();
     1572    rc = CFGMR3InsertString(pLunL0, "Driver",               "MainStatus");              RC_CHECK();
     1573    rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                                   RC_CHECK();
    15741574    rc = CFGMR3InsertInteger(pCfg,  "papLeds", (uintptr_t)&pConsole->mapSharedFolderLed); RC_CHECK();
    1575     rc = CFGMR3InsertInteger(pCfg,  "First",    0);                                 RC_CHECK();
    1576     rc = CFGMR3InsertInteger(pCfg,  "Last",     0);                                 RC_CHECK();
     1575    rc = CFGMR3InsertInteger(pCfg,  "First",    0);                                     RC_CHECK();
     1576    rc = CFGMR3InsertInteger(pCfg,  "Last",     0);                                     RC_CHECK();
    15771577
    15781578    /*
    15791579     * Audio Sniffer Device
    15801580     */
    1581     rc = CFGMR3InsertNode(pDevices, "AudioSniffer", &pDev);                         RC_CHECK();
    1582     rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
    1583     rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
     1581    rc = CFGMR3InsertNode(pDevices, "AudioSniffer", &pDev);                             RC_CHECK();
     1582    rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                       RC_CHECK();
     1583    rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                                   RC_CHECK();
    15841584
    15851585    /* the Audio Sniffer device's Main driver */
    1586     rc = CFGMR3InsertNode(pInst,    "LUN#0", &pLunL0);                              RC_CHECK();
    1587     rc = CFGMR3InsertString(pLunL0, "Driver",               "MainAudioSniffer");    RC_CHECK();
    1588     rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
     1586    rc = CFGMR3InsertNode(pInst,    "LUN#0", &pLunL0);                                  RC_CHECK();
     1587    rc = CFGMR3InsertString(pLunL0, "Driver",               "MainAudioSniffer");        RC_CHECK();
     1588    rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                                   RC_CHECK();
    15891589    AudioSniffer *pAudioSniffer = pConsole->mAudioSniffer;
    1590     rc = CFGMR3InsertInteger(pCfg,  "Object", (uintptr_t)pAudioSniffer);            RC_CHECK();
     1590    rc = CFGMR3InsertInteger(pCfg,  "Object", (uintptr_t)pAudioSniffer);                RC_CHECK();
    15911591
    15921592    /*
    … …  
    15951595    BOOL enabled;
    15961596    ComPtr<IAudioAdapter> audioAdapter;
    1597     hrc = pMachine->COMGETTER(AudioAdapter)(audioAdapter.asOutParam());             H();
     1597    hrc = pMachine->COMGETTER(AudioAdapter)(audioAdapter.asOutParam());                 H();
    15981598    if (audioAdapter)
    1599         hrc = audioAdapter->COMGETTER(Enabled)(&enabled);                           H();
     1599        hrc = audioAdapter->COMGETTER(Enabled)(&enabled);                               H();
    16001600
    16011601    if (enabled)
    16021602    {
    16031603        AudioControllerType_T audioController;
    1604         hrc = audioAdapter->COMGETTER(AudioController)(&audioController);           H();
     1604        hrc = audioAdapter->COMGETTER(AudioController)(&audioController);               H();
    16051605        switch (audioController)
    16061606        {
    … …  
    16081608            {
    16091609                /* default: ICH AC97 */
    1610                 rc = CFGMR3InsertNode(pDevices, "ichac97", &pDev);                  RC_CHECK();
     1610                rc = CFGMR3InsertNode(pDevices, "ichac97", &pDev);                      RC_CHECK();
    16111611                rc = CFGMR3InsertNode(pDev,     "0", &pInst);
    1612                 rc = CFGMR3InsertInteger(pInst, "Trusted",          1); /* bool */  RC_CHECK();
    1613                 rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo",      5);             RC_CHECK();
     1612                rc = CFGMR3InsertInteger(pInst, "Trusted",          1); /* bool */      RC_CHECK();
     1613                rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo",      5);                 RC_CHECK();
    16141614                Assert(!afPciDeviceNo[5]);
    16151615                afPciDeviceNo[5] = true;
    1616                 rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",    0);             RC_CHECK();
    1617                 rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                   RC_CHECK();
     1616                rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",    0);                 RC_CHECK();
     1617                rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                       RC_CHECK();
    16181618                break;
    16191619            }
    … …  
    16211621            {
    16221622                /* legacy SoundBlaster16 */
    1623                 rc = CFGMR3InsertNode(pDevices, "sb16", &pDev);                     RC_CHECK();
    1624                 rc = CFGMR3InsertNode(pDev,     "0", &pInst);                       RC_CHECK();
    1625                 rc = CFGMR3InsertInteger(pInst, "Trusted",          1); /* bool */  RC_CHECK();
    1626                 rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                   RC_CHECK();
    1627                 rc = CFGMR3InsertInteger(pCfg,  "IRQ", 5);                          RC_CHECK();
    1628                 rc = CFGMR3InsertInteger(pCfg,  "DMA", 1);                          RC_CHECK();
    1629                 rc = CFGMR3InsertInteger(pCfg,  "DMA16", 5);                        RC_CHECK();
    1630                 rc = CFGMR3InsertInteger(pCfg,  "Port", 0x220);                     RC_CHECK();
    1631                 rc = CFGMR3InsertInteger(pCfg,  "Version", 0x0405);                 RC_CHECK();
     1623                rc = CFGMR3InsertNode(pDevices, "sb16", &pDev);                         RC_CHECK();
     1624                rc = CFGMR3InsertNode(pDev,     "0", &pInst);                           RC_CHECK();
     1625                rc = CFGMR3InsertInteger(pInst, "Trusted",          1); /* bool */      RC_CHECK();
     1626                rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                       RC_CHECK();
     1627                rc = CFGMR3InsertInteger(pCfg,  "IRQ", 5);                              RC_CHECK();
     1628                rc = CFGMR3InsertInteger(pCfg,  "DMA", 1);                              RC_CHECK();
     1629                rc = CFGMR3InsertInteger(pCfg,  "DMA16", 5);                            RC_CHECK();
     1630                rc = CFGMR3InsertInteger(pCfg,  "Port", 0x220);                         RC_CHECK();
     1631                rc = CFGMR3InsertInteger(pCfg,  "Version", 0x0405);                     RC_CHECK();
    16321632                break;
    16331633            }
    … …  
    16351635
    16361636        /* the Audio driver */
    1637         rc = CFGMR3InsertNode(pInst,    "LUN#0", &pLunL0);                          RC_CHECK();
    1638         rc = CFGMR3InsertString(pLunL0, "Driver",               "AUDIO");           RC_CHECK();
    1639         rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                           RC_CHECK();
     1637        rc = CFGMR3InsertNode(pInst,    "LUN#0", &pLunL0);                              RC_CHECK();
     1638        rc = CFGMR3InsertString(pLunL0, "Driver",               "AUDIO");               RC_CHECK();
     1639        rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
    16401640
    16411641        AudioDriverType_T audioDriver;
    1642         hrc = audioAdapter->COMGETTER(AudioDriver)(&audioDriver);                   H();
     1642        hrc = audioAdapter->COMGETTER(AudioDriver)(&audioDriver);                       H();
    16431643        switch (audioDriver)
    16441644        {
    16451645            case AudioDriverType_Null:
    16461646            {
    1647                 rc = CFGMR3InsertString(pCfg, "AudioDriver", "null");               RC_CHECK();
     1647                rc = CFGMR3InsertString(pCfg, "AudioDriver", "null");                   RC_CHECK();
    16481648                break;
    16491649            }
    … …  
    16521652            case AudioDriverType_WinMM:
    16531653            {
    1654                 rc = CFGMR3InsertString(pCfg, "AudioDriver", "winmm");              RC_CHECK();
     1654                rc = CFGMR3InsertString(pCfg, "AudioDriver", "winmm");                  RC_CHECK();
    16551655                break;
    16561656            }
    … …  
    16581658            case AudioDriverType_DirectSound:
    16591659            {
    1660                 rc = CFGMR3InsertString(pCfg, "AudioDriver", "dsound");             RC_CHECK();
     1660                rc = CFGMR3InsertString(pCfg, "AudioDriver", "dsound");                 RC_CHECK();
    16611661                break;
    16621662            }
    … …  
    16651665            case AudioDriverType_SolAudio:
    16661666            {
    1667                 rc = CFGMR3InsertString(pCfg, "AudioDriver", "solaudio");           RC_CHECK();
     1667                rc = CFGMR3InsertString(pCfg, "AudioDriver", "solaudio");               RC_CHECK();
    16681668                break;
    16691669            }
    … …  
    16731673            case AudioDriverType_ALSA:
    16741674            {
    1675                 rc = CFGMR3InsertString(pCfg, "AudioDriver", "alsa");               RC_CHECK();
     1675                rc = CFGMR3InsertString(pCfg, "AudioDriver", "alsa");                   RC_CHECK();
    16761676                break;
    16771677            }
    … …  
    16801680            case AudioDriverType_Pulse:
    16811681            {
    1682                 rc = CFGMR3InsertString(pCfg, "AudioDriver", "pulse");              RC_CHECK();
     1682                rc = CFGMR3InsertString(pCfg, "AudioDriver", "pulse");                  RC_CHECK();
    16831683                break;
    16841684            }
    … …  
    16881688            case AudioDriverType_OSS:
    16891689            {
    1690                 rc = CFGMR3InsertString(pCfg, "AudioDriver", "oss");                RC_CHECK();
     1690                rc = CFGMR3InsertString(pCfg, "AudioDriver", "oss");                    RC_CHECK();
    16911691                break;
    16921692            }
    … …  
    16961696            case AudioDriverType_Pulse:
    16971697            {
    1698                 rc = CFGMR3InsertString(pCfg, "AudioDriver", "pulse");              RC_CHECK();
     1698                rc = CFGMR3InsertString(pCfg, "AudioDriver", "pulse");                  RC_CHECK();
    16991699                break;
    17001700            }
    … …  
    17041704            case AudioDriverType_CoreAudio:
    17051705            {
    1706                 rc = CFGMR3InsertString(pCfg, "AudioDriver", "coreaudio");          RC_CHECK();
     1706                rc = CFGMR3InsertString(pCfg, "AudioDriver", "coreaudio");              RC_CHECK();
    17071707                break;
    17081708            }
    17091709#endif
    17101710        }
    1711         hrc = pMachine->COMGETTER(Name)(bstr.asOutParam());                         H();
    1712         rc = CFGMR3InsertStringW(pCfg, "StreamName", bstr.raw());                   RC_CHECK();
     1711        hrc = pMachine->COMGETTER(Name)(bstr.asOutParam());                             H();
     1712        rc = CFGMR3InsertStringW(pCfg, "StreamName", bstr.raw());                       RC_CHECK();
    17131713    }
    17141714
    … …  
    20022002     */
    20032003    BOOL fACPI;
    2004     hrc = biosSettings->COMGETTER(ACPIEnabled)(&fACPI);                             H();
     2004    hrc = biosSettings->COMGETTER(ACPIEnabled)(&fACPI);                                 H();
    20052005    if (fACPI)
    20062006    {
    … …  
    20142014            fShowCpu = true;
    20152015
    2016         hrc = pMachine->COMGETTER(CPUHotPlugEnabled)(&fCpuHotPlug);                 H();
    2017 
    2018         rc = CFGMR3InsertNode(pDevices, "acpi", &pDev);                             RC_CHECK();
    2019         rc = CFGMR3InsertNode(pDev,     "0", &pInst);                               RC_CHECK();
    2020         rc = CFGMR3InsertInteger(pInst, "Trusted", 1);              /* boolean */   RC_CHECK();
    2021         rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                           RC_CHECK();
    2022         rc = CFGMR3InsertInteger(pCfg,  "RamSize",          cbRam);                 RC_CHECK();
    2023         rc = CFGMR3InsertInteger(pCfg,  "RamHoleSize",      cbRamHole);             RC_CHECK();
    2024         rc = CFGMR3InsertInteger(pCfg,  "NumCPUs",          cCpus);                 RC_CHECK();
    2025 
    2026         rc = CFGMR3InsertInteger(pCfg,  "IOAPIC", fIOAPIC);                         RC_CHECK();
    2027         rc = CFGMR3InsertInteger(pCfg,  "FdcEnabled", fFdcEnabled);                 RC_CHECK();
    2028         rc = CFGMR3InsertInteger(pCfg,  "HpetEnabled", fHpetEnabled);               RC_CHECK();
    2029         rc = CFGMR3InsertInteger(pCfg,  "SmcEnabled", fSmcEnabled);                 RC_CHECK();
    2030         rc = CFGMR3InsertInteger(pCfg,  "ShowRtc",    fOsXGuest);                   RC_CHECK();
     2016        hrc = pMachine->COMGETTER(CPUHotPlugEnabled)(&fCpuHotPlug);                     H();
     2017
     2018        rc = CFGMR3InsertNode(pDevices, "acpi", &pDev);                                 RC_CHECK();
     2019        rc = CFGMR3InsertNode(pDev,     "0", &pInst);                                   RC_CHECK();
     2020        rc = CFGMR3InsertInteger(pInst, "Trusted", 1);              /* boolean */       RC_CHECK();
     2021        rc = CFGMR3InsertNode(pInst,    "Config", &pCfg);                               RC_CHECK();
     2022        rc = CFGMR3InsertInteger(pCfg,  "RamSize",          cbRam);                     RC_CHECK();
     2023        rc = CFGMR3InsertInteger(pCfg,  "RamHoleSize",      cbRamHole);                 RC_CHECK();
     2024        rc = CFGMR3InsertInteger(pCfg,  "NumCPUs",          cCpus);                     RC_CHECK();
     2025
     2026        rc = CFGMR3InsertInteger(pCfg,  "IOAPIC", fIOAPIC);                             RC_CHECK();
     2027        rc = CFGMR3InsertInteger(pCfg,  "FdcEnabled", fFdcEnabled);                     RC_CHECK();
     2028        rc = CFGMR3InsertInteger(pCfg,  "HpetEnabled", fHpetEnabled);                   RC_CHECK();
     2029        rc = CFGMR3InsertInteger(pCfg,  "SmcEnabled", fSmcEnabled);                     RC_CHECK();
     2030        rc = CFGMR3InsertInteger(pCfg,  "ShowRtc",    fOsXGuest);                       RC_CHECK();
    20312031        if (fOsXGuest && !llBootNics.empty())
    20322032        {
    20332033            BootNic aNic = llBootNics.front();
    20342034            uint32_t u32NicPciAddr = (aNic.mPciDev << 16) | aNic.mPciFn;
    2035             rc = CFGMR3InsertInteger(pCfg,  "NicPciAddress",    u32NicPciAddr);     RC_CHECK();
    2036         }
    2037         rc = CFGMR3InsertInteger(pCfg,  "ShowCpu", fShowCpu);                       RC_CHECK();
    2038         rc = CFGMR3InsertInteger(pCfg,  "CpuHotPlug", fCpuHotPlug);                 RC_CHECK();
    2039         rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo",          7);                 RC_CHECK();
     2035            rc = CFGMR3InsertInteger(pCfg,  "NicPciAddress",    u32NicPciAddr);         RC_CHECK();
     2036        }
     2037        rc = CFGMR3InsertInteger(pCfg,  "ShowCpu", fShowCpu);                           RC_CHECK();
     2038        rc = CFGMR3InsertInteger(pCfg,  "CpuHotPlug", fCpuHotPlug);                     RC_CHECK();
     2039        rc = CFGMR3InsertInteger(pInst, "PCIDeviceNo",          7);                     RC_CHECK();
    20402040        Assert(!afPciDeviceNo[7]);
    20412041        afPciDeviceNo[7] = true;
    2042         rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",        0);                 RC_CHECK();
    2043 
    2044         rc = CFGMR3InsertNode(pInst,    "LUN#0", &pLunL0);                          RC_CHECK();
    2045         rc = CFGMR3InsertString(pLunL0, "Driver",               "ACPIHost");        RC_CHECK();
    2046         rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                           RC_CHECK();
     2042        rc = CFGMR3InsertInteger(pInst, "PCIFunctionNo",        0);                     RC_CHECK();
     2043
     2044        rc = CFGMR3InsertNode(pInst,    "LUN#0", &pLunL0);                              RC_CHECK();
     2045        rc = CFGMR3InsertString(pLunL0, "Driver",               "ACPIHost");            RC_CHECK();
     2046        rc = CFGMR3InsertNode(pLunL0,   "Config", &pCfg);                               RC_CHECK();
    20472047
    20482048        /* Attach the dummy CPU drivers */
    … …  
    20532053            if (fCpuHotPlug)
    20542054            {
    2055                 hrc = pMachine->GetCPUStatus(iCpuCurr, &fCpuAttached);              H();
     2055                hrc = pMachine->GetCPUStatus(iCpuCurr, &fCpuAttached);                  H();
    20562056            }
    20572057
    … …  
    22522252    Bstr    bstr;
    22532253
    2254 #define RC_CHECK()  do { if (RT_FAILURE(rc)) { AssertMsgFailed(("rc=%Rrc\n", rc));  return rc;                   } } while (0)
    2255 #define H()         do { if (FAILED(hrc))    { AssertMsgFailed(("hrc=%Rhrc\n", hrc)); return VERR_GENERAL_FAILURE; } } while (0)
     2254#define RC_CHECK()  AssertMsgReturn(RT_SUCCESS(rc), ("rc=%Rrc\n", rc), rc)
     2255#define H()         AssertMsgReturn(!FAILED(hrc), ("hrc=%Rhrc\n", hrc), VERR_GENERAL_FAILURE)
    22562256
    22572257    LONG lDev;
    2258     hrc = pMediumAtt->COMGETTER(Device)(&lDev);                         H();
     2258    hrc = pMediumAtt->COMGETTER(Device)(&lDev);                                         H();
    22592259    LONG lPort;
    2260     hrc = pMediumAtt->COMGETTER(Port)(&lPort);                          H();
     2260    hrc = pMediumAtt->COMGETTER(Port)(&lPort);                                          H();
    22612261    DeviceType_T lType;
    2262     hrc = pMediumAtt->COMGETTER(Type)(&lType);                          H();
     2262    hrc = pMediumAtt->COMGETTER(Type)(&lType);                                          H();
    22632263
    22642264    unsigned uLUN;
    22652265    PCFGMNODE pLunL0 = NULL;
    22662266    PCFGMNODE pCfg = NULL;
    2267     hrc = Console::convertBusPortDeviceToLun(enmBus, lPort, lDev, uLUN);    H();
     2267    hrc = Console::convertBusPortDeviceToLun(enmBus, lPort, lDev, uLUN);                H();
    22682268
    22692269    /* First check if the LUN already exists. */
    … …  
    23072307    }
    23082308
    2309     rc = CFGMR3InsertNodeF(pCtlInst, &pLunL0, "LUN#%u", uLUN);              RC_CHECK();
     2309    rc = CFGMR3InsertNodeF(pCtlInst, &pLunL0, "LUN#%u", uLUN);                          RC_CHECK();
    23102310
    23112311    /* SCSI has a another driver between device and block. */
    23122312    if (enmBus == StorageBus_SCSI || enmBus == StorageBus_SAS)
    23132313    {
    2314         rc = CFGMR3InsertString(pLunL0, "Driver", "SCSI");              RC_CHECK();
    2315         rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                 RC_CHECK();
    2316 
    2317         rc = CFGMR3InsertNode(pLunL0, "AttachedDriver", &pLunL0);       RC_CHECK();
     2314        rc = CFGMR3InsertString(pLunL0, "Driver", "SCSI");                              RC_CHECK();
     2315        rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                                 RC_CHECK();
     2316
     2317        rc = CFGMR3InsertNode(pLunL0, "AttachedDriver", &pLunL0);                       RC_CHECK();
    23182318    }
    23192319
    23202320    ComPtr<IMedium> pMedium;
    2321     hrc = pMediumAtt->COMGETTER(Medium)(pMedium.asOutParam());          H();
     2321    hrc = pMediumAtt->COMGETTER(Medium)(pMedium.asOutParam());                          H();
    23222322    BOOL fPassthrough;
    2323     hrc = pMediumAtt->COMGETTER(Passthrough)(&fPassthrough);            H();
     2323    hrc = pMediumAtt->COMGETTER(Passthrough)(&fPassthrough);                            H();
    23242324    rc = configMedium(pLunL0,
    23252325                      !!fPassthrough,
    … …  
    23312331                      pMedium,
    23322332                      aMachineState,
    2333                       phrc);                                            RC_CHECK();
     2333                      phrc);                                                            RC_CHECK();
    23342334
    23352335    if (fAttachDetach)
    … …  
    23372337        /* Attach the new driver. */
    23382338        rc = PDMR3DeviceAttach(pVM, pcszDevice, 0, uLUN,
    2339                                PDM_TACH_FLAGS_NOT_HOT_PLUG, NULL /*ppBase*/);   RC_CHECK();
     2339                               PDM_TACH_FLAGS_NOT_HOT_PLUG, NULL /*ppBase*/);           RC_CHECK();
    23402340
    23412341        /* There is no need to handle removable medium mounting, as we
    … …  
    23682368    Bstr bstr;
    23692369
    2370 #define RC_CHECK()  do { if (RT_FAILURE(rc)) { AssertMsgFailed(("rc=%Rrc\n", rc));  return rc;                   } } while (0)
    2371 #define H()         do { if (FAILED(hrc))    { AssertMsgFailed(("hrc=%Rhrc\n", hrc)); if (phrc) *phrc = hrc; return VERR_GENERAL_FAILURE; } } while (0)
     2370#define RC_CHECK()  AssertMsgReturn(RT_SUCCESS(rc), ("rc=%Rrc\n", rc), rc)
     2371#define H()         AssertMsgReturnStmt(!FAILED(hrc), ("hrc=%Rhrc\n", hrc), if (phrc) *phrc = hrc, VERR_GENERAL_FAILURE)
    23722372
    23732373    PCFGMNODE pLunL1 = NULL;
    … …  
    26432643    Bstr bstr;
    26442644
    2645 #define RC_CHECK()  do { if (RT_FAILURE(rc)) { AssertMsgFailed(("rc=%Rrc\n", rc));  return rc;                   } } while (0)
    2646 #define H()         do { if (FAILED(hrc))    { AssertMsgFailed(("hrc=%Rhrc\n", hrc)); return VERR_GENERAL_FAILURE; } } while (0)
     2645#define RC_CHECK()  AssertMsgReturn(RT_SUCCESS(rc), ("rc=%Rrc\n", rc), rc)
     2646#define H()         AssertMsgReturn(!FAILED(hrc), ("hrc=%Rhrc\n", hrc), VERR_GENERAL_FAILURE)
    26472647
    26482648    /*
    … …  
    26932693        {
    26942694            CFGMR3RemoveNode(pLunL0);
    2695             rc = CFGMR3InsertNode(pInst, "LUN#0", &pLunL0);                 RC_CHECK();
    2696             rc = CFGMR3InsertString(pLunL0, "Driver", "NetSniffer");        RC_CHECK();
    2697             rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                 RC_CHECK();
    2698             hrc = aNetworkAdapter->COMGETTER(TraceFile)(bstr.asOutParam()); H();
     2695            rc = CFGMR3InsertNode(pInst, "LUN#0", &pLunL0);                             RC_CHECK();
     2696            rc = CFGMR3InsertString(pLunL0, "Driver", "NetSniffer");                    RC_CHECK();
     2697            rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                             RC_CHECK();
     2698            hrc = aNetworkAdapter->COMGETTER(TraceFile)(bstr.asOutParam());             H();
    26992699            if (!bstr.isEmpty()) /* check convention for indicating default file. */
    27002700            {
    2701                 rc = CFGMR3InsertStringW(pCfg, "File", bstr.raw());         RC_CHECK();
     2701                rc = CFGMR3InsertStringW(pCfg, "File", bstr.raw());                     RC_CHECK();
    27022702            }
    27032703        }
    … …  
    27162716    {
    27172717        /* insert the sniffer filter driver. */
    2718         rc = CFGMR3InsertNode(pInst, "LUN#0", &pLunL0);                 RC_CHECK();
    2719         rc = CFGMR3InsertString(pLunL0, "Driver", "NetSniffer");        RC_CHECK();
    2720         rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                 RC_CHECK();
    2721         hrc = aNetworkAdapter->COMGETTER(TraceFile)(bstr.asOutParam()); H();
     2718        rc = CFGMR3InsertNode(pInst, "LUN#0", &pLunL0);                                 RC_CHECK();
     2719        rc = CFGMR3InsertString(pLunL0, "Driver", "NetSniffer");                        RC_CHECK();
     2720        rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                                 RC_CHECK();
     2721        hrc = aNetworkAdapter->COMGETTER(TraceFile)(bstr.asOutParam());                 H();
    27222722        if (!bstr.isEmpty()) /* check convention for indicating default file. */
    27232723        {
    2724             rc = CFGMR3InsertStringW(pCfg, "File", bstr.raw());         RC_CHECK();
     2724            rc = CFGMR3InsertStringW(pCfg, "File", bstr.raw());                         RC_CHECK();
    27252725        }
    27262726    }
    … …  
    27282728    Bstr networkName, trunkName, trunkType;
    27292729    NetworkAttachmentType_T eAttachmentType;
    2730     hrc = aNetworkAdapter->COMGETTER(AttachmentType)(&eAttachmentType); H();
     2730    hrc = aNetworkAdapter->COMGETTER(AttachmentType)(&eAttachmentType);                 H();
    27312731    switch (eAttachmentType)
    27322732    {
    … …  
    27372737        {
    27382738            ComPtr<INATEngine> natDriver;
    2739             hrc = aNetworkAdapter->COMGETTER(NatDriver)(natDriver.asOutParam());         H();
     2739            hrc = aNetworkAdapter->COMGETTER(NatDriver)(natDriver.asOutParam());        H();
    27402740            if (fSniffer)
    27412741            {
    2742                 rc = CFGMR3InsertNode(pLunL0, "AttachedDriver", &pLunL0); RC_CHECK();
     2742                rc = CFGMR3InsertNode(pLunL0, "AttachedDriver", &pLunL0);               RC_CHECK();
    27432743            }
    27442744            else
    27452745            {
    2746                 rc = CFGMR3InsertNode(pInst, "LUN#0", &pLunL0);         RC_CHECK();
    2747             }
    2748             rc = CFGMR3InsertString(pLunL0, "Driver", "NAT");           RC_CHECK();
    2749             rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);             RC_CHECK();
     2746                rc = CFGMR3InsertNode(pInst, "LUN#0", &pLunL0);                         RC_CHECK();
     2747            }
     2748            rc = CFGMR3InsertString(pLunL0, "Driver", "NAT");                           RC_CHECK();
     2749            rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                             RC_CHECK();
    27502750
    27512751            /* Configure TFTP prefix and boot filename. */
    2752             hrc = virtualBox->COMGETTER(HomeFolder)(bstr.asOutParam()); H();
     2752            hrc = virtualBox->COMGETTER(HomeFolder)(bstr.asOutParam());                 H();
    27532753            if (!bstr.isEmpty())
    27542754            {
    27552755                rc = CFGMR3InsertStringF(pCfg, "TFTPPrefix", "%ls%c%s", bstr.raw(), RTPATH_DELIMITER, "TFTP"); RC_CHECK();
    27562756            }
    2757             hrc = pMachine->COMGETTER(Name)(bstr.asOutParam());         H();
    2758             rc = CFGMR3InsertStringF(pCfg, "BootFile", "%ls.pxe", bstr.raw()); RC_CHECK();
    2759 
    2760             hrc = natDriver->COMGETTER(Network)(bstr.asOutParam());     H();
     2757            hrc = pMachine->COMGETTER(Name)(bstr.asOutParam());                         H();
     2758            rc = CFGMR3InsertStringF(pCfg, "BootFile", "%ls.pxe", bstr.raw());          RC_CHECK();
     2759
     2760            hrc = natDriver->COMGETTER(Network)(bstr.asOutParam());                     H();
    27612761            if (!bstr.isEmpty())
    27622762            {
    2763                 rc = CFGMR3InsertStringW(pCfg, "Network", bstr.raw());  RC_CHECK();
     2763                rc = CFGMR3InsertStringW(pCfg, "Network", bstr.raw());                  RC_CHECK();
    27642764            }
    27652765            else
    27662766            {
    27672767                ULONG uSlot;
    2768                 hrc = aNetworkAdapter->COMGETTER(Slot)(&uSlot);         H();
    2769                 rc = CFGMR3InsertStringF(pCfg, "Network", "10.0.%d.0/24", uSlot+2); RC_CHECK();
    2770             }
    2771             hrc = natDriver->COMGETTER(HostIP)(bstr.asOutParam());      H();
     2768                hrc = aNetworkAdapter->COMGETTER(Slot)(&uSlot);                         H();
     2769                rc = CFGMR3InsertStringF(pCfg, "Network", "10.0.%d.0/24", uSlot+2);     RC_CHECK();
     2770            }
     2771            hrc = natDriver->COMGETTER(HostIP)(bstr.asOutParam());                      H();
    27722772            if (!bstr.isEmpty())
    27732773            {
    2774                 rc = CFGMR3InsertStringW(pCfg, "BindIP", bstr.raw());   RC_CHECK();
     2774                rc = CFGMR3InsertStringW(pCfg, "BindIP", bstr.raw());                   RC_CHECK();
    27752775            }
    27762776            ULONG mtu = 0;
    … …  
    27822782            if (mtu)
    27832783            {
    2784                 rc = CFGMR3InsertInteger(pCfg, "SlirpMTU", mtu);         RC_CHECK();
     2784                rc = CFGMR3InsertInteger(pCfg, "SlirpMTU", mtu);                        RC_CHECK();
    27852785            }
    27862786            if (sockRcv)
    27872787            {
    2788                 rc = CFGMR3InsertInteger(pCfg, "SockRcv", sockRcv);      RC_CHECK();
     2788                rc = CFGMR3InsertInteger(pCfg, "SockRcv", sockRcv);                     RC_CHECK();
    27892789            }
    27902790            if (sockSnd)
    27912791            {
    2792                 rc = CFGMR3InsertInteger(pCfg, "SockSnd", sockSnd);      RC_CHECK();
     2792                rc = CFGMR3InsertInteger(pCfg, "SockSnd", sockSnd);                     RC_CHECK();
    27932793            }
    27942794            if (tcpRcv)
    27952795            {
    2796                 rc = CFGMR3InsertInteger(pCfg, "TcpRcv", tcpRcv);        RC_CHECK();
     2796                rc = CFGMR3InsertInteger(pCfg, "TcpRcv", tcpRcv);                       RC_CHECK();
    27972797            }
    27982798            if (tcpSnd)
    27992799            {
    2800                 rc = CFGMR3InsertInteger(pCfg, "TcpSnd", tcpSnd);        RC_CHECK();
    2801             }
    2802             hrc = natDriver->COMGETTER(TftpPrefix)(bstr.asOutParam());   H();
     2800                rc = CFGMR3InsertInteger(pCfg, "TcpSnd", tcpSnd);                       RC_CHECK();
     2801            }
     2802            hrc = natDriver->COMGETTER(TftpPrefix)(bstr.asOutParam());                  H();
    28032803            if (!bstr.isEmpty())
    28042804            {
    2805                 rc = CFGMR3RemoveValue(pCfg, "TFTPPrefix");              RC_CHECK();
    2806                 rc = CFGMR3InsertStringW(pCfg, "TFTPPrefix", bstr);       RC_CHECK();
    2807             }
    2808             hrc = natDriver->COMGETTER(TftpBootFile)(bstr.asOutParam());              H();
     2805                rc = CFGMR3RemoveValue(pCfg, "TFTPPrefix");                             RC_CHECK();
     2806                rc = CFGMR3InsertStringW(pCfg, "TFTPPrefix", bstr);                     RC_CHECK();
     2807            }
     2808            hrc = natDriver->COMGETTER(TftpBootFile)(bstr.asOutParam());                H();
    28092809            if (!bstr.isEmpty())
    28102810            {
    2811                 rc = CFGMR3RemoveValue(pCfg, "BootFile");                RC_CHECK();
    2812                 rc = CFGMR3InsertStringW(pCfg, "BootFile", bstr);         RC_CHECK();
    2813             }
    2814             hrc = natDriver->COMGETTER(TftpNextServer)(bstr.asOutParam()); H();
     2811                rc = CFGMR3RemoveValue(pCfg, "BootFile");                               RC_CHECK();
     2812                rc = CFGMR3InsertStringW(pCfg, "BootFile", bstr);                       RC_CHECK();
     2813            }
     2814            hrc = natDriver->COMGETTER(TftpNextServer)(bstr.asOutParam());              H();
    28152815            if (!bstr.isEmpty())
    28162816            {
    2817                 rc = CFGMR3InsertStringW(pCfg, "NextServer", bstr);       RC_CHECK();
     2817                rc = CFGMR3InsertStringW(pCfg, "NextServer", bstr);                     RC_CHECK();
    28182818            }
    28192819            BOOL fDnsFlag;
    2820             hrc = natDriver->COMGETTER(DnsPassDomain)(&fDnsFlag);        H();
    2821             rc = CFGMR3InsertInteger(pCfg, "PassDomain", fDnsFlag);      RC_CHECK();
    2822             hrc = natDriver->COMGETTER(DnsProxy)(&fDnsFlag);             H();
    2823             rc = CFGMR3InsertInteger(pCfg, "DNSProxy", fDnsFlag);        RC_CHECK();
    2824             hrc = natDriver->COMGETTER(DnsUseHostResolver)(&fDnsFlag);   H();
    2825             rc = CFGMR3InsertInteger(pCfg, "UseHostResolver", fDnsFlag); RC_CHECK();
     2820            hrc = natDriver->COMGETTER(DnsPassDomain)(&fDnsFlag);                       H();
     2821            rc = CFGMR3InsertInteger(pCfg, "PassDomain", fDnsFlag);                     RC_CHECK();
     2822            hrc = natDriver->COMGETTER(DnsProxy)(&fDnsFlag);                            H();
     2823            rc = CFGMR3InsertInteger(pCfg, "DNSProxy", fDnsFlag);                       RC_CHECK();
     2824            hrc = natDriver->COMGETTER(DnsUseHostResolver)(&fDnsFlag);                  H();
     2825            rc = CFGMR3InsertInteger(pCfg, "UseHostResolver", fDnsFlag);                RC_CHECK();
    28262826
    28272827            ULONG aliasMode;
    2828             hrc = natDriver->COMGETTER(AliasMode)(&aliasMode);           H();
    2829             rc = CFGMR3InsertInteger(pCfg, "AliasMode", aliasMode);      RC_CHECK();
     2828            hrc = natDriver->COMGETTER(AliasMode)(&aliasMode);                          H();
     2829            rc = CFGMR3InsertInteger(pCfg, "AliasMode", aliasMode);                     RC_CHECK();
    28302830
    28312831            /* port-forwarding */
    28322832            SafeArray<BSTR> pfs;
    2833             hrc = natDriver->COMGETTER(Redirects)(ComSafeArrayAsOutParam(pfs)); H();
     2833            hrc = natDriver->COMGETTER(Redirects)(ComSafeArrayAsOutParam(pfs));         H();
    28342834            PCFGMNODE pPF = NULL;          /* /Devices/Dev/.../Config/PF#0/ */
    28352835            for (unsigned int i = 0; i < pfs.size(); ++i)
    … …  
    28462846                size_t pos, ppos;
    28472847                pos = ppos = 0;
    2848                 #define ITERATE_TO_NEXT_TERM(res, str, pos, ppos)  \
    2849                 do {                                                \
    2850                     pos = str.find(",", ppos);                      \
    2851                     if (pos == Utf8Str::npos)                      \
    2852                     {                                              \
    2853                         Log(( #res " extracting from %s is failed\n", str.raw())); \
    2854                         continue;                                  \
    2855                     }                                              \
    2856                     res = str.substr(ppos, pos - ppos);            \
    2857                     Log2((#res " %s pos:%d, ppos:%d\n", res.raw(), pos, ppos)); \
    2858                     ppos = pos + 1;                                \
    2859                 } while (0)
     2848#define ITERATE_TO_NEXT_TERM(res, str, pos, ppos) \
     2849    do { \
     2850        pos = str.find(",", ppos); \
     2851        if (pos == Utf8Str::npos) \
     2852        { \
     2853            Log(( #res " extracting from %s is failed\n", str.raw())); \
     2854            continue; \
     2855        } \
     2856        res = str.substr(ppos, pos - ppos); \
     2857        Log2((#res " %s pos:%d, ppos:%d\n", res.raw(), pos, ppos)); \
     2858        ppos = pos + 1; \
     2859    } while (0)
    28602860                ITERATE_TO_NEXT_TERM(strName, utf, pos, ppos);
    28612861                ITERATE_TO_NEXT_TERM(strProto, utf, pos, ppos);
    … …  
    28642864                ITERATE_TO_NEXT_TERM(strGuestIP, utf, pos, ppos);
    28652865                strGuestPort = utf.substr(ppos, utf.length() - ppos);
    2866                 #undef ITERATE_TO_NEXT_TERM
     2866#undef ITERATE_TO_NEXT_TERM
    28672867
    28682868                uint32_t proto = strProto.toUInt32();
    … …  
    28722872                    case NATProtocol_UDP:
    28732873                        strProto = "UDP";
    2874                     break;
     2874                        break;
    28752875                    case NATProtocol_TCP:
    28762876                        strProto = "TCP";
    2877                     break;
     2877                        break;
    28782878                    default:
    28792879                        fValid = false;
    … …  
    28832883                    continue;
    28842884
    2885                 rc = CFGMR3InsertNode(pCfg, strName.raw(), &pPF);        RC_CHECK();
    2886                 rc = CFGMR3InsertString(pPF, "Protocol", strProto.raw()); RC_CHECK();
     2885                rc = CFGMR3InsertNode(pCfg, strName.raw(), &pPF);                       RC_CHECK();
     2886                rc = CFGMR3InsertString(pPF, "Protocol", strProto.raw());               RC_CHECK();
    28872887
    28882888                if (!strHostIP.isEmpty())
    28892889                {
    2890                     rc = CFGMR3InsertString(pPF, "BindIP", strHostIP.raw()); RC_CHECK();
     2890                    rc = CFGMR3InsertString(pPF, "BindIP", strHostIP.raw());            RC_CHECK();
    28912891                }
    28922892
    28932893                if (!strGuestIP.isEmpty())
    28942894                {
    2895                     rc = CFGMR3InsertString(pPF, "GuestIP", strGuestIP.raw()); RC_CHECK();
     2895                    rc = CFGMR3InsertString(pPF, "GuestIP", strGuestIP.raw());          RC_CHECK();
    28962896                }
    28972897
    … …  
    28992899                if (port)
    29002900                {
    2901                     rc = CFGMR3InsertInteger(pPF, "HostPort", port);     RC_CHECK();
     2901                    rc = CFGMR3InsertInteger(pPF, "HostPort", port);                    RC_CHECK();
    29022902                }
    29032903
    … …  
    29052905                if (port)
    29062906                {
    2907                     rc = CFGMR3InsertInteger(pPF, "GuestPort", port);    RC_CHECK();
     2907                    rc = CFGMR3InsertInteger(pPF, "GuestPort", port);                   RC_CHECK();
    29082908                }
    29092909            }
    … …  
    29212921                    case VERR_ACCESS_DENIED:
    29222922                        return VMSetError(pVM, VERR_HOSTIF_INIT_FAILED, RT_SRC_POS,  N_(
    2923                                          "Failed to open '/dev/net/tun' for read/write access. Please check the "
    2924                                          "permissions of that node. Either run 'chmod 0666 /dev/net/tun' or "
    2925                                          "change the group of that node and make yourself a member of that group. Make "
    2926                                          "sure that these changes are permanent, especially if you are "
    2927                                          "using udev"));
     2923                                          "Failed to open '/dev/net/tun' for read/write access. Please check the "
     2924                                          "permissions of that node. Either run 'chmod 0666 /dev/net/tun' or "
     2925                                          "change the group of that node and make yourself a member of that group. Make "
     2926                                          "sure that these changes are permanent, especially if you are "
     2927                                          "using udev"));
    29282928                    default:
    29292929                        AssertMsgFailed(("Could not attach to host interface! Bad!\n"));
    29302930                        return VMSetError(pVM, VERR_HOSTIF_INIT_FAILED, RT_SRC_POS, N_(
    2931                                          "Failed to initialize Host Interface Networking"));
     2931                                          "Failed to initialize Host Interface Networking"));
    29322932                }
    29332933            }
    … …  
    29382938                if (fSniffer)
    29392939                {
    2940                     rc = CFGMR3InsertNode(pLunL0, "AttachedDriver", &pLunL0);               RC_CHECK();
     2940                    rc = CFGMR3InsertNode(pLunL0, "AttachedDriver", &pLunL0);           RC_CHECK();
    29412941                }
    29422942                else
    29432943                {
    2944                     rc = CFGMR3InsertNode(pInst, "LUN#0", &pLunL0);                         RC_CHECK();
    2945                 }
    2946                 rc = CFGMR3InsertString(pLunL0, "Driver", "HostInterface");                 RC_CHECK();
    2947                 rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                             RC_CHECK();
    2948                 rc = CFGMR3InsertInteger(pCfg, "FileHandle", maTapFD[uInstance]);    RC_CHECK();
     2944                    rc = CFGMR3InsertNode(pInst, "LUN#0", &pLunL0);                     RC_CHECK();
     2945                }
     2946                rc = CFGMR3InsertString(pLunL0, "Driver", "HostInterface");             RC_CHECK();
     2947                rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                         RC_CHECK();
     2948                rc = CFGMR3InsertInteger(pCfg, "FileHandle", maTapFD[uInstance]);       RC_CHECK();
    29492949            }
    29502950
    … …  
    29552955            if (fSniffer)
    29562956            {
    2957                 rc = CFGMR3InsertNode(pLunL0, "AttachedDriver", &pLunL0);                   RC_CHECK();
     2957                rc = CFGMR3InsertNode(pLunL0, "AttachedDriver", &pLunL0);               RC_CHECK();
    29582958            }
    29592959            else
    29602960            {
    2961                 rc = CFGMR3InsertNode(pInst, "LUN#0", &pLunL0);                             RC_CHECK();
     2961                rc = CFGMR3InsertNode(pInst, "LUN#0", &pLunL0);                         RC_CHECK();
    29622962            }
    29632963
    … …  
    31343134                        case VERR_ACCESS_DENIED:
    31353135                            return VMSetError(pVM, VERR_HOSTIF_INIT_FAILED, RT_SRC_POS,  N_(
    3136                                              "Failed to open '/dev/%s' for read/write access.  Please check the "
    3137                                              "permissions of that node, and that the net.link.tap.user_open "
    3138                                              "sysctl is set.  Either run 'chmod 0666 /dev/%s' or "
    3139                                              "change the group of that node to vboxusers and make yourself "
    3140                                              "a member of that group.  Make sure that these changes are permanent."), pszHifName, pszHifName);
     3136                                              "Failed to open '/dev/%s' for read/write access.  Please check the "
     3137                                              "permissions of that node, and that the net.link.tap.user_open "
     3138                                              "sysctl is set.  Either run 'chmod 0666 /dev/%s' or "
     3139                                              "change the group of that node to vboxusers and make yourself "
     3140                                              "a member of that group.  Make sure that these changes are permanent."), pszHifName, pszHifName);
    31413141                        default:
    31423142                            AssertMsgFailed(("Could not attach to tap interface! Bad!\n"));
    31433143                            return VMSetError(pVM, VERR_HOSTIF_INIT_FAILED, RT_SRC_POS, N_(
    3144                                              "Failed to initialize Host Interface Networking"));
     3144                                              "Failed to initialize Host Interface Networking"));
    31453145                    }
    31463146                }
    … …  
    31493149                if ((int)maTapFD[uInstance] >= 0)
    31503150                {
    3151                     rc = CFGMR3InsertString(pLunL0, "Driver", "HostInterface");                 RC_CHECK();
    3152                     rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                             RC_CHECK();
    3153                     rc = CFGMR3InsertInteger(pCfg, "FileHandle", maTapFD[uInstance]);           RC_CHECK();
     3151                    rc = CFGMR3InsertString(pLunL0, "Driver", "HostInterface");         RC_CHECK();
     3152                    rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                     RC_CHECK();
     3153                    rc = CFGMR3InsertInteger(pCfg, "FileHandle", maTapFD[uInstance]);   RC_CHECK();
    31543154                }
    31553155                break;
    … …  
    31823182# endif
    31833183
    3184             rc = CFGMR3InsertString(pLunL0, "Driver", "IntNet");                    RC_CHECK();
    3185             rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                         RC_CHECK();
    3186             rc = CFGMR3InsertString(pCfg, "Trunk", pszTrunk);                       RC_CHECK();
     3184            rc = CFGMR3InsertString(pLunL0, "Driver", "IntNet");                        RC_CHECK();
     3185            rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                             RC_CHECK();
     3186            rc = CFGMR3InsertString(pCfg, "Trunk", pszTrunk);                           RC_CHECK();
    31873187            rc = CFGMR3InsertInteger(pCfg, "TrunkType", kIntNetTrunkType_NetFlt);
    31883188            RC_CHECK();
    31893189            char szNetwork[INTNET_MAX_NETWORK_NAME];
    31903190            RTStrPrintf(szNetwork, sizeof(szNetwork), "HostInterfaceNetworking-%s", pszHifName);
    3191             rc = CFGMR3InsertString(pCfg, "Network", szNetwork);                    RC_CHECK();
     3191            rc = CFGMR3InsertString(pCfg, "Network", szNetwork);                        RC_CHECK();
    31923192            networkName = Bstr(szNetwork);
    31933193            trunkName = Bstr(pszTrunk);
    … …  
    31993199                ||  strstr(pszHifName, "AirPort" ))
    32003200            {
    3201                 rc = CFGMR3InsertInteger(pCfg, "SharedMacOnWire", true);            RC_CHECK();
     3201                rc = CFGMR3InsertInteger(pCfg, "SharedMacOnWire", true);                RC_CHECK();
    32023202            }
    32033203# elif defined(RT_OS_LINUX)
    … …  
    33053305                {
    33063306                    Log(("this is a wireless adapter"));
    3307                     rc = CFGMR3InsertInteger(pCfg, "SharedMacOnWire", true); RC_CHECK();
     3307                    rc = CFGMR3InsertInteger(pCfg, "SharedMacOnWire", true);            RC_CHECK();
    33083308                    Log(("Set SharedMacOnWire\n"));
    33093309                }
    … …  
    33323332            if (ZoneId != GLOBAL_ZONEID)
    33333333            {
    3334                 rc = CFGMR3InsertInteger(pCfg, "IgnoreAllPromisc", true);   RC_CHECK();
     3334                rc = CFGMR3InsertInteger(pCfg, "IgnoreAllPromisc", true);               RC_CHECK();
    33353335            }
    33363336#  endif
    … …  
    33513351        case NetworkAttachmentType_Internal:
    33523352        {
    3353             hrc = aNetworkAdapter->COMGETTER(InternalNetwork)(bstr.asOutParam());    H();
     3353            hrc = aNetworkAdapter->COMGETTER(InternalNetwork)(bstr.asOutParam());       H();
    33543354            if (!bstr.isEmpty())
    33553355            {
    … …  
    33643364                    RC_CHECK();
    33653365                }
    3366                 rc = CFGMR3InsertString(pLunL0, "Driver", "IntNet");    RC_CHECK();
    3367                 rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);         RC_CHECK();
    3368                 rc = CFGMR3InsertStringW(pCfg, "Network", bstr);         RC_CHECK();
     3366                rc = CFGMR3InsertString(pLunL0, "Driver", "IntNet");                    RC_CHECK();
     3367                rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                         RC_CHECK();
     3368                rc = CFGMR3InsertStringW(pCfg, "Network", bstr);                        RC_CHECK();
    33693369                rc = CFGMR3InsertInteger(pCfg, "TrunkType", kIntNetTrunkType_WhateverNone); RC_CHECK();
    33703370                networkName = bstr;
    … …  
    33873387            }
    33883388
    3389             rc = CFGMR3InsertString(pLunL0, "Driver", "IntNet");            RC_CHECK();
    3390             rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                 RC_CHECK();
     3389            rc = CFGMR3InsertString(pLunL0, "Driver", "IntNet");                        RC_CHECK();
     3390            rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                             RC_CHECK();
    33913391
    33923392            Bstr HifName;
    … …  
    35163516            const char *pszTrunk = szTrunkName;
    35173517
    3518             rc = CFGMR3InsertInteger(pCfg, "TrunkType", kIntNetTrunkType_NetAdp);   RC_CHECK();
    3519             rc = CFGMR3InsertString(pCfg, "Trunk", pszTrunk);                       RC_CHECK();
    3520             rc = CFGMR3InsertString(pCfg, "Network", szNetwork);                    RC_CHECK();
     3518            rc = CFGMR3InsertInteger(pCfg, "TrunkType", kIntNetTrunkType_NetAdp);       RC_CHECK();
     3519            rc = CFGMR3InsertString(pCfg, "Trunk", pszTrunk);                           RC_CHECK();
     3520            rc = CFGMR3InsertString(pCfg, "Network", szNetwork);                        RC_CHECK();
    35213521            networkName = Bstr(szNetwork);
    35223522            trunkName   = Bstr(pszTrunk);
    … …  
    35243524# endif /* defined VBOX_WITH_NETFLT*/
    35253525#elif defined(RT_OS_DARWIN)
    3526             rc = CFGMR3InsertString(pCfg, "Trunk", pszHifName);                     RC_CHECK();
    3527             rc = CFGMR3InsertString(pCfg, "Network", szNetwork);                    RC_CHECK();
    3528             rc = CFGMR3InsertInteger(pCfg, "TrunkType", kIntNetTrunkType_NetAdp);   RC_CHECK();
     3526            rc = CFGMR3InsertString(pCfg, "Trunk", pszHifName);                         RC_CHECK();
     3527            rc = CFGMR3InsertString(pCfg, "Network", szNetwork);                        RC_CHECK();
     3528            rc = CFGMR3InsertInteger(pCfg, "TrunkType", kIntNetTrunkType_NetAdp);       RC_CHECK();
    35293529            networkName = Bstr(szNetwork);
    35303530            trunkName   = Bstr(pszHifName);
    35313531            trunkType   = TRUNKTYPE_NETADP;
    35323532#else
    3533             rc = CFGMR3InsertString(pCfg, "Trunk", pszHifName);                     RC_CHECK();
    3534             rc = CFGMR3InsertString(pCfg, "Network", szNetwork);                    RC_CHECK();
    3535             rc = CFGMR3InsertInteger(pCfg, "TrunkType", kIntNetTrunkType_NetFlt);   RC_CHECK();
     3533            rc = CFGMR3InsertString(pCfg, "Trunk", pszHifName);                         RC_CHECK();
     3534            rc = CFGMR3InsertString(pCfg, "Network", szNetwork);                        RC_CHECK();
     3535            rc = CFGMR3InsertInteger(pCfg, "TrunkType", kIntNetTrunkType_NetFlt);       RC_CHECK();
    35363536            networkName = Bstr(szNetwork);
    35373537            trunkName   = Bstr(pszHifName);
    … …  
    35763576        case NetworkAttachmentType_VDE:
    35773577        {
    3578             hrc = aNetworkAdapter->COMGETTER(VDENetwork)(bstr.asOutParam());    H();
    3579             rc = CFGMR3InsertNode(pInst, "LUN#0", &pLunL0);        RC_CHECK();
    3580             rc = CFGMR3InsertString(pLunL0, "Driver", "VDE");      RC_CHECK();
    3581             rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);        RC_CHECK();
     3578            hrc = aNetworkAdapter->COMGETTER(VDENetwork)(bstr.asOutParam());            H();
     3579            rc = CFGMR3InsertNode(pInst, "LUN#0", &pLunL0);                             RC_CHECK();
     3580            rc = CFGMR3InsertString(pLunL0, "Driver", "VDE");                           RC_CHECK();
     3581            rc = CFGMR3InsertNode(pLunL0, "Config", &pCfg);                             RC_CHECK();
    35823582            if (!bstr.isEmpty())
    35833583            {
    3584                 rc = CFGMR3InsertStringW(pCfg, "Network", bstr);    RC_CHECK();
     3584                rc = CFGMR3InsertStringW(pCfg, "Network", bstr);                        RC_CHECK();
    35853585                networkName = bstr;
    35863586            }
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