Justin Erenkrantz writes:
> --On Thursday, February 27, 2003 6:04 PM +0100 Robert Pluim
> <rpluim@bigfoot.com> wrote:
>
> > - return apr_psprintf(resource->pool, "\"%" SVN_REVNUM_T_FMT "/%s\"",
> > + return apr_psprintf(p, "\"%" SVN_REVNUM_T_FMT "/%s\"",
> > created_rev,
> > - apr_xml_quote_string(resource->pool,
> > + apr_xml_quote_string(p,
> > resource->info->repos_path, 1));
>
> This may be bogus because resource->info->pool may be cleaned up
> before the response is written. The test suite has no coverage
> over any of the lifetime issues and probably wouldn't check the
> ETag value for corruption. -- justin
The case that was causing the problem was when dav_svn_insert_prop
calls dav_svn_getetag, since that causes temporary stuff to be
allocated out of the response pool rather than the subpool in
resource->info->pool. In this particular case, the return value of
dav_svn_getetag is copied into the response pool, so I think it's ok
here. I can't speak to the other cases until I find them :-)
Hmm, is the signature of dav_svn_getetag cast in stone?
Robert
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Feb 27 18:40:44 2003