Changeset 44041 in vbox for trunk/src/VBox/Main/src-server

Timestamp:

Dec 5, 2012 2:27:49 PM (12 years ago)

Author:

vboxsync

Message:

Main/Snapshot: re-add a vital fix for taking snapshots which somehow was lost

File:

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

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

@@ -1504 +1504 @@
             throw rc;
 
-        // if we got this far without an error, then save the media registries
-        // that got modified for the diff images
-        mParent->saveModifiedRegistries();
+        // MUST NOT save the settings or the media registry here, because
+        // this causes trouble with rolling back settings if the user cancels
+        // taking the snapshot after the diff images have been created.
     }
     catch (HRESULT hrc)