Changeset 55599 in vbox

Timestamp:

May 2, 2015 5:05:51 AM (9 years ago)

Author:

vboxsync

Message:

GuestProcessImpl.cpp: Turns out the guest doesn't want a terminating string in the environment block.

File:

• trunk/src/VBox/Main/src-client/GuestProcessImpl.cpp (modified) (1 diff)

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

@@ -1081 +1081 @@
         cbArgs = pszArgs ? strlen(pszArgs) + 1 : 0; /* Include terminating zero. */
 
-    /* Prepare environment. */
+    /* Prepare environment.  The guest service dislikes the empty string at the end, so drop it. */
     size_t  cbEnvBlock;
     char   *pszzEnvBlock;
     if (RT_SUCCESS(vrc))
         vrc = mData.mProcess.mEnvironmentChanges.queryUtf8Block(&pszzEnvBlock, &cbEnvBlock);
-
     if (RT_SUCCESS(vrc))
     {
+        Assert(cbEnvBlock > 0);
+        cbEnvBlock--;
+
         /* Prepare HGCM call. */
         VBOXHGCMSVCPARM paParms[16];