Changeset 35136 in vbox for trunk/src

Timestamp:

Dec 15, 2010 2:40:30 PM (14 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

68973

Message:

Frontends/VBoxManage: small functionality/error handling fix with setting the medium type with storageattach

File:

• trunk/src/VBox/Frontends/VBoxManage/VBoxManageStorageController.cpp (modified) (3 diffs)

trunk/src/VBox/Frontends/VBoxManage/VBoxManageStorageController.cpp

@@ -73 +73 @@
     ULONG device = ~0U;
     bool fForceUnmount = false;
+    bool fSetMediumType = false;
     MediumType_T mediumType = MediumType_Normal;
     Bstr bstrComment;
@@ -215 +216 @@
                 if (RT_FAILURE(vrc))
                     return errorArgument("Invalid hard disk type '%s'", ValueUnion.psz);
+                fSetMediumType = true;
                 break;
             }
@@ -559 +561 @@
 
             // set medium type, if so desired
-            if (pMedium2Mount && mediumType != MediumType_Normal)
+            if (pMedium2Mount && fSetMediumType)
             {
                 CHECK_ERROR(pMedium2Mount, COMSETTER(Type)(mediumType));
+                throw  Utf8Str("Failed to set the medium type");
             }