VirtualBox

Changeset 55234 in vbox


Ignore:
Timestamp:
Apr 14, 2015 9:13:33 AM (10 years ago)
Author:
vboxsync
Message:

VBoxManage: ISession->GetMachine() should not overwrite 'machine' but use a separate 'sessionMachine' variable

Location:
trunk/src/VBox/Frontends/VBoxManage
Files:
3 edited

Legend:

Unmodified
Added
Removed

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

    r55214 r55234  
    650650            do
    651651            {
    652                 ComPtr<IMachine> console;
    653                 CHECK_ERROR_BREAK(a->session, COMGETTER(Machine)(machine.asOutParam()));
    654                 CHECK_ERROR_BREAK(console, DiscardSavedState(true /* fDeleteFile */));
     652                ComPtr<IMachine> sessionMachine;
     653                CHECK_ERROR_BREAK(a->session, COMGETTER(Machine)(sessionMachine.asOutParam()));
     654                CHECK_ERROR_BREAK(sessionMachine, DiscardSavedState(true /* fDeleteFile */));
    655655            } while (0);
    656656            CHECK_ERROR_BREAK(a->session, UnlockMachine());
    … …  
    687687            do
    688688            {
    689                 ComPtr<IMachine> machine;
    690                 CHECK_ERROR_BREAK(a->session, COMGETTER(Machine)(machine.asOutParam()));
    691                 CHECK_ERROR_BREAK(machine, AdoptSavedState(Bstr(szStateFileAbs).raw()));
     689                ComPtr<IMachine> sessionMachine;
     690                CHECK_ERROR_BREAK(a->session, COMGETTER(Machine)(sessionMachine.asOutParam()));
     691                CHECK_ERROR_BREAK(sessionMachine, AdoptSavedState(Bstr(szStateFileAbs).raw()));
    692692            } while (0);
    693693            CHECK_ERROR_BREAK(a->session, UnlockMachine());
    … …  
    807807            CHECK_ERROR_RET(machine, LockMachine(a->session, LockType_Shared), 1);
    808808            /* get the session machine */
    809             CHECK_ERROR_RET(a->session, COMGETTER(Machine)(machine.asOutParam()), 1);
     809            ComPtr<IMachine> sessionMachine;
     810            CHECK_ERROR_RET(a->session, COMGETTER(Machine)(sessionMachine.asOutParam()), 1);
    810811            /** @todo passing NULL is deprecated */
    811812            if (a->argc < 3)
    812                 CHECK_ERROR(machine, SetExtraData(Bstr(a->argv[1]).raw(),
    813                                                   NULL));
     813                CHECK_ERROR(sessionMachine, SetExtraData(Bstr(a->argv[1]).raw(),
     814                                                         NULL));
    814815            else if (a->argc == 3)
    815                 CHECK_ERROR(machine, SetExtraData(Bstr(a->argv[1]).raw(),
    816                                                   Bstr(a->argv[2]).raw()));
     816                CHECK_ERROR(sessionMachine, SetExtraData(Bstr(a->argv[1]).raw(),
     817                                                         Bstr(a->argv[2]).raw()));
    817818            else
    818819                return errorSyntax(USAGE_SETEXTRADATA, "Too many parameters");
    … …  
    989990            /* open an existing session for the VM */
    990991            CHECK_ERROR_RET(machine, LockMachine(a->session, LockType_Shared), 1);
     992
    991993            /* get the session machine */
    992             CHECK_ERROR_RET(a->session, COMGETTER(Machine)(machine.asOutParam()), 1);
     994            ComPtr<IMachine> sessionMachine;
     995            CHECK_ERROR_RET(a->session, COMGETTER(Machine)(sessionMachine.asOutParam()), 1);
     996
    993997            /* get the session console */
    994998            CHECK_ERROR_RET(a->session, COMGETTER(Console)(console.asOutParam()), 1);
    … …  
    10061010
    10071011            /* get the mutable session machine */
    1008             a->session->COMGETTER(Machine)(machine.asOutParam());
    1009 
    1010             CHECK_ERROR(machine, CreateSharedFolder(Bstr(name).raw(),
    1011                                                     Bstr(hostpath).raw(),
    1012                                                     fWritable, fAutoMount));
     1012            ComPtr<IMachine> sessionMachine;
     1013            a->session->COMGETTER(Machine)(sessionMachine.asOutParam());
     1014
     1015            CHECK_ERROR(sessionMachine, CreateSharedFolder(Bstr(name).raw(),
     1016                                                           Bstr(hostpath).raw(),
     1017                                                           fWritable, fAutoMount));
    10131018            if (SUCCEEDED(rc))
    1014                 CHECK_ERROR(machine, SaveSettings());
     1019                CHECK_ERROR(sessionMachine, SaveSettings());
    10151020
    10161021            a->session->UnlockMachine();
    … …  
    10561061            CHECK_ERROR_RET(machine, LockMachine(a->session, LockType_Shared), 1);
    10571062            /* get the session machine */
    1058             CHECK_ERROR_RET(a->session, COMGETTER(Machine)(machine.asOutParam()), 1);
     1063            ComPtr<IMachine> sessionMachine;
     1064            CHECK_ERROR_RET(a->session, COMGETTER(Machine)(sessionMachine.asOutParam()), 1);
    10591065            /* get the session console */
    10601066            CHECK_ERROR_RET(a->session, COMGETTER(Console)(console.asOutParam()), 1);
    … …  
    10711077
    10721078            /* get the mutable session machine */
    1073             a->session->COMGETTER(Machine)(machine.asOutParam());
    1074 
    1075             CHECK_ERROR(machine, RemoveSharedFolder(Bstr(name).raw()));
     1079            ComPtr<IMachine> sessionMachine;
     1080            a->session->COMGETTER(Machine)(sessionMachine.asOutParam());
     1081
     1082            CHECK_ERROR(sessionMachine, RemoveSharedFolder(Bstr(name).raw()));
    10761083
    10771084            /* commit and close the session */
    1078             CHECK_ERROR(machine, SaveSettings());
     1085            CHECK_ERROR(sessionMachine, SaveSettings());
    10791086            a->session->UnlockMachine();
    10801087        }

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

    r54911 r55234  
    460460    HRESULT rc;
    461461    Bstr name;
    462     RTGETOPTUNION ValueUnion;
    463     RTGETOPTSTATE GetOptState;
    464     ComPtr<IMachine> machine;
    465     ComPtr<IBIOSSettings> biosSettings;
    466462
    467463    /* VM ID + at least one parameter. Parameter arguments are checked
    … …  
    470466        return errorSyntax(USAGE_MODIFYVM, "Not enough parameters");
    471467
    472     /* try to find the given machine */
     468    /* try to find the given sessionMachine */
     469    ComPtr<IMachine> machine;
    473470    CHECK_ERROR_RET(a->virtualBox, FindMachine(Bstr(a->argv[0]).raw(),
    474471                                               machine.asOutParam()), 1);
    … …  
    481478    CHECK_ERROR_RET(machine, LockMachine(a->session, LockType_Write), 1);
    482479
    483     /* get the mutable session machine */
    484     CHECK_ERROR_RET(a->session, COMGETTER(Machine)(machine.asOutParam()), 1);
    485     machine->COMGETTER(BIOSSettings)(biosSettings.asOutParam());
    486 
     480    /* get the mutable session sessionMachine */
     481    ComPtr<IMachine> sessionMachine;
     482    CHECK_ERROR_RET(a->session, COMGETTER(Machine)(sessionMachine.asOutParam()), 1);
     483
     484    ComPtr<IBIOSSettings> biosSettings;
     485    sessionMachine->COMGETTER(BIOSSettings)(biosSettings.asOutParam());
     486
     487    RTGETOPTSTATE GetOptState;
    487488    RTGetOptInit(&GetOptState, a->argc, a->argv, g_aModifyVMOptions,
    488489                 RT_ELEMENTS(g_aModifyVMOptions), 1, RTGETOPTINIT_FLAGS_NO_STD_OPTS);
    489490
     491    RTGETOPTUNION ValueUnion;
    490492    while (   SUCCEEDED (rc)
    491493           && (c = RTGetOpt(&GetOptState, &ValueUnion)))
    … …  
    495497            case MODIFYVM_NAME:
    496498            {
    497                 CHECK_ERROR(machine, COMSETTER(Name)(Bstr(ValueUnion.psz).raw()));
     499                CHECK_ERROR(sessionMachine, COMSETTER(Name)(Bstr(ValueUnion.psz).raw()));
    498500                break;
    499501            }
    … …  
    502504                com::SafeArray<BSTR> groups;
    503505                parseGroups(ValueUnion.psz, &groups);
    504                 CHECK_ERROR(machine, COMSETTER(Groups)(ComSafeArrayAsInParam(groups)));
     506                CHECK_ERROR(sessionMachine, COMSETTER(Groups)(ComSafeArrayAsInParam(groups)));
    505507                break;
    506508            }
    507509            case MODIFYVM_DESCRIPTION:
    508510            {
    509                 CHECK_ERROR(machine, COMSETTER(Description)(Bstr(ValueUnion.psz).raw()));
     511                CHECK_ERROR(sessionMachine, COMSETTER(Description)(Bstr(ValueUnion.psz).raw()));
    510512                break;
    511513            }
    … …  
    517519                if (SUCCEEDED(rc) && guestOSType)
    518520                {
    519                     CHECK_ERROR(machine, COMSETTER(OSTypeId)(Bstr(ValueUnion.psz).raw()));
     521                    CHECK_ERROR(sessionMachine, COMSETTER(OSTypeId)(Bstr(ValueUnion.psz).raw()));
    520522                }
    521523                else
    … …  
    560562                }
    561563                RTFileClose(iconFile);
    562                 CHECK_ERROR(machine, COMSETTER(Icon)(ComSafeArrayAsInParam(icon)));
     564                CHECK_ERROR(sessionMachine, COMSETTER(Icon)(ComSafeArrayAsInParam(icon)));
    563565                break;
    564566            }
    … …  
    566568            case MODIFYVM_MEMORY:
    567569            {
    568                 CHECK_ERROR(machine, COMSETTER(MemorySize)(ValueUnion.u32));
     570                CHECK_ERROR(sessionMachine, COMSETTER(MemorySize)(ValueUnion.u32));
    569571                break;
    570572            }
    … …  
    572574            case MODIFYVM_PAGEFUSION:
    573575            {
    574                 CHECK_ERROR(machine, COMSETTER(PageFusionEnabled)(ValueUnion.f));
     576                CHECK_ERROR(sessionMachine, COMSETTER(PageFusionEnabled)(ValueUnion.f));
    575577                break;
    576578            }
    … …  
    578580            case MODIFYVM_VRAM:
    579581            {
    580                 CHECK_ERROR(machine, COMSETTER(VRAMSize)(ValueUnion.u32));
     582                CHECK_ERROR(sessionMachine, COMSETTER(VRAMSize)(ValueUnion.u32));
    581583                break;
    582584            }
    … …  
    586588                if (!RTStrICmp(ValueUnion.psz, "efi"))
    587589                {
    588                     CHECK_ERROR(machine, COMSETTER(FirmwareType)(FirmwareType_EFI));
     590                    CHECK_ERROR(sessionMachine, COMSETTER(FirmwareType)(FirmwareType_EFI));
    589591                }
    590592                else if (!RTStrICmp(ValueUnion.psz, "efi32"))
    591593                {
    592                     CHECK_ERROR(machine, COMSETTER(FirmwareType)(FirmwareType_EFI32));
     594                    CHECK_ERROR(sessionMachine, COMSETTER(FirmwareType)(FirmwareType_EFI32));
    593595                }
    594596                else if (!RTStrICmp(ValueUnion.psz, "efi64"))
    595597                {
    596                     CHECK_ERROR(machine, COMSETTER(FirmwareType)(FirmwareType_EFI64));
     598                    CHECK_ERROR(sessionMachine, COMSETTER(FirmwareType)(FirmwareType_EFI64));
    597599                }
    598600                else if (!RTStrICmp(ValueUnion.psz, "efidual"))
    599601                {
    600                     CHECK_ERROR(machine, COMSETTER(FirmwareType)(FirmwareType_EFIDUAL));
     602                    CHECK_ERROR(sessionMachine, COMSETTER(FirmwareType)(FirmwareType_EFIDUAL));
    601603                }
    602604                else if (!RTStrICmp(ValueUnion.psz, "bios"))
    603605                {
    604                     CHECK_ERROR(machine, COMSETTER(FirmwareType)(FirmwareType_BIOS));
     606                    CHECK_ERROR(sessionMachine, COMSETTER(FirmwareType)(FirmwareType_BIOS));
    605607                }
    606608                else
    … …  
    626628            case MODIFYVM_PAE:
    627629            {
    628                 CHECK_ERROR(machine, SetCPUProperty(CPUPropertyType_PAE, ValueUnion.f));
     630                CHECK_ERROR(sessionMachine, SetCPUProperty(CPUPropertyType_PAE, ValueUnion.f));
    629631                break;
    630632            }
    … …  
    632634            case MODIFYVM_LONGMODE:
    633635            {
    634                 CHECK_ERROR(machine, SetCPUProperty(CPUPropertyType_LongMode, ValueUnion.f));
     636                CHECK_ERROR(sessionMachine, SetCPUProperty(CPUPropertyType_LongMode, ValueUnion.f));
    635637                break;
    636638            }
    … …  
    638640            case MODIFYVM_SYNTHCPU:
    639641            {
    640                 CHECK_ERROR(machine, SetCPUProperty(CPUPropertyType_Synthetic, ValueUnion.f));
     642                CHECK_ERROR(sessionMachine, SetCPUProperty(CPUPropertyType_Synthetic, ValueUnion.f));
    641643                break;
    642644            }
    … …  
    644646            case MODIFYVM_TFRESET:
    645647            {
    646                 CHECK_ERROR(machine, SetCPUProperty(CPUPropertyType_TripleFaultReset, ValueUnion.f));
     648                CHECK_ERROR(sessionMachine, SetCPUProperty(CPUPropertyType_TripleFaultReset, ValueUnion.f));
    647649                break;
    648650            }
    … …  
    652654                if (   !RTStrICmp(ValueUnion.psz, "none")
    653655                    || !RTStrICmp(ValueUnion.psz, "disabled"))
    654                     CHECK_ERROR(machine, COMSETTER(ParavirtProvider)(ParavirtProvider_None));
     656                    CHECK_ERROR(sessionMachine, COMSETTER(ParavirtProvider)(ParavirtProvider_None));
    655657                else if (!RTStrICmp(ValueUnion.psz, "default"))
    656                     CHECK_ERROR(machine, COMSETTER(ParavirtProvider)(ParavirtProvider_Default));
     658                    CHECK_ERROR(sessionMachine, COMSETTER(ParavirtProvider)(ParavirtProvider_Default));
    657659                else if (!RTStrICmp(ValueUnion.psz, "legacy"))
    658                     CHECK_ERROR(machine, COMSETTER(ParavirtProvider)(ParavirtProvider_Legacy));
     660                    CHECK_ERROR(sessionMachine, COMSETTER(ParavirtProvider)(ParavirtProvider_Legacy));
    659661                else if (!RTStrICmp(ValueUnion.psz, "minimal"))
    660                     CHECK_ERROR(machine, COMSETTER(ParavirtProvider)(ParavirtProvider_Minimal));
     662                    CHECK_ERROR(sessionMachine, COMSETTER(ParavirtProvider)(ParavirtProvider_Minimal));
    661663                else if (!RTStrICmp(ValueUnion.psz, "hyperv"))
    662                     CHECK_ERROR(machine, COMSETTER(ParavirtProvider)(ParavirtProvider_HyperV));
     664                    CHECK_ERROR(sessionMachine, COMSETTER(ParavirtProvider)(ParavirtProvider_HyperV));
    663665                else if (!RTStrICmp(ValueUnion.psz, "kvm"))
    664                     CHECK_ERROR(machine, COMSETTER(ParavirtProvider)(ParavirtProvider_KVM));
     666                    CHECK_ERROR(sessionMachine, COMSETTER(ParavirtProvider)(ParavirtProvider_KVM));
    665667                else
    666668                {
    … …  
    673675            case MODIFYVM_HWVIRTEX:
    674676            {
    675                 CHECK_ERROR(machine, SetHWVirtExProperty(HWVirtExPropertyType_Enabled, ValueUnion.f));
     677                CHECK_ERROR(sessionMachine, SetHWVirtExProperty(HWVirtExPropertyType_Enabled, ValueUnion.f));
    676678                break;
    677679            }
    … …  
    691693                    aValue[i] = ValueUnion.u32;
    692694                }
    693                 CHECK_ERROR(machine, SetCPUIDLeaf(id, aValue[0], aValue[1], aValue[2], aValue[3]));
     695                CHECK_ERROR(sessionMachine, SetCPUIDLeaf(id, aValue[0], aValue[1], aValue[2], aValue[3]));
    694696                break;
    695697            }
    … …  
    697699            case MODIFYVM_DELCPUID:
    698700            {
    699                 CHECK_ERROR(machine, RemoveCPUIDLeaf(ValueUnion.u32));
     701                CHECK_ERROR(sessionMachine, RemoveCPUIDLeaf(ValueUnion.u32));
    700702                break;
    701703            }
    … …  
    703705            case MODIFYVM_DELALLCPUID:
    704706            {
    705                 CHECK_ERROR(machine, RemoveAllCPUIDLeaves());
     707                CHECK_ERROR(sessionMachine, RemoveAllCPUIDLeaves());
    706708                break;
    707709            }
    … …  
    709711            case MODIFYVM_NESTEDPAGING:
    710712            {
    711                 CHECK_ERROR(machine, SetHWVirtExProperty(HWVirtExPropertyType_NestedPaging, ValueUnion.f));
     713                CHECK_ERROR(sessionMachine, SetHWVirtExProperty(HWVirtExPropertyType_NestedPaging, ValueUnion.f));
    712714                break;
    713715            }
    … …  
    715717            case MODIFYVM_LARGEPAGES:
    716718            {
    717                 CHECK_ERROR(machine, SetHWVirtExProperty(HWVirtExPropertyType_LargePages, ValueUnion.f));
     719                CHECK_ERROR(sessionMachine, SetHWVirtExProperty(HWVirtExPropertyType_LargePages, ValueUnion.f));
    718720                break;
    719721            }
    … …  
    721723            case MODIFYVM_VTXVPID:
    722724            {
    723                 CHECK_ERROR(machine, SetHWVirtExProperty(HWVirtExPropertyType_VPID, ValueUnion.f));
     725                CHECK_ERROR(sessionMachine, SetHWVirtExProperty(HWVirtExPropertyType_VPID, ValueUnion.f));
    724726                break;
    725727            }
    … …  
    727729            case MODIFYVM_VTXUX:
    728730            {
    729                 CHECK_ERROR(machine, SetHWVirtExProperty(HWVirtExPropertyType_UnrestrictedExecution, ValueUnion.f));
     731                CHECK_ERROR(sessionMachine, SetHWVirtExProperty(HWVirtExPropertyType_UnrestrictedExecution, ValueUnion.f));
    730732                break;
    731733            }
    … …  
    733735            case MODIFYVM_CPUS:
    734736            {
    735                 CHECK_ERROR(machine, COMSETTER(CPUCount)(ValueUnion.u32));
     737                CHECK_ERROR(sessionMachine, COMSETTER(CPUCount)(ValueUnion.u32));
    736738                break;
    737739            }
    … …  
    739741            case MODIFYVM_RTCUSEUTC:
    740742            {
    741                 CHECK_ERROR(machine, COMSETTER(RTCUseUTC)(ValueUnion.f));
     743                CHECK_ERROR(sessionMachine, COMSETTER(RTCUseUTC)(ValueUnion.f));
    742744                break;
    743745            }
    … …  
    745747            case MODIFYVM_CPUHOTPLUG:
    746748            {
    747                 CHECK_ERROR(machine, COMSETTER(CPUHotPlugEnabled)(ValueUnion.f));
     749                CHECK_ERROR(sessionMachine, COMSETTER(CPUHotPlugEnabled)(ValueUnion.f));
    748750                break;
    749751            }
    … …  
    751753            case MODIFYVM_PLUGCPU:
    752754            {
    753                 CHECK_ERROR(machine, HotPlugCPU(ValueUnion.u32));
     755                CHECK_ERROR(sessionMachine, HotPlugCPU(ValueUnion.u32));
    754756                break;
    755757            }
    … …  
    757759            case MODIFYVM_UNPLUGCPU:
    758760            {
    759                 CHECK_ERROR(machine, HotUnplugCPU(ValueUnion.u32));
     761                CHECK_ERROR(sessionMachine, HotUnplugCPU(ValueUnion.u32));
    760762                break;
    761763            }
    … …  
    763765            case MODIFYVM_CPU_EXECTUION_CAP:
    764766            {
    765                 CHECK_ERROR(machine, COMSETTER(CPUExecutionCap)(ValueUnion.u32));
     767                CHECK_ERROR(sessionMachine, COMSETTER(CPUExecutionCap)(ValueUnion.u32));
    766768                break;
    767769            }
    … …  
    771773                if (   !RTStrICmp(ValueUnion.psz, "none")
    772774                    || !RTStrICmp(ValueUnion.psz, "disabled"))
    773                     CHECK_ERROR(machine, COMSETTER(GraphicsControllerType)(GraphicsControllerType_Null));
     775                    CHECK_ERROR(sessionMachine, COMSETTER(GraphicsControllerType)(GraphicsControllerType_Null));
    774776                else if (   !RTStrICmp(ValueUnion.psz, "vboxvga")
    775777                         || !RTStrICmp(ValueUnion.psz, "vbox")
    776778                         || !RTStrICmp(ValueUnion.psz, "vga")
    777779                         || !RTStrICmp(ValueUnion.psz, "vesa"))
    778                     CHECK_ERROR(machine, COMSETTER(GraphicsControllerType)(GraphicsControllerType_VBoxVGA));
     780                    CHECK_ERROR(sessionMachine, COMSETTER(GraphicsControllerType)(GraphicsControllerType_VBoxVGA));
    779781#ifdef VBOX_WITH_VMSVGA
    780782                else if (   !RTStrICmp(ValueUnion.psz, "vmsvga")
    781783                         || !RTStrICmp(ValueUnion.psz, "vmware"))
    782                     CHECK_ERROR(machine, COMSETTER(GraphicsControllerType)(GraphicsControllerType_VMSVGA));
     784                    CHECK_ERROR(sessionMachine, COMSETTER(GraphicsControllerType)(GraphicsControllerType_VMSVGA));
    783785#endif
    784786                else
    … …  
    792794            case MODIFYVM_MONITORCOUNT:
    793795            {
    794                 CHECK_ERROR(machine, COMSETTER(MonitorCount)(ValueUnion.u32));
     796                CHECK_ERROR(sessionMachine, COMSETTER(MonitorCount)(ValueUnion.u32));
    795797                break;
    796798            }
    … …  
    798800            case MODIFYVM_ACCELERATE3D:
    799801            {
    800                 CHECK_ERROR(machine, COMSETTER(Accelerate3DEnabled)(ValueUnion.f));
     802                CHECK_ERROR(sessionMachine, COMSETTER(Accelerate3DEnabled)(ValueUnion.f));
    801803                break;
    802804            }
    … …  
    805807            case MODIFYVM_ACCELERATE2DVIDEO:
    806808            {
    807                 CHECK_ERROR(machine, COMSETTER(Accelerate2DVideoEnabled)(ValueUnion.f));
     809                CHECK_ERROR(sessionMachine, COMSETTER(Accelerate2DVideoEnabled)(ValueUnion.f));
    808810                break;
    809811            }
    … …  
    872874                if (!RTStrICmp(ValueUnion.psz, "none"))
    873875                {
    874                     CHECK_ERROR(machine, SetBootOrder(GetOptState.uIndex, DeviceType_Null));
     876                    CHECK_ERROR(sessionMachine, SetBootOrder(GetOptState.uIndex, DeviceType_Null));
    875877                }
    876878                else if (!RTStrICmp(ValueUnion.psz, "floppy"))
    877879                {
    878                     CHECK_ERROR(machine, SetBootOrder(GetOptState.uIndex, DeviceType_Floppy));
     880                    CHECK_ERROR(sessionMachine, SetBootOrder(GetOptState.uIndex, DeviceType_Floppy));
    879881                }
    880882                else if (!RTStrICmp(ValueUnion.psz, "dvd"))
    881883                {
    882                     CHECK_ERROR(machine, SetBootOrder(GetOptState.uIndex, DeviceType_DVD));
     884                    CHECK_ERROR(sessionMachine, SetBootOrder(GetOptState.uIndex, DeviceType_DVD));
    883885                }
    884886                else if (!RTStrICmp(ValueUnion.psz, "disk"))
    885887                {
    886                     CHECK_ERROR(machine, SetBootOrder(GetOptState.uIndex, DeviceType_HardDisk));
     888                    CHECK_ERROR(sessionMachine, SetBootOrder(GetOptState.uIndex, DeviceType_HardDisk));
    887889                }
    888890                else if (!RTStrICmp(ValueUnion.psz, "net"))
    889891                {
    890                     CHECK_ERROR(machine, SetBootOrder(GetOptState.uIndex, DeviceType_Network));
     892                    CHECK_ERROR(sessionMachine, SetBootOrder(GetOptState.uIndex, DeviceType_Network));
    891893                }
    892894                else
    … …  
    927929                if (!RTStrICmp(ValueUnion.psz, "none"))
    928930                {
    929                     machine->DetachDevice(bstrController.raw(), u1, u2);
     931                    sessionMachine->DetachDevice(bstrController.raw(), u1, u2);
    930932                }
    931933                else
    … …  
    940942                    if (hardDisk)
    941943                    {
    942                         CHECK_ERROR(machine, AttachDevice(bstrController.raw(),
     944                        CHECK_ERROR(sessionMachine, AttachDevice(bstrController.raw(),
    943945                                                          u1, u2,
    944946                                                          DeviceType_HardDisk,
    … …  
    954956            {
    955957                ComPtr<IStorageController> storageController;
    956                 CHECK_ERROR(machine, GetStorageControllerByName(Bstr("IDE Controller").raw(),
     958                CHECK_ERROR(sessionMachine, GetStorageControllerByName(Bstr("IDE Controller").raw(),
    957959                                                                 storageController.asOutParam()));
    958960
    … …  
    980982            {
    981983                ComPtr<IStorageController> SataCtl;
    982                 CHECK_ERROR(machine, GetStorageControllerByName(Bstr("SATA").raw(),
     984                CHECK_ERROR(sessionMachine, GetStorageControllerByName(Bstr("SATA").raw(),
    983985                                                                SataCtl.asOutParam()));
    984986
    … …  
    993995                {
    994996                    ComPtr<IStorageController> ctl;
    995                     CHECK_ERROR(machine, AddStorageController(Bstr("SATA").raw(),
     997                    CHECK_ERROR(sessionMachine, AddStorageController(Bstr("SATA").raw(),
    996998                                                              StorageBus_SATA,
    997999                                                              ctl.asOutParam()));
    … …  
    9991001                }
    10001002                else if (!RTStrICmp(ValueUnion.psz, "off") || !RTStrICmp(ValueUnion.psz, "disable"))
    1001                     CHECK_ERROR(machine, RemoveStorageController(Bstr("SATA").raw()));
     1003                    CHECK_ERROR(sessionMachine, RemoveStorageController(Bstr("SATA").raw()));
    10021004                else
    10031005                    return errorArgument("Invalid --usb argument '%s'", ValueUnion.psz);
    … …  
    10091011                if (!RTStrICmp(ValueUnion.psz, "none"))
    10101012                {
    1011                     rc = machine->DetachDevice(Bstr("LsiLogic").raw(),
     1013                    rc = sessionMachine->DetachDevice(Bstr("LsiLogic").raw(),
    10121014                                               GetOptState.uIndex, 0);
    10131015                    if (FAILED(rc))
    1014                         CHECK_ERROR(machine, DetachDevice(Bstr("BusLogic").raw(),
     1016                        CHECK_ERROR(sessionMachine, DetachDevice(Bstr("BusLogic").raw(),
    10151017                                                          GetOptState.uIndex, 0));
    10161018                }
    … …  
    10261028                    if (hardDisk)
    10271029                    {
    1028                         rc = machine->AttachDevice(Bstr("LsiLogic").raw(),
     1030                        rc = sessionMachine->AttachDevice(Bstr("LsiLogic").raw(),
    10291031                                                   GetOptState.uIndex, 0,
    10301032                                                   DeviceType_HardDisk,
    10311033                                                   hardDisk);
    10321034                        if (FAILED(rc))
    1033                             CHECK_ERROR(machine,
     1035                            CHECK_ERROR(sessionMachine,
    10341036                                        AttachDevice(Bstr("BusLogic").raw(),
    10351037                                                     GetOptState.uIndex, 0,
    … …  
    10491051                if (!RTStrICmp(ValueUnion.psz, "LsiLogic"))
    10501052                {
    1051                     rc = machine->RemoveStorageController(Bstr("BusLogic").raw());
     1053                    rc = sessionMachine->RemoveStorageController(Bstr("BusLogic").raw());
    10521054                    if (FAILED(rc))
    1053                         CHECK_ERROR(machine, RemoveStorageController(Bstr("LsiLogic").raw()));
    1054 
    1055                     CHECK_ERROR(machine,
     1055                        CHECK_ERROR(sessionMachine, RemoveStorageController(Bstr("LsiLogic").raw()));
     1056
     1057                    CHECK_ERROR(sessionMachine,
    10561058                                 AddStorageController(Bstr("LsiLogic").raw(),
    10571059                                                      StorageBus_SCSI,
    … …  
    10631065                else if (!RTStrICmp(ValueUnion.psz, "BusLogic"))
    10641066                {
    1065                     rc = machine->RemoveStorageController(Bstr("LsiLogic").raw());
     1067                    rc = sessionMachine->RemoveStorageController(Bstr("LsiLogic").raw());
    10661068                    if (FAILED(rc))
    1067                         CHECK_ERROR(machine, RemoveStorageController(Bstr("BusLogic").raw()));
    1068 
    1069                     CHECK_ERROR(machine,
     1069                        CHECK_ERROR(sessionMachine, RemoveStorageController(Bstr("BusLogic").raw()));
     1070
     1071                    CHECK_ERROR(sessionMachine,
    10701072                                 AddStorageController(Bstr("BusLogic").raw(),
    10711073                                                      StorageBus_SCSI,
    … …  
    10861088                    ComPtr<IStorageController> ctl;
    10871089
    1088                     CHECK_ERROR(machine, AddStorageController(Bstr("BusLogic").raw(),
     1090                    CHECK_ERROR(sessionMachine, AddStorageController(Bstr("BusLogic").raw(),
    10891091                                                              StorageBus_SCSI,
    10901092                                                              ctl.asOutParam()));
    … …  
    10941096                else if (!RTStrICmp(ValueUnion.psz, "off") || !RTStrICmp(ValueUnion.psz, "disable"))
    10951097                {
    1096                     rc = machine->RemoveStorageController(Bstr("BusLogic").raw());
     1098                    rc = sessionMachine->RemoveStorageController(Bstr("BusLogic").raw());
    10971099                    if (FAILED(rc))
    1098                         CHECK_ERROR(machine, RemoveStorageController(Bstr("LsiLogic").raw()));
     1100                        CHECK_ERROR(sessionMachine, RemoveStorageController(Bstr("LsiLogic").raw()));
    10991101                }
    11001102                break;
    … …  
    11031105            case MODIFYVM_DVDPASSTHROUGH: // deprecated
    11041106            {
    1105                 CHECK_ERROR(machine, PassthroughDevice(Bstr("IDE Controller").raw(),
     1107                CHECK_ERROR(sessionMachine, PassthroughDevice(Bstr("IDE Controller").raw(),
    11061108                                                       1, 0,
    11071109                                                       !RTStrICmp(ValueUnion.psz, "on")));
    … …  
    11601162                }
    11611163
    1162                 CHECK_ERROR(machine, MountMedium(Bstr("IDE Controller").raw(),
     1164                CHECK_ERROR(sessionMachine, MountMedium(Bstr("IDE Controller").raw(),
    11631165                                                 1, 0,
    11641166                                                 dvdMedium,
    … …  
    11711173                ComPtr<IMedium> floppyMedium;
    11721174                ComPtr<IMediumAttachment> floppyAttachment;
    1173                 machine->GetMediumAttachment(Bstr("Floppy Controller").raw(),
     1175                sessionMachine->GetMediumAttachment(Bstr("Floppy Controller").raw(),
    11741176                                             0, 0, floppyAttachment.asOutParam());
    11751177
    … …  
    11791181                    /* disable the controller */
    11801182                    if (floppyAttachment)
    1181                         CHECK_ERROR(machine, DetachDevice(Bstr("Floppy Controller").raw(),
     1183                        CHECK_ERROR(sessionMachine, DetachDevice(Bstr("Floppy Controller").raw(),
    11821184                                                          0, 0));
    11831185                }
    … …  
    11861188                    /* enable the controller */
    11871189                    if (!floppyAttachment)
    1188                         CHECK_ERROR(machine, AttachDeviceWithoutMedium(Bstr("Floppy Controller").raw(),
     1190                        CHECK_ERROR(sessionMachine, AttachDeviceWithoutMedium(Bstr("Floppy Controller").raw(),
    11891191                                                                            0, 0,
    11901192                                                                            DeviceType_Floppy));
    … …  
    12241226                        }
    12251227                    }
    1226                     CHECK_ERROR(machine, MountMedium(Bstr("Floppy Controller").raw(),
     1228                    CHECK_ERROR(sessionMachine, MountMedium(Bstr("Floppy Controller").raw(),
    12271229                                                     0, 0,
    12281230                                                     floppyMedium,
    … …  
    12391241
    12401242                ComPtr<INetworkAdapter> nic;
    1241                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1243                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    12421244                ASSERT(nic);
    12431245
    … …  
    12521254
    12531255                ComPtr<INetworkAdapter> nic;
    1254                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1256                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    12551257                ASSERT(nic);
    12561258
    … …  
    12651267
    12661268                ComPtr<INetworkAdapter> nic;
    1267                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1269                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    12681270                ASSERT(nic);
    12691271
    … …  
    13041306
    13051307                ComPtr<INetworkAdapter> nic;
    1306                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1308                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    13071309                ASSERT(nic);
    13081310
    … …  
    13491351
    13501352                ComPtr<INetworkAdapter> nic;
    1351                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1353                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    13521354                ASSERT(nic);
    13531355
    … …  
    13621364
    13631365                ComPtr<INetworkAdapter> nic;
    1364                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1366                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    13651367                ASSERT(nic);
    13661368
    … …  
    14021404
    14031405                ComPtr<INetworkAdapter> nic;
    1404                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1406                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    14051407                ASSERT(nic);
    14061408
    … …  
    14151417
    14161418                ComPtr<INetworkAdapter> nic;
    1417                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1419                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    14181420                ASSERT(nic);
    14191421
    … …  
    14281430                    ComPtr<IBandwidthGroup> bwGroup;
    14291431
    1430                     CHECK_ERROR(machine, COMGETTER(BandwidthControl)(bwCtrl.asOutParam()));
     1432                    CHECK_ERROR(sessionMachine, COMGETTER(BandwidthControl)(bwCtrl.asOutParam()));
    14311433
    14321434                    if (SUCCEEDED(rc))
    … …  
    14481450
    14491451                ComPtr<INetworkAdapter> nic;
    1450                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1452                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    14511453                ASSERT(nic);
    14521454
    … …  
    15081510
    15091511                ComPtr<INetworkAdapter> nic;
    1510                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1512                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    15111513                ASSERT(nic);
    15121514
    … …  
    15211523
    15221524                ComPtr<INetworkAdapter> nic;
    1523                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1525                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    15241526                ASSERT(nic);
    15251527
    … …  
    15421544
    15431545                ComPtr<INetworkAdapter> nic;
    1544                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1546                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    15451547                ASSERT(nic);
    15461548
    … …  
    15631565
    15641566                ComPtr<INetworkAdapter> nic;
    1565                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1567                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    15661568                ASSERT(nic);
    15671569
    … …  
    15841586
    15851587                ComPtr<INetworkAdapter> nic;
    1586                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1588                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    15871589                ASSERT(nic);
    15881590
    … …  
    15971599
    15981600                ComPtr<INetworkAdapter> nic;
    1599                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1601                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    16001602                ASSERT(nic);
    16011603
    … …  
    16101612
    16111613                ComPtr<INetworkAdapter> nic;
    1612                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1614                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    16131615                ASSERT(nic);
    16141616
    … …  
    16301632
    16311633                ComPtr<INetworkAdapter> nic;
    1632                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1634                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    16331635                ASSERT(nic);
    16341636
    … …  
    16801682                    break;
    16811683
    1682                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1684                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    16831685                ASSERT(nic);
    16841686
    … …  
    16961698
    16971699                ComPtr<INetworkAdapter> nic;
    1698                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1700                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    16991701                ASSERT(nic);
    17001702
    … …  
    17591761                uint32_t aliasMode = 0;
    17601762
    1761                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1763                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    17621764                ASSERT(nic);
    17631765
    … …  
    17921794
    17931795                ComPtr<INetworkAdapter> nic;
    1794                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1796                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    17951797                ASSERT(nic);
    17961798
    … …  
    18081810
    18091811                ComPtr<INetworkAdapter> nic;
    1810                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1812                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    18111813                ASSERT(nic);
    18121814
    … …  
    18241826
    18251827                ComPtr<INetworkAdapter> nic;
    1826                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1828                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    18271829                ASSERT(nic);
    18281830
    … …  
    18391841
    18401842                ComPtr<INetworkAdapter> nic;
    1841                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1843                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    18421844                ASSERT(nic);
    18431845
    … …  
    18551857
    18561858                ComPtr<INetworkAdapter> nic;
    1857                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1859                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    18581860                ASSERT(nic);
    18591861
    … …  
    18711873
    18721874                ComPtr<INetworkAdapter> nic;
    1873                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1875                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    18741876                ASSERT(nic);
    18751877
    … …  
    18861888
    18871889                ComPtr<INetworkAdapter> nic;
    1888                 CHECK_ERROR_BREAK(machine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
     1890                CHECK_ERROR_BREAK(sessionMachine, GetNetworkAdapter(GetOptState.uIndex - 1, nic.asOutParam()));
    18891891                ASSERT(nic);
    18901892
    … …  
    19061908                if (!RTStrICmp(ValueUnion.psz, "ps2"))
    19071909                {
    1908                     CHECK_ERROR(machine, COMSETTER(PointingHIDType)(PointingHIDType_PS2Mouse));
     1910                    CHECK_ERROR(sessionMachine, COMSETTER(PointingHIDType)(PointingHIDType_PS2Mouse));
    19091911                }
    19101912                else if (!RTStrICmp(ValueUnion.psz, "usb"))
    19111913                {
    1912                     CHECK_ERROR(machine, COMSETTER(PointingHIDType)(PointingHIDType_USBMouse));
     1914                    CHECK_ERROR(sessionMachine, COMSETTER(PointingHIDType)(PointingHIDType_USBMouse));
    19131915                    if (SUCCEEDED(rc))
    19141916                        fEnableUsb = true;
    … …  
    19161918                else if (!RTStrICmp(ValueUnion.psz, "usbtablet"))
    19171919                {
    1918                     CHECK_ERROR(machine, COMSETTER(PointingHIDType)(PointingHIDType_USBTablet));
     1920                    CHECK_ERROR(sessionMachine, COMSETTER(PointingHIDType)(PointingHIDType_USBTablet));
    19191921                    if (SUCCEEDED(rc))
    19201922                        fEnableUsb = true;
    … …  
    19221924                else if (!RTStrICmp(ValueUnion.psz, "usbmultitouch"))
    19231925                {
    1924                     CHECK_ERROR(machine, COMSETTER(PointingHIDType)(PointingHIDType_USBMultiTouch));
     1926                    CHECK_ERROR(sessionMachine, COMSETTER(PointingHIDType)(PointingHIDType_USBMultiTouch));
    19251927                    if (SUCCEEDED(rc))
    19261928                        fEnableUsb = true;
    … …  
    19351937                    /* Make sure the OHCI controller is enabled. */
    19361938                    ULONG cOhciCtrls = 0;
    1937                     rc = machine->GetUSBControllerCountByType(USBControllerType_OHCI, &cOhciCtrls);
     1939                    rc = sessionMachine->GetUSBControllerCountByType(USBControllerType_OHCI, &cOhciCtrls);
    19381940                    if (   SUCCEEDED(rc)
    19391941                        && !cOhciCtrls)
    19401942                    {
    19411943                        ComPtr<IUSBController> UsbCtl;
    1942                         CHECK_ERROR(machine, AddUSBController(Bstr("OHCI").raw(), USBControllerType_OHCI,
     1944                        CHECK_ERROR(sessionMachine, AddUSBController(Bstr("OHCI").raw(), USBControllerType_OHCI,
    19431945                                                              UsbCtl.asOutParam()));
    19441946                    }
    … …  
    19521954                if (!RTStrICmp(ValueUnion.psz, "ps2"))
    19531955                {
    1954                     CHECK_ERROR(machine, COMSETTER(KeyboardHIDType)(KeyboardHIDType_PS2Keyboard));
     1956                    CHECK_ERROR(sessionMachine, COMSETTER(KeyboardHIDType)(KeyboardHIDType_PS2Keyboard));
    19551957                }
    19561958                else if (!RTStrICmp(ValueUnion.psz, "usb"))
    19571959                {
    1958                     CHECK_ERROR(machine, COMSETTER(KeyboardHIDType)(KeyboardHIDType_USBKeyboard));
     1960                    CHECK_ERROR(sessionMachine, COMSETTER(KeyboardHIDType)(KeyboardHIDType_USBKeyboard));
    19591961                    if (SUCCEEDED(rc))
    19601962                        fEnableUsb = true;
    … …  
    19691971                    /* Make sure the OHCI controller is enabled. */
    19701972                    ULONG cOhciCtrls = 0;
    1971                     rc = machine->GetUSBControllerCountByType(USBControllerType_OHCI, &cOhciCtrls);
     1973                    rc = sessionMachine->GetUSBControllerCountByType(USBControllerType_OHCI, &cOhciCtrls);
    19721974                    if (   SUCCEEDED(rc)
    19731975                        && !cOhciCtrls)
    19741976                    {
    19751977                        ComPtr<IUSBController> UsbCtl;
    1976                         CHECK_ERROR(machine, AddUSBController(Bstr("OHCI").raw(), USBControllerType_OHCI,
     1978                        CHECK_ERROR(sessionMachine, AddUSBController(Bstr("OHCI").raw(), USBControllerType_OHCI,
    19771979                                                              UsbCtl.asOutParam()));
    19781980                    }
    … …  
    19861988                char *pszIRQ = NULL;
    19871989
    1988                 CHECK_ERROR_BREAK(machine, GetSerialPort(GetOptState.uIndex - 1, uart.asOutParam()));
     1990                CHECK_ERROR_BREAK(sessionMachine, GetSerialPort(GetOptState.uIndex - 1, uart.asOutParam()));
    19891991                ASSERT(uart);
    19901992
    … …  
    20342036                ComPtr<ISerialPort> uart;
    20352037
    2036                 CHECK_ERROR_BREAK(machine, GetSerialPort(GetOptState.uIndex - 1, uart.asOutParam()));
     2038                CHECK_ERROR_BREAK(sessionMachine, GetSerialPort(GetOptState.uIndex - 1, uart.asOutParam()));
    20372039                ASSERT(uart);
    20382040
    … …  
    20682070                char *pszIRQ = NULL;
    20692071
    2070                 CHECK_ERROR_BREAK(machine, GetParallelPort(GetOptState.uIndex - 1, lpt.asOutParam()));
     2072                CHECK_ERROR_BREAK(sessionMachine, GetParallelPort(GetOptState.uIndex - 1, lpt.asOutParam()));
    20712073                ASSERT(lpt);
    20722074
    … …  
    20792081                ComPtr<IParallelPort> lpt;
    20802082
    2081                 CHECK_ERROR_BREAK(machine, GetParallelPort(GetOptState.uIndex - 1, lpt.asOutParam()));
     2083                CHECK_ERROR_BREAK(sessionMachine, GetParallelPort(GetOptState.uIndex - 1, lpt.asOutParam()));
    20822084                ASSERT(lpt);
    20832085
    … …  
    21102112            case MODIFYVM_GUESTMEMORYBALLOON:
    21112113            {
    2112                 CHECK_ERROR(machine, COMSETTER(MemoryBalloonSize)(ValueUnion.u32));
     2114                CHECK_ERROR(sessionMachine, COMSETTER(MemoryBalloonSize)(ValueUnion.u32));
    21132115                break;
    21142116            }
    … …  
    21172119            {
    21182120                ComPtr<IAudioAdapter> audioAdapter;
    2119                 machine->COMGETTER(AudioAdapter)(audioAdapter.asOutParam());
     2121                sessionMachine->COMGETTER(AudioAdapter)(audioAdapter.asOutParam());
    21202122                ASSERT(audioAdapter);
    21212123
    … …  
    21372139            {
    21382140                ComPtr<IAudioAdapter> audioAdapter;
    2139                 machine->COMGETTER(AudioAdapter)(audioAdapter.asOutParam());
     2141                sessionMachine->COMGETTER(AudioAdapter)(audioAdapter.asOutParam());
    21402142                ASSERT(audioAdapter);
    21412143
    … …  
    22422244                if (SUCCEEDED(rc))
    22432245                {
    2244                     CHECK_ERROR(machine, COMSETTER(ClipboardMode)(mode));
     2246                    CHECK_ERROR(sessionMachine, COMSETTER(ClipboardMode)(mode));
    22452247                }
    22462248                break;
    … …  
    22652267                if (SUCCEEDED(rc))
    22662268                {
    2267                     CHECK_ERROR(machine, COMSETTER(DnDMode)(mode));
     2269                    CHECK_ERROR(sessionMachine, COMSETTER(DnDMode)(mode));
    22682270                }
    22692271                break;
    … …  
    22732275            {
    22742276                ComPtr<IVRDEServer> vrdeServer;
    2275                 machine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
     2277                sessionMachine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
    22762278                ASSERT(vrdeServer);
    22772279
    … …  
    22922294            {
    22932295                ComPtr<IVRDEServer> vrdeServer;
    2294                 machine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
     2296                sessionMachine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
    22952297                ASSERT(vrdeServer);
    22962298
    … …  
    23352337            {
    23362338                ComPtr<IVRDEServer> vrdeServer;
    2337                 machine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
     2339                sessionMachine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
    23382340                ASSERT(vrdeServer);
    23392341
    … …  
    23512353            {
    23522354                ComPtr<IVRDEServer> vrdeServer;
    2353                 machine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
     2355                sessionMachine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
    23542356                ASSERT(vrdeServer);
    23552357
    … …  
    23632365            {
    23642366                ComPtr<IVRDEServer> vrdeServer;
    2365                 machine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
     2367                sessionMachine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
    23662368                ASSERT(vrdeServer);
    23672369
    … …  
    23892391            {
    23902392                ComPtr<IVRDEServer> vrdeServer;
    2391                 machine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
     2393                sessionMachine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
    23922394                ASSERT(vrdeServer);
    23932395
    … …  
    24102412            {
    24112413                ComPtr<IVRDEServer> vrdeServer;
    2412                 machine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
     2414                sessionMachine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
    24132415                ASSERT(vrdeServer);
    24142416
    … …  
    24222424            {
    24232425                ComPtr<IVRDEServer> vrdeServer;
    2424                 machine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
     2426                sessionMachine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
    24252427                ASSERT(vrdeServer);
    24262428
    … …  
    24342436            {
    24352437                ComPtr<IVRDEServer> vrdeServer;
    2436                 machine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
     2438                sessionMachine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
    24372439                ASSERT(vrdeServer);
    24382440
    … …  
    24472449            {
    24482450                ComPtr<IVRDEServer> vrdeServer;
    2449                 machine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
     2451                sessionMachine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
    24502452                ASSERT(vrdeServer);
    24512453
    … …  
    24602462            {
    24612463                ComPtr<IVRDEServer> vrdeServer;
    2462                 machine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
     2464                sessionMachine->COMGETTER(VRDEServer)(vrdeServer.asOutParam());
    24632465                ASSERT(vrdeServer);
    24642466
    … …  
    24702472            {
    24712473                ULONG cXhciCtrls = 0;
    2472                 rc = machine->GetUSBControllerCountByType(USBControllerType_XHCI, &cXhciCtrls);
     2474                rc = sessionMachine->GetUSBControllerCountByType(USBControllerType_XHCI, &cXhciCtrls);
    24732475                if (SUCCEEDED(rc))
    24742476                {
    … …  
    24762478                    {
    24772479                        ComPtr<IUSBController> UsbCtl;
    2478                         CHECK_ERROR(machine, AddUSBController(Bstr("XHCI").raw(), USBControllerType_XHCI,
     2480                        CHECK_ERROR(sessionMachine, AddUSBController(Bstr("XHCI").raw(), USBControllerType_XHCI,
    24792481                                                              UsbCtl.asOutParam()));
    24802482                    }
    24812483                    else if (cXhciCtrls && !ValueUnion.f)
    2482                         CHECK_ERROR(machine, RemoveUSBController(Bstr("XHCI").raw()));
     2484                        CHECK_ERROR(sessionMachine, RemoveUSBController(Bstr("XHCI").raw()));
    24832485                }
    24842486                break;
    … …  
    24882490            {
    24892491                ULONG cEhciCtrls = 0;
    2490                 rc = machine->GetUSBControllerCountByType(USBControllerType_EHCI, &cEhciCtrls);
     2492                rc = sessionMachine->GetUSBControllerCountByType(USBControllerType_EHCI, &cEhciCtrls);
    24912493                if (SUCCEEDED(rc))
    24922494                {
    … …  
    24942496                    {
    24952497                        ComPtr<IUSBController> UsbCtl;
    2496                         CHECK_ERROR(machine, AddUSBController(Bstr("EHCI").raw(), USBControllerType_EHCI,
     2498                        CHECK_ERROR(sessionMachine, AddUSBController(Bstr("EHCI").raw(), USBControllerType_EHCI,
    24972499                                                              UsbCtl.asOutParam()));
    24982500                    }
    24992501                    else if (cEhciCtrls && !ValueUnion.f)
    2500                         CHECK_ERROR(machine, RemoveUSBController(Bstr("EHCI").raw()));
     2502                        CHECK_ERROR(sessionMachine, RemoveUSBController(Bstr("EHCI").raw()));
    25012503                }
    25022504                break;
    … …  
    25062508            {
    25072509                ULONG cOhciCtrls = 0;
    2508                 rc = machine->GetUSBControllerCountByType(USBControllerType_OHCI, &cOhciCtrls);
     2510                rc = sessionMachine->GetUSBControllerCountByType(USBControllerType_OHCI, &cOhciCtrls);
    25092511                if (SUCCEEDED(rc))
    25102512                {
    … …  
    25122514                    {
    25132515                        ComPtr<IUSBController> UsbCtl;
    2514                         CHECK_ERROR(machine, AddUSBController(Bstr("OHCI").raw(), USBControllerType_OHCI,
     2516                        CHECK_ERROR(sessionMachine, AddUSBController(Bstr("OHCI").raw(), USBControllerType_OHCI,
    25152517                                                              UsbCtl.asOutParam()));
    25162518                    }
    25172519                    else if (cOhciCtrls && !ValueUnion.f)
    2518                         CHECK_ERROR(machine, RemoveUSBController(Bstr("OHCI").raw()));
     2520                        CHECK_ERROR(sessionMachine, RemoveUSBController(Bstr("OHCI").raw()));
    25192521                }
    25202522                break;
    … …  
    25242526            {
    25252527                if (!RTStrICmp(ValueUnion.psz, "default"))
    2526                     CHECK_ERROR(machine, COMSETTER(SnapshotFolder)(Bstr().raw()));
    2527                 else
    2528                     CHECK_ERROR(machine, COMSETTER(SnapshotFolder)(Bstr(ValueUnion.psz).raw()));
     2528                    CHECK_ERROR(sessionMachine, COMSETTER(SnapshotFolder)(Bstr().raw()));
     2529                else
     2530                    CHECK_ERROR(sessionMachine, COMSETTER(SnapshotFolder)(Bstr(ValueUnion.psz).raw()));
    25292531                break;
    25302532            }
    … …  
    25322534            case MODIFYVM_TELEPORTER_ENABLED:
    25332535            {
    2534                 CHECK_ERROR(machine, COMSETTER(TeleporterEnabled)(ValueUnion.f));
     2536                CHECK_ERROR(sessionMachine, COMSETTER(TeleporterEnabled)(ValueUnion.f));
    25352537                break;
    25362538            }
    … …  
    25382540            case MODIFYVM_TELEPORTER_PORT:
    25392541            {
    2540                 CHECK_ERROR(machine, COMSETTER(TeleporterPort)(ValueUnion.u32));
     2542                CHECK_ERROR(sessionMachine, COMSETTER(TeleporterPort)(ValueUnion.u32));
    25412543                break;
    25422544            }
    … …  
    25442546            case MODIFYVM_TELEPORTER_ADDRESS:
    25452547            {
    2546                 CHECK_ERROR(machine, COMSETTER(TeleporterAddress)(Bstr(ValueUnion.psz).raw()));
     2548                CHECK_ERROR(sessionMachine, COMSETTER(TeleporterAddress)(Bstr(ValueUnion.psz).raw()));
    25472549                break;
    25482550            }
    … …  
    25502552            case MODIFYVM_TELEPORTER_PASSWORD:
    25512553            {
    2552                 CHECK_ERROR(machine, COMSETTER(TeleporterPassword)(Bstr(ValueUnion.psz).raw()));
     2554                CHECK_ERROR(sessionMachine, COMSETTER(TeleporterPassword)(Bstr(ValueUnion.psz).raw()));
    25532555                break;
    25542556            }
    … …  
    25612563                    rc = E_FAIL;
    25622564                else
    2563                     CHECK_ERROR(machine, COMSETTER(TeleporterPassword)(Bstr(password).raw()));
     2565                    CHECK_ERROR(sessionMachine, COMSETTER(TeleporterPassword)(Bstr(password).raw()));
    25642566                break;
    25652567            }
    … …  
    25672569            case MODIFYVM_TRACING_ENABLED:
    25682570            {
    2569                 CHECK_ERROR(machine, COMSETTER(TracingEnabled)(ValueUnion.f));
     2571                CHECK_ERROR(sessionMachine, COMSETTER(TracingEnabled)(ValueUnion.f));
    25702572                break;
    25712573            }
    … …  
    25732575            case MODIFYVM_TRACING_CONFIG:
    25742576            {
    2575                 CHECK_ERROR(machine, COMSETTER(TracingConfig)(Bstr(ValueUnion.psz).raw()));
     2577                CHECK_ERROR(sessionMachine, COMSETTER(TracingConfig)(Bstr(ValueUnion.psz).raw()));
    25762578                break;
    25772579            }
    … …  
    25792581            case MODIFYVM_TRACING_ALLOW_VM_ACCESS:
    25802582            {
    2581                 CHECK_ERROR(machine, COMSETTER(AllowTracingToAccessVM)(ValueUnion.f));
     2583                CHECK_ERROR(sessionMachine, COMSETTER(AllowTracingToAccessVM)(ValueUnion.f));
    25822584                break;
    25832585            }
    … …  
    25872589                if (!RTStrICmp(ValueUnion.psz, "master"))
    25882590                {
    2589                     CHECK_ERROR(machine, COMSETTER(FaultToleranceState(FaultToleranceState_Master)));
     2591                    CHECK_ERROR(sessionMachine, COMSETTER(FaultToleranceState(FaultToleranceState_Master)));
    25902592                }
    25912593                else
    25922594                if (!RTStrICmp(ValueUnion.psz, "standby"))
    25932595                {
    2594                     CHECK_ERROR(machine, COMSETTER(FaultToleranceState(FaultToleranceState_Standby)));
     2596                    CHECK_ERROR(sessionMachine, COMSETTER(FaultToleranceState(FaultToleranceState_Standby)));
    25952597                }
    25962598                else
    … …  
    26042606            case MODIFYVM_FAULT_TOLERANCE_ADDRESS:
    26052607            {
    2606                 CHECK_ERROR(machine, COMSETTER(FaultToleranceAddress)(Bstr(ValueUnion.psz).raw()));
     2608                CHECK_ERROR(sessionMachine, COMSETTER(FaultToleranceAddress)(Bstr(ValueUnion.psz).raw()));
    26072609                break;
    26082610            }
    … …  
    26102612            case MODIFYVM_FAULT_TOLERANCE_PORT:
    26112613            {
    2612                 CHECK_ERROR(machine, COMSETTER(FaultTolerancePort)(ValueUnion.u32));
     2614                CHECK_ERROR(sessionMachine, COMSETTER(FaultTolerancePort)(ValueUnion.u32));
    26132615                break;
    26142616            }
    … …  
    26162618            case MODIFYVM_FAULT_TOLERANCE_PASSWORD:
    26172619            {
    2618                 CHECK_ERROR(machine, COMSETTER(FaultTolerancePassword)(Bstr(ValueUnion.psz).raw()));
     2620                CHECK_ERROR(sessionMachine, COMSETTER(FaultTolerancePassword)(Bstr(ValueUnion.psz).raw()));
    26192621                break;
    26202622            }
    … …  
    26222624            case MODIFYVM_FAULT_TOLERANCE_SYNC_INTERVAL:
    26232625            {
    2624                 CHECK_ERROR(machine, COMSETTER(FaultToleranceSyncInterval)(ValueUnion.u32));
     2626                CHECK_ERROR(sessionMachine, COMSETTER(FaultToleranceSyncInterval)(ValueUnion.u32));
    26252627                break;
    26262628            }
    … …  
    26282630            case MODIFYVM_HARDWARE_UUID:
    26292631            {
    2630                 CHECK_ERROR(machine, COMSETTER(HardwareUUID)(Bstr(ValueUnion.psz).raw()));
     2632                CHECK_ERROR(sessionMachine, COMSETTER(HardwareUUID)(Bstr(ValueUnion.psz).raw()));
    26312633                break;
    26322634            }
    … …  
    26342636            case MODIFYVM_HPET:
    26352637            {
    2636                 CHECK_ERROR(machine, COMSETTER(HPETEnabled)(ValueUnion.f));
     2638                CHECK_ERROR(sessionMachine, COMSETTER(HPETEnabled)(ValueUnion.f));
    26372639                break;
    26382640            }
    … …  
    26402642            case MODIFYVM_IOCACHE:
    26412643            {
    2642                 CHECK_ERROR(machine, COMSETTER(IOCacheEnabled)(ValueUnion.f));
     2644                CHECK_ERROR(sessionMachine, COMSETTER(IOCacheEnabled)(ValueUnion.f));
    26432645                break;
    26442646            }
    … …  
    26462648            case MODIFYVM_IOCACHESIZE:
    26472649            {
    2648                 CHECK_ERROR(machine, COMSETTER(IOCacheSize)(ValueUnion.u32));
     2650                CHECK_ERROR(sessionMachine, COMSETTER(IOCacheSize)(ValueUnion.u32));
    26492651                break;
    26502652            }
    … …  
    26542656                if (!RTStrICmp(ValueUnion.psz, "piix3"))
    26552657                {
    2656                     CHECK_ERROR(machine, COMSETTER(ChipsetType)(ChipsetType_PIIX3));
     2658                    CHECK_ERROR(sessionMachine, COMSETTER(ChipsetType)(ChipsetType_PIIX3));
    26572659                }
    26582660                else if (!RTStrICmp(ValueUnion.psz, "ich9"))
    26592661                {
    2660                     CHECK_ERROR(machine, COMSETTER(ChipsetType)(ChipsetType_ICH9));
     2662                    CHECK_ERROR(sessionMachine, COMSETTER(ChipsetType)(ChipsetType_ICH9));
    26612663                    BOOL fIoApic = FALSE;
    26622664                    CHECK_ERROR(biosSettings, COMGETTER(IOAPICEnabled)(&fIoApic));
    … …  
    26772679            case MODIFYVM_VCP:
    26782680            {
    2679                 CHECK_ERROR(machine, COMSETTER(VideoCaptureEnabled)(ValueUnion.f));
     2681                CHECK_ERROR(sessionMachine, COMSETTER(VideoCaptureEnabled)(ValueUnion.f));
    26802682                break;
    26812683            }
    … …  
    26832685            {
    26842686                ULONG cMonitors = 64;
    2685                 CHECK_ERROR(machine, COMGETTER(MonitorCount)(&cMonitors));
     2687                CHECK_ERROR(sessionMachine, COMGETTER(MonitorCount)(&cMonitors));
    26862688                com::SafeArray<BOOL> screens(cMonitors);
    26872689                if (parseScreens(ValueUnion.psz, &screens))
    … …  
    26912693                    break;
    26922694                }
    2693                 CHECK_ERROR(machine, COMSETTER(VideoCaptureScreens)(ComSafeArrayAsInParam(screens)));
     2695                CHECK_ERROR(sessionMachine, COMSETTER(VideoCaptureScreens)(ComSafeArrayAsInParam(screens)));
    26942696                break;
    26952697            }
    … …  
    27102712                    bstr = szVCFileAbs;
    27112713                }
    2712                 CHECK_ERROR(machine, COMSETTER(VideoCaptureFile)(bstr.raw()));
     2714                CHECK_ERROR(sessionMachine, COMSETTER(VideoCaptureFile)(bstr.raw()));
    27132715                break;
    27142716            }
    27152717            case MODIFYVM_VCP_WIDTH:
    27162718            {
    2717                 CHECK_ERROR(machine, COMSETTER(VideoCaptureWidth)(ValueUnion.u32));
     2719                CHECK_ERROR(sessionMachine, COMSETTER(VideoCaptureWidth)(ValueUnion.u32));
    27182720                break;
    27192721            }
    27202722            case MODIFYVM_VCP_HEIGHT:
    27212723            {
    2722                 CHECK_ERROR(machine, COMSETTER(VideoCaptureHeight)(ValueUnion.u32));
     2724                CHECK_ERROR(sessionMachine, COMSETTER(VideoCaptureHeight)(ValueUnion.u32));
    27232725                break;
    27242726            }
    27252727            case MODIFYVM_VCP_RATE:
    27262728            {
    2727                 CHECK_ERROR(machine, COMSETTER(VideoCaptureRate)(ValueUnion.u32));
     2729                CHECK_ERROR(sessionMachine, COMSETTER(VideoCaptureRate)(ValueUnion.u32));
    27282730                break;
    27292731            }
    27302732            case MODIFYVM_VCP_FPS:
    27312733            {
    2732                 CHECK_ERROR(machine, COMSETTER(VideoCaptureFPS)(ValueUnion.u32));
     2734                CHECK_ERROR(sessionMachine, COMSETTER(VideoCaptureFPS)(ValueUnion.u32));
    27332735                break;
    27342736            }
    27352737            case MODIFYVM_VCP_MAXTIME:
    27362738            {
    2737                 CHECK_ERROR(machine, COMSETTER(VideoCaptureMaxTime)(ValueUnion.u32));
     2739                CHECK_ERROR(sessionMachine, COMSETTER(VideoCaptureMaxTime)(ValueUnion.u32));
    27382740                break;
    27392741            }
    27402742            case MODIFYVM_VCP_MAXSIZE:
    27412743            {
    2742                 CHECK_ERROR(machine, COMSETTER(VideoCaptureMaxFileSize)(ValueUnion.u32));
     2744                CHECK_ERROR(sessionMachine, COMSETTER(VideoCaptureMaxFileSize)(ValueUnion.u32));
    27432745                break;
    27442746            }
    … …  
    27462748            {
    27472749                Bstr bstr(ValueUnion.psz);
    2748                 CHECK_ERROR(machine, COMSETTER(VideoCaptureOptions)(bstr.raw()));
     2750                CHECK_ERROR(sessionMachine, COMSETTER(VideoCaptureOptions)(bstr.raw()));
    27492751                break;
    27502752            }
    … …  
    27522754            case MODIFYVM_AUTOSTART_ENABLED:
    27532755            {
    2754                 CHECK_ERROR(machine, COMSETTER(AutostartEnabled)(ValueUnion.f));
     2756                CHECK_ERROR(sessionMachine, COMSETTER(AutostartEnabled)(ValueUnion.f));
    27552757                break;
    27562758            }
    … …  
    27582760            case MODIFYVM_AUTOSTART_DELAY:
    27592761            {
    2760                 CHECK_ERROR(machine, COMSETTER(AutostartDelay)(ValueUnion.u32));
     2762                CHECK_ERROR(sessionMachine, COMSETTER(AutostartDelay)(ValueUnion.u32));
    27612763                break;
    27622764            }
    … …  
    27812783
    27822784                if (SUCCEEDED(rc))
    2783                     CHECK_ERROR(machine, COMSETTER(AutostopType)(enmAutostopType));
     2785                    CHECK_ERROR(sessionMachine, COMSETTER(AutostopType)(enmAutostopType));
    27842786                break;
    27852787            }
    … …  
    28002802                else
    28012803                {
    2802                     CHECK_ERROR(machine, AttachHostPCIDevice(iHostAddr, iGuestAddr, TRUE));
     2804                    CHECK_ERROR(sessionMachine, AttachHostPCIDevice(iHostAddr, iGuestAddr, TRUE));
    28032805                }
    28042806
    … …  
    28172819                else
    28182820                {
    2819                     CHECK_ERROR(machine, DetachHostPCIDevice(iHostAddr));
     2821                    CHECK_ERROR(sessionMachine, DetachHostPCIDevice(iHostAddr));
    28202822                }
    28212823
    … …  
    28272829            case MODIFYVM_USBCARDREADER:
    28282830            {
    2829                 CHECK_ERROR(machine, COMSETTER(EmulatedUSBCardReaderEnabled)(ValueUnion.f));
     2831                CHECK_ERROR(sessionMachine, COMSETTER(EmulatedUSBCardReaderEnabled)(ValueUnion.f));
    28302832                break;
    28312833            }
    … …  
    28372839                if (bstr == "default")
    28382840                    bstr = Bstr::Empty;
    2839                 CHECK_ERROR(machine, COMSETTER(DefaultFrontend)(bstr.raw()));
     2841                CHECK_ERROR(sessionMachine, COMSETTER(DefaultFrontend)(bstr.raw()));
    28402842                break;
    28412843            }
    … …  
    28522854    /* commit changes */
    28532855    if (SUCCEEDED(rc))
    2854         CHECK_ERROR(machine, SaveSettings());
     2856        CHECK_ERROR(sessionMachine, SaveSettings());
    28552857
    28562858    /* it's important to always close sessions */

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

    r55214 r55234  
    198198    {
    199199        ComPtr<ISnapshot> pCurrentSnapshot;
    200         CHECK_ERROR2_RET(pMachine,COMGETTER(CurrentSnapshot)(pCurrentSnapshot.asOutParam()), RTEXITCODE_FAILURE);
     200        CHECK_ERROR2_RET(pMachine, COMGETTER(CurrentSnapshot)(pCurrentSnapshot.asOutParam()), RTEXITCODE_FAILURE);
    201201        hrc = showSnapshots(pSnapshot, pCurrentSnapshot, enmDetails);
    202202        if (FAILED(hrc))
    … …  
    286286    {
    287287        /* replace the (read-only) IMachine object by a writable one */
    288         CHECK_ERROR_BREAK(a->session, COMGETTER(Machine)(pMachine.asOutParam()));
     288        ComPtr<IMachine> sessionMachine;
     289        CHECK_ERROR_BREAK(a->session, COMGETTER(Machine)(sessionMachine.asOutParam()));
    289290
    290291        /* switch based on the command */
    … …  
    347348
    348349            ComPtr<IProgress> progress;
    349             CHECK_ERROR_BREAK(pMachine, TakeSnapshot(name.raw(), desc.raw(),
    350                                                      fPause,
    351                                                      progress.asOutParam()));
     350            CHECK_ERROR_BREAK(sessionMachine, TakeSnapshot(name.raw(), desc.raw(),
     351                                                           fPause,
     352                                                           progress.asOutParam()));
    352353
    353354            rc = showProgress(progress);
    … …  
    382383            if (fRestoreCurrent)
    383384            {
    384                 CHECK_ERROR_BREAK(pMachine, COMGETTER(CurrentSnapshot)(pSnapshot.asOutParam()));
     385                CHECK_ERROR_BREAK(sessionMachine, COMGETTER(CurrentSnapshot)(pSnapshot.asOutParam()));
    385386            }
    386387            else
    387388            {
    388389                // restore or delete snapshot: then resolve cmd line argument to snapshot instance
    389                 CHECK_ERROR_BREAK(pMachine, FindSnapshot(Bstr(a->argv[2]).raw(),
     390                CHECK_ERROR_BREAK(sessionMachine, FindSnapshot(Bstr(a->argv[2]).raw(),
    390391                                                         pSnapshot.asOutParam()));
    391392            }
    … …  
    395396            if (fDelete)
    396397            {
    397                 CHECK_ERROR_BREAK(pMachine, DeleteSnapshot(bstrSnapGuid.raw(),
     398                CHECK_ERROR_BREAK(sessionMachine, DeleteSnapshot(bstrSnapGuid.raw(),
    398399                                                           pProgress.asOutParam()));
    399400            }
    … …  
    402403                // restore or restore current
    403404                RTPrintf("Restoring snapshot %ls\n", bstrSnapGuid.raw());
    404                 CHECK_ERROR_BREAK(pMachine, RestoreSnapshot(pSnapshot, pProgress.asOutParam()));
     405                CHECK_ERROR_BREAK(sessionMachine, RestoreSnapshot(pSnapshot, pProgress.asOutParam()));
    405406            }
    406407
    … …  
    422423                || !strcmp(a->argv[2], "-current"))
    423424            {
    424                 CHECK_ERROR_BREAK(pMachine, COMGETTER(CurrentSnapshot)(pSnapshot.asOutParam()));
     425                CHECK_ERROR_BREAK(sessionMachine, COMGETTER(CurrentSnapshot)(pSnapshot.asOutParam()));
    425426            }
    426427            else
    427428            {
    428                 CHECK_ERROR_BREAK(pMachine, FindSnapshot(Bstr(a->argv[2]).raw(),
     429                CHECK_ERROR_BREAK(sessionMachine, FindSnapshot(Bstr(a->argv[2]).raw(),
    429430                                                         pSnapshot.asOutParam()));
    430431            }
    … …  
    480481            ComPtr<ISnapshot> pSnapshot;
    481482
    482             CHECK_ERROR_BREAK(pMachine, FindSnapshot(Bstr(a->argv[2]).raw(),
    483                                                      pSnapshot.asOutParam()));
     483            CHECK_ERROR_BREAK(sessionMachine, FindSnapshot(Bstr(a->argv[2]).raw(),
     484                                                           pSnapshot.asOutParam()));
    484485
    485486            /* get the machine of the given snapshot */
    … …  
    489490        }
    490491        else if (!strcmp(a->argv[1], "list"))
    491             rc = handleSnapshotList(a, pMachine) == RTEXITCODE_SUCCESS ? S_OK : E_FAIL;
     492            rc = handleSnapshotList(a, sessionMachine) == RTEXITCODE_SUCCESS ? S_OK : E_FAIL;
    492493        else if (!strcmp(a->argv[1], "dump"))          // undocumented parameter to debug snapshot info
    493             DumpSnapshot(pMachine);
     494            DumpSnapshot(sessionMachine);
    494495        else
    495496        {
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