Author: rhuijben
Date: Mon Apr 25 11:47:42 2011
New Revision: 1096460

URL: http://svn.apache.org/viewvc?rev=1096460&view=rev
Log:
For ra_local, allocate the result for svn_ra_get_repos_root() and
svn_ra_get_uuid() in the right pool. (see svn_ra.h)

Found by testing with apr_allocator_max_free_set(allocator, 1); in
subversion/svn/main.c.

* subversion/libsvn_ra_local/ra_plugin.c
  (svn_ra_local__get_uuid,
   svn_ra_local__get_repos_root): Duplicate paths in the right pool.

Modified:
    subversion/trunk/subversion/libsvn_ra_local/ra_plugin.c

Modified: subversion/trunk/subversion/libsvn_ra_local/ra_plugin.c
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra_local/ra_plugin.c?rev=1096460&r1=1096459&r2=1096460&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_ra_local/ra_plugin.c (original)
+++ subversion/trunk/subversion/libsvn_ra_local/ra_plugin.c Mon Apr 25 11:47:42 
2011
@@ -586,7 +586,7 @@ svn_ra_local__get_uuid(svn_ra_session_t 
                        apr_pool_t *pool)
 {
   svn_ra_local__session_baton_t *sess = session->priv;
-  *uuid = sess->uuid;
+  *uuid = apr_pstrdup(pool, sess->uuid);
   return SVN_NO_ERROR;
 }
 
@@ -596,7 +596,7 @@ svn_ra_local__get_repos_root(svn_ra_sess
                              apr_pool_t *pool)
 {
   svn_ra_local__session_baton_t *sess = session->priv;
-  *url = sess->repos_url;
+  *url = apr_pstrdup(pool, sess->repos_url);
   return SVN_NO_ERROR;
 }
 


Reply via email to