VirtualBox

Ignore:
Timestamp:
Sep 23, 2010 12:57:52 PM (14 years ago)
Author:
vboxsync
Message:

com/string: Remove bool conversion operator and other convenience error operators. They are hiding programming errors (like incorrect empty string checks, and in one case a free of the wrong pointer).

File:
1 edited

Legend:

Unmodified
Added
Removed

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

    r32672 r32718  
    231231    Bstr aFilePath, empty;
    232232
    233     rc = vbox->CheckFirmwarePresent(aFirmwareType, empty,
     233    rc = vbox->CheckFirmwarePresent(aFirmwareType, empty.raw(),
    234234                                    empty.asOutParam(), aFilePath.asOutParam(), &fPresent);
    235235    if (RT_FAILURE(rc))
    … …  
    251251     * The extra data takes precedence (if non-zero).
    252252     */
    253     HRESULT hrc = pMachine->GetExtraData(Bstr("VBoxInternal2/SmcDeviceKey"), aKey);
     253    HRESULT hrc = pMachine->GetExtraData(Bstr("VBoxInternal2/SmcDeviceKey").raw(),
     254                                         aKey);
    254255    if (FAILED(hrc))
    255256        return Global::vboxStatusCodeFromCOM(hrc);
    … …  
    555556
    556557    ComPtr<IGuestOSType> guestOSType;
    557     hrc = virtualBox->GetGuestOSType(osTypeId, guestOSType.asOutParam());               H();
     558    hrc = virtualBox->GetGuestOSType(osTypeId.raw(), guestOSType.asOutParam());         H();
    558559
    559560    Bstr guestTypeFamilyId;
    … …  
    659660           - With more than one virtual CPU, raw-mode isn't a fallback option. */
    660661        fHwVirtExtForced = fHWVirtExEnabled
    661                         && (   cbRam > (_4G - cbRamHole)
     662                        && (   cbRam + cbRamHole > _4G
    662663                            || cCpus > 1);
    663664# endif
    … …  
    10461047        Bstr logoImagePath;
    10471048        hrc = biosSettings->COMGETTER(LogoImagePath)(logoImagePath.asOutParam());           H();
    1048         InsertConfigString(pCfg,   "LogoFile", Utf8Str(logoImagePath ? logoImagePath : "") );
     1049        InsertConfigString(pCfg,   "LogoFile", Utf8Str(!logoImagePath.isEmpty() ? logoImagePath : "") );
    10491050
    10501051        /*
    … …  
    10681069            char szExtraDataKey[sizeof("CustomVideoModeXX")];
    10691070            RTStrPrintf(szExtraDataKey, sizeof(szExtraDataKey), "CustomVideoMode%u", iMode);
    1070             hrc = pMachine->GetExtraData(Bstr(szExtraDataKey), bstr.asOutParam());          H();
     1071            hrc = pMachine->GetExtraData(Bstr(szExtraDataKey).raw(), bstr.asOutParam());    H();
    10711072            if (bstr.isEmpty())
    10721073                break;
    … …  
    11891190            /* Get boot args */
    11901191            Bstr bootArgs;
    1191             hrc = pMachine->GetExtraData(Bstr("VBoxInternal2/EfiBootArgs"), bootArgs.asOutParam()); H();
     1192            hrc = pMachine->GetExtraData(Bstr("VBoxInternal2/EfiBootArgs").raw(), bootArgs.asOutParam()); H();
    11921193
    11931194            /* Get device props */
    11941195            Bstr deviceProps;
    1195             hrc = pMachine->GetExtraData(Bstr("VBoxInternal2/EfiDeviceProps"), deviceProps.asOutParam()); H();
     1196            hrc = pMachine->GetExtraData(Bstr("VBoxInternal2/EfiDeviceProps").raw(), deviceProps.asOutParam()); H();
    11961197            /* Get GOP mode settings */
    11971198            uint32_t u32GopMode = UINT32_MAX;
    1198             hrc = pMachine->GetExtraData(Bstr("VBoxInternal2/EfiGopMode"), bstr.asOutParam()); H();
     1199            hrc = pMachine->GetExtraData(Bstr("VBoxInternal2/EfiGopMode").raw(), bstr.asOutParam()); H();
    11991200            if (!bstr.isEmpty())
    12001201                u32GopMode = Utf8Str(bstr).toUInt32();
    … …  
    12021203            /* UGA mode settings */
    12031204            uint32_t u32UgaHorisontal = 0;
    1204             hrc = pMachine->GetExtraData(Bstr("VBoxInternal2/EfiUgaHorizontalResolution"), bstr.asOutParam()); H();
     1205            hrc = pMachine->GetExtraData(Bstr("VBoxInternal2/EfiUgaHorizontalResolution").raw(), bstr.asOutParam()); H();
    12051206            if (!bstr.isEmpty())
    12061207                u32UgaHorisontal = Utf8Str(bstr).toUInt32();
    12071208
    12081209            uint32_t u32UgaVertical = 0;
    1209             hrc = pMachine->GetExtraData(Bstr("VBoxInternal2/EfiUgaVerticalResolution"), bstr.asOutParam()); H();
     1210            hrc = pMachine->GetExtraData(Bstr("VBoxInternal2/EfiUgaVerticalResolution").raw(), bstr.asOutParam()); H();
    12101211            if (!bstr.isEmpty())
    12111212                u32UgaVertical = Utf8Str(bstr).toUInt32();
    … …  
    14461447            /* Attach the media to the storage controllers. */
    14471448            com::SafeIfaceArray<IMediumAttachment> atts;
    1448             hrc = pMachine->GetMediumAttachmentsOfController(controllerName,
     1449            hrc = pMachine->GetMediumAttachmentsOfController(controllerName.raw(),
    14491450                                                            ComSafeArrayAsOutParam(atts)); H();
    14501451
    … …  
    14521453            {
    14531454                rc = pConsole->configMediumAttachment(pCtlInst,
    1454                                                     pszCtrlDev,
    1455                                                     ulInstance,
    1456                                                     enmBus,
    1457                                                     fUseHostIOCache,
    1458                                                     false /* fSetupMerge */,
    1459                                                     0 /* uMergeSource */,
    1460                                                     0 /* uMergeTarget */,
    1461                                                     atts[j],
    1462                                                     pConsole->mMachineState,
    1463                                                     NULL /* phrc */,
    1464                                                     false /* fAttachDetach */,
    1465                                                     false /* fForceUnmount */,
    1466                                                     pVM,
    1467                                                     paLedDevType);
     1455                                                      pszCtrlDev,
     1456                                                      ulInstance,
     1457                                                      enmBus,
     1458                                                      fUseHostIOCache,
     1459                                                      false /* fSetupMerge */,
     1460                                                      0 /* uMergeSource */,
     1461                                                      0 /* uMergeTarget */,
     1462                                                      atts[j],
     1463                                                      pConsole->mMachineState,
     1464                                                      NULL /* phrc */,
     1465                                                      false /* fAttachDetach */,
     1466                                                      false /* fForceUnmount */,
     1467                                                      pVM,
     1468                                                      paLedDevType);
    14681469                if (RT_FAILURE(rc))
    14691470                    return rc;
    … …  
    16091610            Bstr macAddr;
    16101611            hrc = networkAdapter->COMGETTER(MACAddress)(macAddr.asOutParam());              H();
    1611             Assert(macAddr);
     1612            Assert(!macAddr.isEmpty());
    16121613            Utf8Str macAddrUtf8 = macAddr;
    16131614            char *macStr = (char*)macAddrUtf8.c_str();
    … …  
    17931794        hrc = pMachine->COMGETTER(HardwareVersion)(hwVersion.asOutParam());                 H();
    17941795        InsertConfigInteger(pCfg, "RamSize",              cbRam);
    1795         if (hwVersion.compare(Bstr("1")) == 0) /* <= 2.0.x */
     1796        if (hwVersion.compare(Bstr("1").raw()) == 0) /* <= 2.0.x */
    17961797            InsertConfigInteger(pCfg, "HeapEnabled", 0);
    17971798        Bstr snapshotFolder;
    … …  
    23862387            if (i2 < cGlobalValues)
    23872388                // this is still one of the global values:
    2388                 hrc = virtualBox->GetExtraData(Bstr(strKey), bstrExtraDataValue.asOutParam());
     2389                hrc = virtualBox->GetExtraData(Bstr(strKey).raw(),
     2390                                               bstrExtraDataValue.asOutParam());
    23892391            else
    2390                 hrc = pMachine->GetExtraData(Bstr(strKey), bstrExtraDataValue.asOutParam());
     2392                hrc = pMachine->GetExtraData(Bstr(strKey).raw(),
     2393                                             bstrExtraDataValue.asOutParam());
    23912394            if (FAILED(hrc))
    23922395                LogRel(("Warning: Cannot get extra data key %s, rc = %Rrc\n", strKey.c_str(), hrc));
    … …  
    38643867                const char *pszHifName = HifNameUtf8.c_str();
    38653868                ComPtr<IHostNetworkInterface> hostInterface;
    3866                 rc = host->FindHostNetworkInterfaceByName(HifName, hostInterface.asOutParam());
     3869                rc = host->FindHostNetworkInterfaceByName(HifName.raw(),
     3870                                                          hostInterface.asOutParam());
    38673871                if (!SUCCEEDED(rc))
    38683872                {
    … …  
    40054009                Bstr tmpAddr, tmpMask;
    40064010
    4007                 hrc = virtualBox->GetExtraData(BstrFmt("HostOnly/%s/IPAddress", pszHifName), tmpAddr.asOutParam());
     4011                hrc = virtualBox->GetExtraData(BstrFmt("HostOnly/%s/IPAddress",
     4012                                                       pszHifName).raw(),
     4013                                               tmpAddr.asOutParam());
    40084014                if (SUCCEEDED(hrc) && !tmpAddr.isEmpty())
    40094015                {
    4010                     hrc = virtualBox->GetExtraData(BstrFmt("HostOnly/%s/IPNetMask", pszHifName), tmpMask.asOutParam());
     4016                    hrc = virtualBox->GetExtraData(BstrFmt("HostOnly/%s/IPNetMask",
     4017                                                           pszHifName).raw(),
     4018                                                   tmpMask.asOutParam());
    40114019                    if (SUCCEEDED(hrc) && !tmpMask.isEmpty())
    4012                         hrc = hostInterface->EnableStaticIpConfig(tmpAddr, tmpMask);
     4020                        hrc = hostInterface->EnableStaticIpConfig(tmpAddr.raw(),
     4021                                                                  tmpMask.raw());
    40134022                    else
    4014                         hrc = hostInterface->EnableStaticIpConfig(tmpAddr,
    4015                                                                 Bstr(VBOXNET_IPV4MASK_DEFAULT));
     4023                        hrc = hostInterface->EnableStaticIpConfig(tmpAddr.raw(),
     4024                                                                  Bstr(VBOXNET_IPV4MASK_DEFAULT).raw());
    40164025                }
    40174026                else
    40184027                {
    40194028                    /* Grab the IP number from the 'vboxnetX' instance number (see netif.h) */
    4020                     hrc = hostInterface->EnableStaticIpConfig(getDefaultIPv4Address(Bstr(pszHifName)),
    4021                                                             Bstr(VBOXNET_IPV4MASK_DEFAULT));
     4029                    hrc = hostInterface->EnableStaticIpConfig(getDefaultIPv4Address(Bstr(pszHifName)).raw(),
     4030                                                              Bstr(VBOXNET_IPV4MASK_DEFAULT).raw());
    40224031                }
    40234032
    40244033                ComAssertComRC(hrc); /** @todo r=bird: Why this isn't fatal? (H()) */
    40254034
    4026                 hrc = virtualBox->GetExtraData(BstrFmt("HostOnly/%s/IPV6Address", pszHifName), tmpAddr.asOutParam());
     4035                hrc = virtualBox->GetExtraData(BstrFmt("HostOnly/%s/IPV6Address",
     4036                                                       pszHifName).raw(),
     4037                                               tmpAddr.asOutParam());
    40274038                if (SUCCEEDED(hrc))
    4028                     hrc = virtualBox->GetExtraData(BstrFmt("HostOnly/%s/IPV6NetMask", pszHifName), tmpMask.asOutParam());
     4039                    hrc = virtualBox->GetExtraData(BstrFmt("HostOnly/%s/IPV6NetMask", pszHifName).raw(),
     4040                                                   tmpMask.asOutParam());
    40294041                if (SUCCEEDED(hrc) && !tmpAddr.isEmpty() && !tmpMask.isEmpty())
    40304042                {
    4031                     hrc = hostInterface->EnableStaticIpConfigV6(tmpAddr, Utf8Str(tmpMask).toUInt32());
     4043                    hrc = hostInterface->EnableStaticIpConfigV6(tmpAddr.raw(),
     4044                                                                Utf8Str(tmpMask).toUInt32());
    40324045                    ComAssertComRC(hrc); /** @todo r=bird: Why this isn't fatal? (H()) */
    40334046                }
    … …  
    40974110                         */
    40984111                        ComPtr<IDHCPServer> dhcpServer;
    4099                         hrc = virtualBox->FindDHCPServerByNetworkName(networkName, dhcpServer.asOutParam());
     4112                        hrc = virtualBox->FindDHCPServerByNetworkName(networkName.raw(),
     4113                                                                      dhcpServer.asOutParam());
    41004114                        if (SUCCEEDED(hrc))
    41014115                        {
    … …  
    41104124
    41114125                            if (fEnabled)
    4112                                 hrc = dhcpServer->Start(networkName, trunkName, trunkType);
     4126                                hrc = dhcpServer->Start(networkName.raw(),
     4127                                                        trunkName.raw(),
     4128                                                        trunkType.raw());
    41134129                        }
    41144130                        else
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