You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by da...@apache.org on 2012/03/30 02:33:33 UTC

svn commit: r1307179 - /subversion/branches/1.7.x-r1306111/subversion/svnserve/serve.c

Author: danielsh
Date: Fri Mar 30 00:33:32 2012
New Revision: 1307179

URL: http://svn.apache.org/viewvc?rev=1307179&view=rev
Log:
On the 1.7.x-r1306111 branch, don't allocate a static variable in a short-lived pool.

Found by: gstein

* subversion/svnserve/serve.c
  (get_dir): Fix lifefime of MISSING_DATE.

Modified:
    subversion/branches/1.7.x-r1306111/subversion/svnserve/serve.c

Modified: subversion/branches/1.7.x-r1306111/subversion/svnserve/serve.c
URL: http://svn.apache.org/viewvc/subversion/branches/1.7.x-r1306111/subversion/svnserve/serve.c?rev=1307179&r1=1307178&r2=1307179&view=diff
==============================================================================
--- subversion/branches/1.7.x-r1306111/subversion/svnserve/serve.c (original)
+++ subversion/branches/1.7.x-r1306111/subversion/svnserve/serve.c Fri Mar 30 00:33:32 2012
@@ -1621,9 +1621,9 @@ static svn_error_t *get_dir(svn_ra_svn_c
   SVN_ERR(svn_ra_svn_write_tuple(conn, pool, "!)(!"));
   if (want_contents)
     {
+      const char *missing_date = svn_time_to_cstring(0, pool);
       for (hi = apr_hash_first(pool, entries); hi; hi = apr_hash_next(hi))
         {
-          static const char *missing_date = svn_time_to_cstring(0, pool);
           const char *name = svn__apr_hash_index_key(hi);
           svn_dirent_t *entry = svn__apr_hash_index_val(hi);