VirtualBox

Ignore:
Timestamp:
Dec 11, 2009 12:28:58 PM (15 years ago)
Author:
vboxsync
Message:
VBoxManage: 32-bit getopt index; kicked out non-working (&& vs
) parameter checking that main shoudl be doing.
Location:
trunk/src/VBox/Frontends/VBoxManage
Files:
2 edited

Legend:

Unmodified
Added
Removed

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

    r24979 r25324  
    495495            case MODIFYVM_BOOT:
    496496            {
    497                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > 4))
    498                     return errorSyntax(USAGE_MODIFYVM,
    499                                        "Missing or Invalid boot slot number in '%s'",
    500                                        GetOptState.pDef->pszLong);
    501 
    502497                if (!strcmp(ValueUnion.psz, "none"))
    503498                {
    … …  
    522517                else
    523518                    return errorArgument("Invalid boot device '%s'", ValueUnion.psz);
    524 
    525519                break;
    526520            }
    … …  
    665659                CHECK_ERROR(machine, GetStorageControllerByName(Bstr("SATA"), SataCtl.asOutParam()));
    666660
    667                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > 4))
    668                     return errorSyntax(USAGE_MODIFYVM,
    669                                        "Missing or Invalid SATA boot slot number in '%s'",
    670                                        GetOptState.pDef->pszLong);
    671 
    672                 if ((ValueUnion.u32 < 1) && (ValueUnion.u32 > 30))
    673                     return errorSyntax(USAGE_MODIFYVM,
    674                                        "Missing or Invalid SATA port number in '%s'",
    675                                        GetOptState.pDef->pszLong);
    676 
    677661                if (SUCCEEDED(rc))
    678662                    CHECK_ERROR(SataCtl, SetIDEEmulationPort(GetOptState.uIndex, ValueUnion.u32));
    679 
    680663                break;
    681664            }
    … …  
    688671                if (SUCCEEDED(rc) && ValueUnion.u32 > 0)
    689672                    CHECK_ERROR(SataCtl, COMSETTER(PortCount)(ValueUnion.u32));
    690 
    691673                break;
    692674            }
    … …  
    694676            case MODIFYVM_SATAPORT: // deprecated
    695677            {
    696                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > 30))
    697                     return errorSyntax(USAGE_MODIFYVM,
    698                                        "Missing or Invalid SATA port number in '%s'",
    699                                        GetOptState.pDef->pszLong);
    700 
    701678                if (!strcmp(ValueUnion.psz, "none"))
    702679                {
    … …  
    717694                            /* open the new hard disk object */
    718695                            CHECK_ERROR(a->virtualBox,
    719                                          OpenHardDisk(Bstr(ValueUnion.psz), AccessMode_ReadWrite,
    720                                                       false, Bstr(""), false,
    721                                                       Bstr(""), hardDisk.asOutParam()));
     696                                        OpenHardDisk(Bstr(ValueUnion.psz), AccessMode_ReadWrite,
     697                                                     false, Bstr(""), false,
     698                                                     Bstr(""), hardDisk.asOutParam()));
    722699                        }
    723700                    }
    … …  
    726703                        hardDisk->COMGETTER(Id)(uuid.asOutParam());
    727704                        CHECK_ERROR(machine,
    728                                      AttachDevice(Bstr("SATA"), GetOptState.uIndex,
    729                                                   0, DeviceType_HardDisk, uuid));
     705                                    AttachDevice(Bstr("SATA"), GetOptState.uIndex,
     706                                                 0, DeviceType_HardDisk, uuid));
    730707                    }
    731708                    else
    … …  
    752729            case MODIFYVM_SCSIPORT: // deprecated
    753730            {
    754                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > 16))
    755                     return errorSyntax(USAGE_MODIFYVM,
    756                                        "Missing or Invalid SCSI port number in '%s'",
    757                                        GetOptState.pDef->pszLong);
    758 
    759731                if (!strcmp(ValueUnion.psz, "none"))
    760732                {
    … …  
    1000972                ComPtr<INetworkAdapter> nic;
    1001973
    1002                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > NetworkAdapterCount))
    1003                     return errorSyntax(USAGE_MODIFYVM,
    1004                                        "Missing or Invalid NIC slot number in '%s'",
    1005                                        GetOptState.pDef->pszLong);
    1006 
    1007974                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    1008975                ASSERT(nic);
    … …  
    1016983                ComPtr<INetworkAdapter> nic;
    1017984
    1018                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > NetworkAdapterCount))
    1019                     return errorSyntax(USAGE_MODIFYVM,
    1020                                        "Missing or Invalid NIC slot number in '%s'",
    1021                                        GetOptState.pDef->pszLong);
    1022 
    1023985                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    1024986                ASSERT(nic);
    … …  
    1031993            {
    1032994                ComPtr<INetworkAdapter> nic;
    1033 
    1034                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > NetworkAdapterCount))
    1035                     return errorSyntax(USAGE_MODIFYVM,
    1036                                        "Missing or Invalid NIC slot number in '%s'",
    1037                                        GetOptState.pDef->pszLong);
    1038995
    1039996                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    … …  
    10701027                else
    10711028                {
    1072                     errorArgument("Invalid NIC type '%s' specified for NIC %lu", ValueUnion.psz, GetOptState.uIndex);
     1029                    errorArgument("Invalid NIC type '%s' specified for NIC %u", ValueUnion.psz, GetOptState.uIndex);
    10731030                    rc = E_FAIL;
    10741031                }
    … …  
    10791036            {
    10801037                ComPtr<INetworkAdapter> nic;
    1081 
    1082                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > NetworkAdapterCount))
    1083                     return errorSyntax(USAGE_MODIFYVM,
    1084                                        "Missing or Invalid NIC slot number in '%s'",
    1085                                        GetOptState.pDef->pszLong);
    1086 
    1087                 if ((ValueUnion.u32 < 1000) && (ValueUnion.u32 > 4000000))
    1088                 {
    1089                     errorArgument("Invalid --nicspeed%lu argument '%u'", GetOptState.uIndex, ValueUnion.u32);
    1090                     rc = E_FAIL;
    1091                     break;
    1092                 }
    10931038
    10941039                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    … …  
    11031048                ComPtr<INetworkAdapter> nic;
    11041049
    1105                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > NetworkAdapterCount))
    1106                     return errorSyntax(USAGE_MODIFYVM,
    1107                                        "Missing or Invalid NIC slot number in '%s'",
    1108                                        GetOptState.pDef->pszLong);
    1109 
    11101050                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    11111051                ASSERT(nic);
    … …  
    11461086                else
    11471087                {
    1148                     errorArgument("Invalid type '%s' specfied for NIC %lu", ValueUnion.psz, GetOptState.uIndex);
     1088                    errorArgument("Invalid type '%s' specfied for NIC %u", ValueUnion.psz, GetOptState.uIndex);
    11491089                    rc = E_FAIL;
    11501090                }
    … …  
    11551095            {
    11561096                ComPtr<INetworkAdapter> nic;
    1157 
    1158                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > NetworkAdapterCount))
    1159                     return errorSyntax(USAGE_MODIFYVM,
    1160                                        "Missing or Invalid NIC slot number in '%s'",
    1161                                        GetOptState.pDef->pszLong);
    11621097
    11631098                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    … …  
    11731108                ComPtr<INetworkAdapter> nic;
    11741109
    1175                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > NetworkAdapterCount))
    1176                     return errorSyntax(USAGE_MODIFYVM,
    1177                                        "Missing or Invalid NIC slot number in '%s'",
    1178                                        GetOptState.pDef->pszLong);
    1179 
    11801110                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    11811111                ASSERT(nic);
    … …  
    11971127                ComPtr<INetworkAdapter> nic;
    11981128
    1199                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > NetworkAdapterCount))
    1200                     return errorSyntax(USAGE_MODIFYVM,
    1201                                        "Missing or Invalid NIC slot number in '%s'",
    1202                                        GetOptState.pDef->pszLong);
    1203 
    12041129                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    12051130                ASSERT(nic);
    … …  
    12211146                ComPtr<INetworkAdapter> nic;
    12221147
    1223                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > NetworkAdapterCount))
    1224                     return errorSyntax(USAGE_MODIFYVM,
    1225                                        "Missing or Invalid NIC slot number in '%s'",
    1226                                        GetOptState.pDef->pszLong);
    1227 
    12281148                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    12291149                ASSERT(nic);
    12301150
    12311151                CHECK_ERROR(nic, COMSETTER(NATNetwork)(Bstr(ValueUnion.psz)));
    1232 
    12331152                break;
    12341153            }
    … …  
    12371156            {
    12381157                ComPtr<INetworkAdapter> nic;
    1239 
    1240                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > NetworkAdapterCount))
    1241                     return errorSyntax(USAGE_MODIFYVM,
    1242                                        "Missing or Invalid NIC slot number in '%s'",
    1243                                        GetOptState.pDef->pszLong);
    12441158
    12451159                CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    … …  
    12621176                ComPtr<ISerialPort> uart;
    12631177                char *pszIRQ = NULL;
    1264 
    1265                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > SerialPortCount))
    1266                     return errorSyntax(USAGE_MODIFYVM,
    1267                                        "Missing or Invalid Serial Port number in '%s'",
    1268                                        GetOptState.pDef->pszLong);
    12691178
    12701179                CHECK_ERROR_BREAK(machine, GetSerialPort(GetOptState.uIndex - 1, uart.asOutParam()));
    … …  
    13161225                ComPtr<ISerialPort> uart;
    13171226
    1318                 if ((GetOptState.uIndex < 1) && (GetOptState.uIndex > SerialPortCount))
    1319                     return errorSyntax(USAGE_MODIFYVM,
    1320                                        "Missing or Invalid Serial Port number in '%s'",
    1321                                        GetOptState.pDef->pszLong);
    1322 
    13231227                CHECK_ERROR_BREAK(machine, GetSerialPort(GetOptState.uIndex - 1, uart.asOutParam()));
    13241228                ASSERT(uart);

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

    r24780 r25324  
    715715            case 'e':   // sataideemulation
    716716            {
    717                 if ((GetState.uIndex < 1) && (GetState.uIndex > 4))
    718                     return errorSyntax(USAGE_STORAGECONTROLLER,
    719                                        "Missing or Invalid SATA boot slot number in '%s'",
    720                                        GetState.pDef->pszLong);
    721 
    722                 if ((ValueUnion.u32 < 1) && (ValueUnion.u32 > 30))
    723                     return errorSyntax(USAGE_STORAGECONTROLLER,
    724                                        "Missing or Invalid SATA port number in '%s'",
    725                                        GetState.pDef->pszLong);
    726 
    727717                satabootdev = GetState.uIndex;
    728718                sataidedev = ValueUnion.u32;
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