You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by st...@apache.org on 2014/12/08 14:43:52 UTC

svn commit: r1643818 - /subversion/trunk/subversion/mod_dav_svn/repos.c

Author: stsp
Date: Mon Dec  8 13:43:51 2014
New Revision: 1643818

URL: http://svn.apache.org/r1643818
Log:
In mod_dav_svn, destroy an iterpool before returning error.

* subversion/mod_dav_svn/repos.c
  (do_walk): Destroy the iterpool created here before returning an error.

Modified:
    subversion/trunk/subversion/mod_dav_svn/repos.c

Modified: subversion/trunk/subversion/mod_dav_svn/repos.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/mod_dav_svn/repos.c?rev=1643818&r1=1643817&r2=1643818&view=diff
==============================================================================
--- subversion/trunk/subversion/mod_dav_svn/repos.c (original)
+++ subversion/trunk/subversion/mod_dav_svn/repos.c Mon Dec  8 13:43:51 2014
@@ -4322,7 +4322,10 @@ do_walk(walker_ctx_t *ctx,
         {
           err = (*params->func)(&ctx->wres, DAV_CALLTYPE_MEMBER);
           if (err != NULL)
-            return err;
+            {
+              svn_pool_destroy(iterpool);
+              return err;
+            }
         }
       else
         {
@@ -4336,7 +4339,10 @@ do_walk(walker_ctx_t *ctx,
           /* recurse on this collection */
           err = do_walk(ctx, depth - 1, FALSE, iterpool);
           if (err != NULL)
-            return err;
+            {
+              svn_pool_destroy(iterpool);
+              return err;
+            }
 
           /* restore the data */
           ctx->res.collection = FALSE;