Changeset 15334 in vbox for trunk/src/VBox/Main/DVDDriveImpl.cpp

Timestamp:

Dec 11, 2008 7:37:55 PM (16 years ago)

Author:

vboxsync

Message:

Main: #3377: Implicitly detach DVD/Floppy from an unregistered machine (as opposed to hard disks, this does not have side effects).

File:

• trunk/src/VBox/Main/DVDDriveImpl.cpp (modified) (10 diffs)

trunk/src/VBox/Main/DVDDriveImpl.cpp

@@ -275 +275 @@
             if (SUCCEEDED (rc))
             {
-                mData.backup();
-
-                unmount();
-
-                mData->mImage = image;
-                mData->mState = DriveState_ImageMounted;
-
-                /* leave the lock before informing callbacks */
-                alock.unlock();
-
-                mParent->onDVDDriveChange();
+                /* umount() will backup data */
+                rc = unmount();
+                if (SUCCEEDED (rc))
+                {
+                    mData->mImage = image;
+                    mData->mState = DriveState_ImageMounted;
+
+                    /* leave the lock before informing callbacks */
+                    alock.unlock();
+
+                    mParent->onDVDDriveChange();
+                }
             }
         }
@@ -309 +310 @@
         !mData->mHostDrive.equalsTo (aHostDVDDrive))
     {
-        mData.backup();
-
-        unmount();
-
-        mData->mHostDrive = aHostDVDDrive;
-        mData->mState = DriveState_HostDriveCaptured;
-
-        /* leave the lock before informing callbacks */
-        alock.unlock();
-
-        mParent->onDVDDriveChange();
+        /* umount() will backup data */
+        HRESULT rc = unmount();
+        if (SUCCEEDED (rc))
+        {
+            mData->mHostDrive = aHostDVDDrive;
+            mData->mState = DriveState_HostDriveCaptured;
+
+            /* leave the lock before informing callbacks */
+            alock.unlock();
+
+            mParent->onDVDDriveChange();
+        }
     }
 
@@ -338 +340 @@
     if (mData->mState != DriveState_NotMounted)
     {
-        mData.backup();
-
-        unmount();
-
-        mData->mState = DriveState_NotMounted;
-
-        /* leave the lock before informing callbacks */
-        alock.unlock();
-
-        mParent->onDVDDriveChange();
+        /* umount() will backup data */
+        HRESULT rc = unmount();
+        if (SUCCEEDED (rc))
+        {
+            mData->mState = DriveState_NotMounted;
+
+            /* leave the lock before informing callbacks */
+            alock.unlock();
+
+            mParent->onDVDDriveChange();
+        }
     }
 
@@ -385 +388 @@
 
 /**
- *  Loads settings from the given machine node.
- *  May be called once right after this object creation.
- *
- *  @param aMachineNode <Machine> node.
- *
- *  @note Locks this object for writing.
+ * Loads settings from the given machine node. May be called once right after
+ * this object creation.
+ *
+ * @param aMachineNode  <Machine> node.
+ *
+ * @note Locks this object for writing.
  */
 HRESULT DVDDrive::loadSettings (const settings::Key &aMachineNode)
@@ -469 +472 @@
 
 /**
- *  Saves settings to the given machine node.
- *
- *  @param aMachineNode <Machine> node.
- *
- *  @note Locks this object for reading.
+ * Saves settings to the given machine node.
+ *
+ * @param aMachineNode  <Machine> node.
+ *
+ * @note Locks this object for reading.
  */
 HRESULT DVDDrive::saveSettings (settings::Key &aMachineNode)
@@ -530 +533 @@
 
 /**
- *  @note Locks this object for writing.
+ * @note Locks this object for writing.
  */
 bool DVDDrive::rollback()
@@ -567 +570 @@
 
 /**
- *  @note Locks this object for writing, together with the peer object (also
- *  for writing) if there is one.
+ * @note Locks this object for writing, together with the peer object (also for
+ *       writing) if there is one.
  */
 void DVDDrive::commit()
@@ -605 +608 @@
 
 /**
- *  @note Locks this object for writing, together with the peer object
- *  represented by @a aThat (locked for reading).
+ * @note Locks this object for writing, together with the peer object
+ *       represented by @a aThat (locked for reading).
  */
 void DVDDrive::copyFrom (DVDDrive *aThat)
@@ -628 +631 @@
 }
 
-// private methods
-////////////////////////////////////////////////////////////////////////////////
-
-/**
- *  Helper to unmount a drive.
- *
- *  @return COM status code
- *
+/**
+ * Helper to unmount a drive.
+ *
+ * @note Must be called from under this object's write lock.
  */
 HRESULT DVDDrive::unmount()
 {
     AssertReturn (isWriteLockOnCurrentThread(), E_FAIL);
+
+    mData.backup();
 
     if (mData->mImage)
@@ -646 +647 @@
         mData->mHostDrive.setNull();
 
-    return S_OK;
-}
+    mData->mState = DriveState_NotMounted;
+
+    return S_OK;
+}
+
+// private methods
+////////////////////////////////////////////////////////////////////////////////
+
 /* vi: set tabstop=4 shiftwidth=4 expandtab: */