Changeset 80613 in vbox for trunk/src/VBox/Main/src-all

Timestamp:

Sep 5, 2019 10:18:23 PM (6 years ago)

Author:

vboxsync

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

133140

Message:

Main/MachineLaunchVMCommonWorker: Simplified darwin specific code for kicking off VirtualVM or override. Was making too many unnecessary string copies. [fix] bugref:9341

File:

• trunk/src/VBox/Main/src-all/MachineLaunchVMCommonWorker.cpp (modified) (1 diff)

trunk/src/VBox/Main/src-all/MachineLaunchVMCommonWorker.cpp

@@ -93 +93 @@
     int vrc = RTPathAppPrivateArch(szPath, sizeof(szPath));
     AssertRCReturn(vrc, vrc);
-    vrc = RTPathEnsureTrailingSeparator(szPath, sizeof(szPath));
-    AssertRCReturn(vrc, vrc);
-    size_t cbBufLeft = strlen(szPath);
+    size_t cchBufLeft = RTPathEnsureTrailingSeparator(szPath, sizeof(szPath));
+    AssertReturn(cchBufLeft > 0, VERR_FILENAME_TOO_LONG);
     char *pszNamePart = &szPath[cbBufLeft]; NOREF(pszNamePart);
     cbBufLeft = sizeof(szPath) - cbBufLeft;