Changeset 77279 in vbox for trunk/src/VBox/Main/src-client

Timestamp:

Feb 12, 2019 2:47:36 PM (6 years ago)

Author:

vboxsync

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

128794

Message:

Guest Control/Main + FE/Qt: Reverted r128734 (Be more strict for IGuestSession::copyToGuest() / copyFromGuest() and set/validate the filters + flags); not needed and is good enough.

File:

• trunk/src/VBox/Main/src-client/GuestSessionImpl.cpp (modified) (2 diffs)

trunk/src/VBox/Main/src-client/GuestSessionImpl.cpp

@@ -2907 +2907 @@
 
     const size_t cSources = aSources.size();
-    if (!cSources)
-        return setError(E_INVALIDARG, tr("No sources specified"));
-
-    if (   (aFilters.size() != cSources)
-        || (aFlags.size()   != cSources))
-    {
-        return setError(E_INVALIDARG, tr("Filter / flag parameter array sizes don't match to the number of sources specified"));
+    if (   (aFilters.size() && aFilters.size() != cSources)
+        || (aFlags.size()   && aFlags.size()   != cSources))
+    {
+        return setError(E_INVALIDARG, tr("Parameter array sizes don't match to the number of sources specified"));
     }
 
@@ -2989 +2986 @@
 
     const size_t cSources = aSources.size();
-    if (!cSources)
-        return setError(E_INVALIDARG, tr("No sources specified"));
-
-    if (   (aFilters.size() != cSources)
-        || (aFlags.size()   != cSources))
-    {
-        return setError(E_INVALIDARG, tr("Filter / flag parameter array sizes don't match to the number of sources specified"));
+    if (   (aFilters.size() && aFilters.size() != cSources)
+        || (aFlags.size()   && aFlags.size()   != cSources))
+    {
+        return setError(E_INVALIDARG, tr("Parameter array sizes don't match to the number of sources specified"));
     }