You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ph...@apache.org on 2010/01/27 04:08:52 UTC
svn commit: r903535 - /subversion/trunk/subversion/libsvn_client/mergeinfo.c
Author: philip
Date: Wed Jan 27 03:08:52 2010
New Revision: 903535
URL: http://svn.apache.org/viewvc?rev=903535&view=rev
Log:
Fix a free memory read tracked down with valgrind.
* subversion/libsvn_client/mergeinfo.c
(svn_client_mergeinfo_log): Use scratch_pool rather than iterpool
to duplicate a range that must persist during the loop.
Modified:
subversion/trunk/subversion/libsvn_client/mergeinfo.c
Modified: subversion/trunk/subversion/libsvn_client/mergeinfo.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_client/mergeinfo.c?rev=903535&r1=903534&r2=903535&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_client/mergeinfo.c (original)
+++ subversion/trunk/subversion/libsvn_client/mergeinfo.c Wed Jan 27 03:08:52 2010
@@ -2015,7 +2015,7 @@
svn_merge_range_t *youngest_range = svn_merge_range_dup(
APR_ARRAY_IDX(master_inheritable_rangelist,
master_inheritable_rangelist->nelts - 1,
- svn_merge_range_t *), iterpool);
+ svn_merge_range_t *), scratch_pool);
apr_array_header_t *youngest_rangelist;
youngest_range->start = youngest_range->end - 1;