Changeset 27172 in vbox

Timestamp:

Mar 8, 2010 3:55:12 PM (15 years ago)

Author:

vboxsync

Message:

Main: fix trunk regression that caused Machine::SetHWVirtExProperty to not work with VBoxManage (due to pointer voodoo in Backupable<> template, we can't take an address and then call backup())

File:

• trunk/src/VBox/Main/MachineImpl.cpp (modified) (2 diffs)

trunk/src/VBox/Main/MachineImpl.cpp

@@ -1709 +1709 @@
     {
         case HWVirtExPropertyType_Enabled:
-            pb = &mHWData->mHWVirtExEnabled;
+            setModified(IsModified_MachineData);
+            mHWData.backup();
+            mHWData->mHWVirtExEnabled = !!aVal;
             break;
 
         case HWVirtExPropertyType_Exclusive:
-            pb = &mHWData->mHWVirtExExclusive;
+            setModified(IsModified_MachineData);
+            mHWData.backup();
+            mHWData->mHWVirtExExclusive = !!aVal;
             break;
 
         case HWVirtExPropertyType_VPID:
-            pb = &mHWData->mHWVirtExVPIDEnabled;
+            setModified(IsModified_MachineData);
+            mHWData.backup();
+            mHWData->mHWVirtExVPIDEnabled = !!aVal;
             break;
 
         case HWVirtExPropertyType_NestedPaging:
-            pb = &mHWData->mHWVirtExNestedPagingEnabled;
+            setModified(IsModified_MachineData);
+            mHWData.backup();
+            mHWData->mHWVirtExNestedPagingEnabled = !!aVal;
             break;
 
@@ -1732 +1740 @@
     }
 
-    if (*pb != !!aVal)
-    {
-        setModified(IsModified_MachineData);
-        mHWData.backup();
-        *pb = !!aVal;
-    }
     return S_OK;
 }