Changeset 85412 in vbox for trunk/src/VBox/Frontends/VirtualBox

Timestamp:

Jul 22, 2020 11:23:05 AM (4 years ago)

Author:

vboxsync

Message:

FE/Qt: bugref:9722: VirtualBox Manager: Another fix for cloud console external terminal execution, this time for Linux host; Do not pass arguments as a single line, split them to list by space delimiter for now, more wise splitting will come later.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/manager/UIVirtualBoxManager.cpp (modified) (2 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/manager/UIVirtualBoxManager.cpp

@@ -1241 +1241 @@
     AssertMsgReturnVoid(pAction, ("This slot should be called by action only!\n"));
     const QString strPath = pAction->property("path").toString();
-    QStringList arguments = QStringList() << pAction->property("arguments").toString();
+    QStringList arguments = pAction->property("arguments").toString().split(' ');
 
     /* Get current-item: */
@@ -1258 +1258 @@
 
     /* Execute console application finally: */
-    QProcess::startDetached(QString("%1 %2").arg(strPath, arguments.join(' ')));
+    QProcess::startDetached(strPath, arguments);
 }