Changeset 35227 in vbox

Timestamp:

Dec 17, 2010 2:39:59 PM (14 years ago)

Author:

vboxsync

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

69072

Message:

VBoxExtPackHelperApp: Older gksu does not grok --description or --.

File:

• trunk/src/VBox/Main/VBoxExtPackHelperApp.cpp (modified) (5 diffs)

trunk/src/VBox/Main/VBoxExtPackHelperApp.cpp

@@ -1316 +1316 @@
 
 #else
+
+    /*
+     * Several of the alternatives below will require a command line.
+     */
+    char *pszCmdLine;
+    int rc = RTGetOptArgvToString(&pszCmdLine, &papszArgs[cSuArgs], RTGETOPTARGV_CNV_QUOTE_BOURNE_SH);
+    if (RT_FAILURE(rc))
+        return RTMsgErrorExit(RTEXITCODE_FAILURE, "RTGetOptArgvToString failed: %Rrc");
+
     /*
      * Look for various standard stuff for executing a program as root.
@@ -1326 +1335 @@
     bool        fHaveDisplayVar = RTEnvExist("DISPLAY");
     int         iSuArg          = cSuArgs;
-    char       *pszCmdLine      = NULL;
     PRTHANDLE   pStdNull        = NULL;
     RTHANDLE    StdNull;
     char        szExecTool[260];
     char        szXterm[260];
-    int         rc;
 
     /*
@@ -1372 +1379 @@
             pStdNull = &StdNull;
 
+#if 0 /* older gksu does not grok --description nor '--' and multiple args. */
             iSuArg = cSuArgs - 4;
             papszArgs[cSuArgs - 4] = szExecTool;
@@ -1381 +1389 @@
                                    : "VirtualBox extension pack maintainer";
             papszArgs[cSuArgs - 1] = "--";
+#else
+            iSuArg = cSuArgs - 2;
+            papszArgs[cSuArgs - 2] = szExecTool;
+            papszArgs[cSuArgs - 1] = pszCmdLine;
+            papszArgs[cSuArgs] = NULL;
+#endif
         }
         else
@@ -1403 +1417 @@
              && FindExecTool(szXterm, sizeof(szXterm), "xterm"))
     {
-        rc = RTGetOptArgvToString(&pszCmdLine, &papszArgs[cSuArgs], RTGETOPTARGV_CNV_QUOTE_BOURNE_SH);
-        if (RT_SUCCESS(rc))
-        {
-            iSuArg = cSuArgs - 9;
-            papszArgs[cSuArgs - 9] = szXterm;
-            papszArgs[cSuArgs - 8] = "-T";
-            papszArgs[cSuArgs - 7] = iCmd == CMD_INSTALL
-                                   ? "VirtualBox extension pack installer - su"
-                                   : iCmd == CMD_UNINSTALL
-                                   ? "VirtualBox extension pack uninstaller - su"
-                                   : "VirtualBox extension pack maintainer - su";
-            papszArgs[cSuArgs - 6] = "-e";
-            papszArgs[cSuArgs - 5] = szExecTool;
-            papszArgs[cSuArgs - 4] = "-";
-            papszArgs[cSuArgs - 3] = "root";
-            papszArgs[cSuArgs - 2] = "-c";
-            papszArgs[cSuArgs - 1] = pszCmdLine;
-            papszArgs[cSuArgs] = NULL;
-        }
-        else
-            RTMsgError("RTGetOptArgvToString failed: %Rrc");
+        iSuArg = cSuArgs - 9;
+        papszArgs[cSuArgs - 9] = szXterm;
+        papszArgs[cSuArgs - 8] = "-T";
+        papszArgs[cSuArgs - 7] = iCmd == CMD_INSTALL
+                               ? "VirtualBox extension pack installer - su"
+                               : iCmd == CMD_UNINSTALL
+                               ? "VirtualBox extension pack uninstaller - su"
+                               : "VirtualBox extension pack maintainer - su";
+        papszArgs[cSuArgs - 6] = "-e";
+        papszArgs[cSuArgs - 5] = szExecTool;
+        papszArgs[cSuArgs - 4] = "-";
+        papszArgs[cSuArgs - 3] = "root";
+        papszArgs[cSuArgs - 2] = "-c";
+        papszArgs[cSuArgs - 1] = pszCmdLine;
+        papszArgs[cSuArgs] = NULL;
     }
     else if (fHaveDisplayVar)