Changeset 64212 in vbox for trunk/src

Timestamp:

Oct 11, 2016 3:56:15 PM (8 years ago)

Author:

vboxsync

Message:

Main/Snapshot: make sure that the source medium has no parent any more when we're certain that we no longer want to know about it (even if an error happened), as otherwise this leads to confusion and crashes later, because there would be an attempt to save it as part of a VM's medium registry when the medium isn't operational any more

File:

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

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

@@ -3127 +3127 @@
 
             if (fNeedSourceUninit)
+            {
+                // make sure that the diff image to be deleted has no parent,
+                // even in error cases (where the deparenting may be missing)
+                it->mpSource->i_deparent();
                 it->mpSource->uninit();
+            }
 
             // One attachment is merged, must save the settings