Changeset 29638 in vbox

Timestamp:

May 18, 2010 1:53:07 PM (15 years ago)

Author:

vboxsync

Message:

Guest Control/Main: Better credentials checking.

File:

• trunk/src/VBox/Main/GuestImpl.cpp (modified) (2 diffs)

trunk/src/VBox/Main/GuestImpl.cpp

@@ -779 +779 @@
     CheckComArgStrNotEmptyOrNull(aCommand);
     CheckComArgOutPointerValid(aPID);
-    CheckComArgStrNotEmptyOrNull(aUserName); /* Do not allow anonymous executions (with system rights). */
     CheckComArgOutPointerValid(aProgress);
+
+    /* Do not allow anonymous executions (with system rights). */
+    if (RT_UNLIKELY((aUserName) == NULL || *(aUserName) == '\0'))
+        return setError(E_INVALIDARG, tr("No user name specified"));
 
     AutoCaller autoCaller(this);
@@ -1046 +1049 @@
                 {
                     rc = setError(VBOX_E_IPRT_ERROR,
-                                  tr("The operation was canceled."));
+                                  tr("The operation was canceled"));
                 }
             }