[ros-diffs] [gschneider] 47305: [SHELL32] - Duplicate the string instead of freeing the memory twice at two different positions - Fixes a heap warning when stating cmd from the desktop See issue #4924 for more details.

gschneider at svn.reactos.org gschneider at svn.reactos.org
Sat May 22 17:16:14 CEST 2010


Author: gschneider
Date: Sat May 22 17:16:14 2010
New Revision: 47305

URL: http://svn.reactos.org/svn/reactos?rev=47305&view=rev
Log:
[SHELL32]
- Duplicate the string instead of freeing the memory twice at two different positions
- Fixes a heap warning when stating cmd from the desktop
See issue #4924 for more details.

Modified:
    trunk/reactos/dll/win32/shell32/shelllink.c

Modified: trunk/reactos/dll/win32/shell32/shelllink.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/shelllink.c?rev=47305&r1=47304&r2=47305&view=diff
==============================================================================
--- trunk/reactos/dll/win32/shell32/shelllink.c [iso-8859-1] (original)
+++ trunk/reactos/dll/win32/shell32/shelllink.c [iso-8859-1] Sat May 22 17:16:14 2010
@@ -2828,7 +2828,7 @@
     }
     else if (This->sArgs != NULL)
     {
-        args = This->sArgs;
+        args = strdupW( This->sArgs );
     }
 
     memset( &sei, 0, sizeof sei );




More information about the Ros-diffs mailing list