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;