Changeset 36365 in vbox

Timestamp:

Mar 23, 2011 12:26:45 PM (14 years ago)

Author:

vboxsync

Message:

Main: Fix for parameter handling in setGuestPropertyToVM().

File:

• trunk/src/VBox/Main/src-server/MachineImpl.cpp (modified) (2 diffs)

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

@@ -4994 +4994 @@
                                       IN_BSTR aFlags)
 {
+    CheckComArgStrNotEmptyOrNull(aName);
+    if ((aValue != NULL) && !VALID_PTR(aValue))
+        return E_INVALIDARG;
+    if ((aFlags != NULL) && !VALID_PTR(aFlags))
+        return E_INVALIDARG;
+
     HRESULT rc;
 
@@ -5009 +5015 @@
                       /** @todo Fix when adding DeleteGuestProperty(),
                                    see defect. */
-                      *aValue ? aValue : NULL, *aFlags ? aFlags : NULL,
+                      aValue, aFlags,
                       true /* isSetter */,
                       &dummy, &dummy64, &dummy);