Changeset 100200 in vbox for trunk

Timestamp:

Jun 16, 2023 3:07:52 PM (18 months ago)

Author:

vboxsync

Message:

Main/Snapshot: Attempting to take a live snapshot, i.e. calling
IMachine::takeSnapshot() with 'pause=false', of a non-running VM should
fail and provide an appropriate error message.

File:

• trunk/src/VBox/Main/src-server/SnapshotImpl.cpp (modified) (1 diff)

trunk/src/VBox/Main/src-server/SnapshotImpl.cpp

@@ -1628 +1628 @@
     if (Global::IsTransient(mData->mMachineState))
         return setError(VBOX_E_INVALID_VM_STATE,
-                        tr("Cannot take a snapshot of the machine while it is changing the state (machine state: %s)"),
+                        tr("Cannot take a snapshot of the virtual machine while it is changing state (machine state: %s)"),
                         Global::stringifyMachineState(mData->mMachineState));
+
+    if (!fPause && mData->mMachineState != MachineState_Running)
+        return setError(VBOX_E_INVALID_VM_STATE,
+                        tr("Cannot take a live snapshot of a virtual machine unless it is running."));
 
     HRESULT hrc = i_checkStateDependency(MutableOrSavedOrRunningStateDep);