Changeset 24780 in vbox for trunk/src/VBox/Frontends/VBoxManage

Timestamp:

Nov 19, 2009 10:14:17 AM (15 years ago)

Author:

vboxsync

Message:

Frontends/VBoxManage: fix error message when trying to change any drive at runtime (as that's not the same as changing the medium), and clearly separate it from hard disk hot-plugging attempts. no functionality change.

File:

• trunk/src/VBox/Frontends/VBoxManage/VBoxManageStorageController.cpp (modified) (1 diff)

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

@@ -185 +185 @@
     }
 
-    if (   fRunTime
-        && (   !RTStrICmp(pszMedium, "none")
-            || !RTStrICmp(pszType, "hdd")))
-    {
-        errorArgument("DVD/HardDisk Drives can't be changed while the VM is still running\n");
+    if (fRunTime && !RTStrICmp(pszType, "hdd"))
+    {
+        errorArgument("Hard disk drives cannot be changed while the VM is running\n");
         goto leave;
     }
 
+    if (fRunTime && !RTStrICmp(pszMedium, "none"))
+    {
+        errorArgument("Drives cannot be removed while the VM is running\n");
+        goto leave;
+    }
+
     if (fRunTime && pszPassThrough)
     {
-        errorArgument("Drive passthrough state can't be changed while the VM is still running\n");
+        errorArgument("Drive passthrough state can't be changed while the VM is running\n");
         goto leave;
     }