Changeset 55579 in vbox for trunk/src/VBox/Additions

Timestamp:

May 1, 2015 10:02:24 AM (10 years ago)

Author:

vboxsync

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

99952

Message:

RTStrCopy is a better choice here.

File:

• trunk/src/VBox/Additions/common/VBoxService/VBoxServiceControlSession.cpp (modified) (1 diff)

trunk/src/VBox/Additions/common/VBoxService/VBoxServiceControlSession.cpp

@@ -2350 +2350 @@
         {
             case VBOXSERVICESESSIONOPT_LOG_FILE:
-                if (!RTStrPrintf(g_szLogFile, sizeof(g_szLogFile), "%s", ValueUnion.psz))
-                    return RTMsgErrorExit(RTEXITCODE_FAILURE, "Unable to set logfile name to '%s'",
-                                          ValueUnion.psz);
+                rc = RTStrCopy(g_szLogFile, sizeof(g_szLogFile), ValueUnion.psz);
+                if (RT_FAILURE(rc))
+                    return RTMsgErrorExit(RTEXITCODE_FAILURE, "Error copying log file name: %Rrc", rc);
                 break;
 #ifdef DEBUG