VirtualBox

Changeset 52185 in vbox for trunk/src


Ignore:
Timestamp:
Jul 25, 2014 12:50:28 PM (11 years ago)
Author:
vboxsync
svn:sync-xref-src-repo-rev:
95240
Message:

VBoxManage: better check the NIC number during modifyvm

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/VBox/Frontends/VBoxManage/VBoxManageModifyVM.cpp

    r51476 r52185  
    416416}
    417417
    418 int parseScreens(const char *pcszScreens, com::SafeArray<BOOL> *pScreens)
     418static int parseScreens(const char *pcszScreens, com::SafeArray<BOOL> *pScreens)
    419419{
    420420    while (pcszScreens && *pcszScreens)
    … …  
    437437        pcszScreens = pszNext;
    438438    }
     439    return 0;
     440}
     441
     442static int parseNum(uint32_t uIndex, unsigned cMaxIndex, const char *pszName)
     443{
     444    if (   uIndex >= 1
     445        && uIndex <= cMaxIndex)
     446        return uIndex;
     447    errorArgument("Invalid %s number %u", pszName, uIndex);
    439448    return 0;
    440449}
    … …  
    12171226            case MODIFYVM_NICTRACEFILE:
    12181227            {
     1228
     1229                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1230                    break;
     1231
    12191232                ComPtr<INetworkAdapter> nic;
    1220 
    12211233                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    12221234                ASSERT(nic);
    … …  
    12281240            case MODIFYVM_NICTRACE:
    12291241            {
     1242                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1243                    break;
     1244
    12301245                ComPtr<INetworkAdapter> nic;
    1231 
    12321246                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    12331247                ASSERT(nic);
    … …  
    12391253            case MODIFYVM_NICPROPERTY:
    12401254            {
     1255                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1256                    break;
     1257
    12411258                ComPtr<INetworkAdapter> nic;
    1242 
    12431259                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    12441260                ASSERT(nic);
    … …  
    12761292            case MODIFYVM_NICTYPE:
    12771293            {
     1294                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1295                    break;
     1296
    12781297                ComPtr<INetworkAdapter> nic;
    1279 
    12801298                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    12811299                ASSERT(nic);
    … …  
    13191337            case MODIFYVM_NICSPEED:
    13201338            {
     1339                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1340                    break;
     1341
    13211342                ComPtr<INetworkAdapter> nic;
    1322 
    13231343                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    13241344                ASSERT(nic);
    … …  
    13301350            case MODIFYVM_NICBOOTPRIO:
    13311351            {
     1352                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1353                    break;
     1354
    13321355                ComPtr<INetworkAdapter> nic;
    1333 
    13341356                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    13351357                ASSERT(nic);
    … …  
    13681390                }
    13691391
     1392                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1393                    break;
     1394
    13701395                ComPtr<INetworkAdapter> nic;
    13711396                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    … …  
    13781403            case MODIFYVM_NICBWGROUP:
    13791404            {
     1405                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1406                    break;
     1407
    13801408                ComPtr<INetworkAdapter> nic;
    13811409                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    … …  
    14081436            case MODIFYVM_NIC:
    14091437            {
     1438                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1439                    break;
     1440
    14101441                ComPtr<INetworkAdapter> nic;
    1411 
    14121442                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    14131443                ASSERT(nic);
    … …  
    14661496            case MODIFYVM_CABLECONNECTED:
    14671497            {
     1498                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1499                    break;
     1500
    14681501                ComPtr<INetworkAdapter> nic;
    1469 
    14701502                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    14711503                ASSERT(nic);
    … …  
    14771509            case MODIFYVM_BRIDGEADAPTER:
    14781510            {
     1511                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1512                    break;
     1513
    14791514                ComPtr<INetworkAdapter> nic;
    1480 
    14811515                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    14821516                ASSERT(nic);
    … …  
    14961530            case MODIFYVM_HOSTONLYADAPTER:
    14971531            {
     1532                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1533                    break;
     1534
    14981535                ComPtr<INetworkAdapter> nic;
    1499 
    15001536                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    15011537                ASSERT(nic);
    … …  
    15151551            case MODIFYVM_INTNET:
    15161552            {
     1553                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1554                    break;
     1555
    15171556                ComPtr<INetworkAdapter> nic;
    1518 
    15191557                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    15201558                ASSERT(nic);
    … …  
    15341572            case MODIFYVM_GENERICDRV:
    15351573            {
     1574                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1575                    break;
     1576
    15361577                ComPtr<INetworkAdapter> nic;
    1537 
    15381578                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    15391579                ASSERT(nic);
    … …  
    15451585            case MODIFYVM_NATNETWORKNAME:
    15461586            {
     1587                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1588                    break;
     1589
    15471590                ComPtr<INetworkAdapter> nic;
    1548 
    15491591                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    15501592                ASSERT(nic);
    … …  
    15561598            case MODIFYVM_NATNET:
    15571599            {
     1600                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1601                    break;
     1602
    15581603                ComPtr<INetworkAdapter> nic;
    1559                 ComPtr<INATEngine> engine;
    1560 
    15611604                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    15621605                ASSERT(nic);
    15631606
     1607                ComPtr<INATEngine> engine;
    15641608                CHECK_ERROR(nic, COMGETTER(NATEngine)(engine.asOutParam()));
    15651609
    … …  
    15741618            case MODIFYVM_NATBINDIP:
    15751619            {
     1620                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1621                    break;
     1622
    15761623                ComPtr<INetworkAdapter> nic;
    1577                 ComPtr<INATEngine> engine;
    1578 
    15791624                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    15801625                ASSERT(nic);
    15811626
     1627                ComPtr<INATEngine> engine;
    15821628                CHECK_ERROR(nic, COMGETTER(NATEngine)(engine.asOutParam()));
     1629
    15831630                CHECK_ERROR(engine, COMSETTER(HostIP)(Bstr(ValueUnion.psz).raw()));
    15841631                break;
    … …  
    16221669                strTcpRcv = RTStrStrip(ch);
    16231670
     1671                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1672                    break;
     1673
    16241674                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    16251675                ASSERT(nic);
    … …  
    16341684            case MODIFYVM_NATPF:
    16351685            {
     1686                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1687                    break;
     1688
    16361689                ComPtr<INetworkAdapter> nic;
    1637                 ComPtr<INATEngine> engine;
    1638 
    16391690                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    16401691                ASSERT(nic);
    16411692
     1693                ComPtr<INATEngine> engine;
    16421694                CHECK_ERROR(nic, COMGETTER(NATEngine)(engine.asOutParam()));
     1695
    16431696                /* format name:proto:hostip:hostport:guestip:guestport*/
    16441697                if (RTStrCmp(ValueUnion.psz, "delete") != 0)
    … …  
    17271780            case MODIFYVM_NATTFTPPREFIX:
    17281781            {
     1782                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1783                    break;
     1784
    17291785                ComPtr<INetworkAdapter> nic;
    1730                 ComPtr<INATEngine> engine;
    1731 
    17321786                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    17331787                ASSERT(nic);
    17341788
     1789                ComPtr<INATEngine> engine;
    17351790                CHECK_ERROR(nic, COMGETTER(NATEngine)(engine.asOutParam()));
     1791
    17361792                CHECK_ERROR(engine, COMSETTER(TFTPPrefix)(Bstr(ValueUnion.psz).raw()));
    17371793                break;
    … …  
    17401796            case MODIFYVM_NATTFTPFILE:
    17411797            {
     1798                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1799                    break;
     1800
    17421801                ComPtr<INetworkAdapter> nic;
    1743                 ComPtr<INATEngine> engine;
    1744 
    17451802                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    17461803                ASSERT(nic);
    17471804
     1805                ComPtr<INATEngine> engine;
    17481806                CHECK_ERROR(nic, COMGETTER(NATEngine)(engine.asOutParam()));
     1807
    17491808                CHECK_ERROR(engine, COMSETTER(TFTPBootFile)(Bstr(ValueUnion.psz).raw()));
    17501809                break;
    … …  
    17531812            case MODIFYVM_NATTFTPSERVER:
    17541813            {
     1814                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1815                    break;
     1816
    17551817                ComPtr<INetworkAdapter> nic;
    1756                 ComPtr<INATEngine> engine;
    1757 
    17581818                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    17591819                ASSERT(nic);
    17601820
     1821                ComPtr<INATEngine> engine;
    17611822                CHECK_ERROR(nic, COMGETTER(NATEngine)(engine.asOutParam()));
     1823
    17621824                CHECK_ERROR(engine, COMSETTER(TFTPNextServer)(Bstr(ValueUnion.psz).raw()));
    17631825                break;
    … …  
    17651827            case MODIFYVM_NATDNSPASSDOMAIN:
    17661828            {
     1829                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1830                    break;
     1831
    17671832                ComPtr<INetworkAdapter> nic;
    1768                 ComPtr<INATEngine> engine;
    1769 
    17701833                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    17711834                ASSERT(nic);
    17721835
     1836                ComPtr<INATEngine> engine;
    17731837                CHECK_ERROR(nic, COMGETTER(NATEngine)(engine.asOutParam()));
     1838
    17741839                CHECK_ERROR(engine, COMSETTER(DNSPassDomain)(ValueUnion.f));
    17751840                break;
    … …  
    17781843            case MODIFYVM_NATDNSPROXY:
    17791844            {
     1845                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1846                    break;
     1847
    17801848                ComPtr<INetworkAdapter> nic;
    1781                 ComPtr<INATEngine> engine;
    1782 
    17831849                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    17841850                ASSERT(nic);
    17851851
     1852                ComPtr<INATEngine> engine;
    17861853                CHECK_ERROR(nic, COMGETTER(NATEngine)(engine.asOutParam()));
     1854
    17871855                CHECK_ERROR(engine, COMSETTER(DNSProxy)(ValueUnion.f));
    17881856                break;
    … …  
    17911859            case MODIFYVM_NATDNSHOSTRESOLVER:
    17921860            {
     1861                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1862                    break;
     1863
    17931864                ComPtr<INetworkAdapter> nic;
    1794                 ComPtr<INATEngine> engine;
    1795 
    17961865                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    17971866                ASSERT(nic);
    17981867
     1868                ComPtr<INATEngine> engine;
    17991869                CHECK_ERROR(nic, COMGETTER(NATEngine)(engine.asOutParam()));
     1870
    18001871                CHECK_ERROR(engine, COMSETTER(DNSUseHostResolver)(ValueUnion.f));
    18011872                break;
    … …  
    18031874            case MODIFYVM_MACADDRESS:
    18041875            {
     1876                if (!parseNum(GetOptState.uIndex, NetworkAdapterCount, "NIC"))
     1877                    break;
     1878
    18051879                ComPtr<INetworkAdapter> nic;
    1806 
    18071880                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    18081881                ASSERT(nic);
Note: See TracChangeset for help on using the changeset viewer.

© 2025 Oracle Support Privacy / Do Not Sell My Info Terms of Use Trademark Policy Automated Access Etiquette