Changeset 35984 in vbox for trunk/src/VBox

Timestamp:

Feb 15, 2011 4:16:11 PM (14 years ago)

Author:

vboxsync

Message:

Main: prohibit setting a UUID as the machine or snapshot name

Location:

trunk/src/VBox/Main/src-server

Files:

• MachineImpl.cpp (modified) (1 diff)
• SnapshotImpl.cpp (modified) (1 diff)

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

@@ -938 +938 @@
     if (FAILED(autoCaller.rc())) return autoCaller.rc();
 
+    // prohibit setting a UUID only as the machine name, or else it can
+    // never be found by findMachine()
+    Guid test(aName);
+    if (test.isNotEmpty())
+        return setError(E_INVALIDARG,  tr("A machine cannot have a UUID as its name"));
+
     AutoWriteLock alock(this COMMA_LOCKVAL_SRC_POS);
 

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

@@ -349 +349 @@
     HRESULT rc = S_OK;
     CheckComArgStrNotEmptyOrNull(aName);
+
+    // prohibit setting a UUID only as the machine name, or else it can
+    // never be found by findMachine()
+    Guid test(aName);
+    if (test.isNotEmpty())
+        return setError(E_INVALIDARG,  tr("A machine cannot have a UUID as its name"));
 
     AutoCaller autoCaller(this);