Re: svn commit: r22729 - in trunk/subversion: libsvn_fs_base libsvn_fs_fs

Ben, I've nominated this for backport to the 1.4.x line.

On Fri, 15 Dec 2006, sussman@tigris.org wrote:

> Author: sussman
> Date: Fri Dec 15 15:43:59 2006
> New Revision: 22729
>
> Log:
> Fix potential filesystem memleak in commit-finalization loop.
>
> * subversion/libsvn_fs_base/tree.c (merge),
> * subversion/libsvn_fs_fs/tree.c (merge): use iterpool in the entries loop.
>
>
> Modified:
> trunk/subversion/libsvn_fs_base/tree.c
> trunk/subversion/libsvn_fs_fs/tree.c
>
> Modified: trunk/subversion/libsvn_fs_base/tree.c
> URL: http://svn.collab.net/viewvc/svn/trunk/subversion/libsvn_fs_base/tree.c?pathrev=22729&r1=22728&r2=22729
> ==============================================================================
> --- trunk/subversion/libsvn_fs_base/tree.c (original)
> +++ trunk/subversion/libsvn_fs_base/tree.c Fri Dec 15 15:43:59 2006
> @@ -2032,12 +2032,12 @@
> if (s_entry)
> {
> SVN_ERR(svn_fs_base__dag_set_entry(target, key, s_entry->id,
> - txn_id, trail, pool));
> + txn_id, trail, iterpool));
> }
> else
> {
> SVN_ERR(svn_fs_base__dag_delete(target, key, txn_id,
> - trail, pool));
> + trail, iterpool));
> }
> }
>
>
> Modified: trunk/subversion/libsvn_fs_fs/tree.c
> URL: http://svn.collab.net/viewvc/svn/trunk/subversion/libsvn_fs_fs/tree.c?pathrev=22729&r1=22728&r2=22729
> ==============================================================================
> --- trunk/subversion/libsvn_fs_fs/tree.c (original)
> +++ trunk/subversion/libsvn_fs_fs/tree.c Fri Dec 15 15:43:59 2006
> @@ -1431,11 +1431,11 @@
> s_entry->id,
> s_entry->kind,
> txn_id,
> - pool));
> + iterpool));
> }
> else
> {
> - SVN_ERR(svn_fs_fs__dag_delete(target, key, txn_id, pool));
> + SVN_ERR(svn_fs_fs__dag_delete(target, key, txn_id, iterpool));
> }
> }
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: svn-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: svn-help@subversion.tigris.org