allocate string freed with free() later using malloc(), not new[] (fixes patch 772404)

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@22044 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

diff --git a/src/msw/main.cpp b/src/msw/main.cpp
index 5617b1f15caa3ae57d0da7ee14457a0aeee6463b..f761ec971a33e85714679c6de9bc90d1498141e0 100644 (file)
--- a/src/msw/main.cpp
+++ b/src/msw/main.cpp
@@ -172,7 +172,9 @@ wxChar **ConvertToStandardCommandArgs(const wxChar *p, int& argc)
     // and +1 here for the terminating NULL
     wxChar **argv = new wxChar *[argc + 1];
 
     // and +1 here for the terminating NULL
     wxChar **argv = new wxChar *[argc + 1];
 

-    argv[0] = new wxChar[MAX_PATH];
+    // as we use wxStrdup below we must allocate the first argument using
+    // malloc(), not new[], as well
+    argv[0] = (wxChar *)malloc(MAX_PATH * sizeof(wxChar));

     ::GetModuleFileName(wxhInstance, argv[0], MAX_PATH);
 
     // copy all the other arguments to wxApp::argv[]
     ::GetModuleFileName(wxhInstance, argv[0], MAX_PATH);
 
     // copy all the other arguments to wxApp::argv[]