You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by tr...@apache.org on 2010/04/09 23:41:36 UTC

svn commit: r932627 - /apr/apr/branches/1.5.x/file_io/win32/filepath.c

Author: trawick
Date: Fri Apr  9 21:41:36 2010
New Revision: 932627

URL: http://svn.apache.org/viewvc?rev=932627&view=rev
Log:
Merge r932067 from trunk:

avoid uninitialized memory access in apr_filepath_merge()

it was harmless other than triggering errors from tools
that check memory access

Submitted by: Arsen Chaloyan <achaloyan yahoo.com>
Reviewed by: trawick


Modified:
    apr/apr/branches/1.5.x/file_io/win32/filepath.c

Modified: apr/apr/branches/1.5.x/file_io/win32/filepath.c
URL: http://svn.apache.org/viewvc/apr/apr/branches/1.5.x/file_io/win32/filepath.c?rev=932627&r1=932626&r2=932627&view=diff
==============================================================================
--- apr/apr/branches/1.5.x/file_io/win32/filepath.c (original)
+++ apr/apr/branches/1.5.x/file_io/win32/filepath.c Fri Apr  9 21:41:36 2010
@@ -949,8 +949,7 @@ APR_DECLARE(apr_status_t) apr_filepath_m
         }
     }
 
-    *newpath = apr_pmemdup(p, path, pathlen + 1);
-    (*newpath)[pathlen] = '\0';
+    *newpath = apr_pstrmemdup(p, path, pathlen);
     return APR_SUCCESS;
 }