Changeset 22733 in vbox for trunk/src

Timestamp:

Sep 3, 2009 10:00:08 AM (15 years ago)

Author:

vboxsync

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

51863

Message:

VBoxService: Corrected validation.

File:

• trunk/src/VBox/Additions/common/VBoxService/VBoxServiceExec.cpp (modified) (3 diffs)

trunk/src/VBox/Additions/common/VBoxService/VBoxServiceExec.cpp

@@ -114 +114 @@
 bool VBoxServiceExecValidateFlags(char* pszFlags)
 {
-    if (pszFlags == NULL)
-        return false;
+    bool ret = false;
 
     if (   (NULL != RTStrStr(pszFlags, "TRANSIENT"))
         && (NULL != RTStrStr(pszFlags, "RDONLYGUEST")))
     {
-        return true;
+        ret = true;
     }
-    return false;
+    VBoxServiceVerbose(3, "Validating flags %s = %s\n", 
+                       ((pszFlags == NULL) || (RTStrICmp(pszFlags, "") == 0)) ? "<NULL>" : pszFlags, 
+                       ret ? "true" : "false");
+    return ret;
 }
 
@@ -186 +188 @@
                 else
                 {
-                    !VBoxServiceExecValidateFlags(pszSysprepCmdFlags) ? rc = rc : rc = VERR_ACCESS_DENIED;
+                    VBoxServiceExecValidateFlags(pszSysprepCmdFlags) ? rc = rc : rc = VERR_ACCESS_DENIED;
                     if (RT_SUCCESS(rc))
                     {
@@ -232 +234 @@
                 else
                 {
-                    !VBoxServiceExecValidateFlags(pszSysprepArgsFlags) ? rc = rc : rc = VERR_ACCESS_DENIED;
+                    VBoxServiceExecValidateFlags(pszSysprepArgsFlags) ? rc = rc : rc = VERR_ACCESS_DENIED;
                     if (RT_SUCCESS(rc))
                     {