On 18 Jul 2005 14:24:57 -0500, kfogel@collab.net <kfogel@collab.net> wrote:
> Erik Huelsmann <ehuels@gmail.com> writes:
> > > > ------------------------------------------------------------------------
> > > >
> > > > Index: subversion/libsvn_ra_dav/fetch.c
> > > > ===================================================================
> > > > --- subversion/libsvn_ra_dav/fetch.c (Revision 15306)
> > > > +++ subversion/libsvn_ra_dav/fetch.c (Arbeitskopie)
> > > > @@ -1578,7 +1578,7 @@
> > > >
> > > > /* ### Should svn_ra_dav__parsed_request() take care of storing auth
> > > > ### info itself? */
> > > > - SVN_ERR( svn_ra_dav__maybe_store_auth_info_after_result(err, ras) );
> > > > + err = svn_ra_dav__maybe_store_auth_info_after_result(err, ras);
> > > >
> > > > if (baton.err)
> > > > return baton.err;
> > > >
> >
> > If baton.err != NULL, you leak err here if that too is not NULL.
>
> But note that Steve *fixed* a bug (one that I wrote), whereby
I'm sorry if I made the impression I'm not appreciating his efforts! I
never meant to put him down! But, he asked for comments, so I did a
review - admittedly without a repository or working copy available.
> baton.err was failing to dominate when it should dominate, because I
> used SVN_ERR() too hastily. On trunk, there's code after the 'if'
> above that takes care of the error leak, you just don't see that code
> here because Steve's patch is against non-trunk code.
>
> Ben Collins-Sussman is fixing this right now, basically using Steve's
> patch, I believe. Then there will be a separate commit to fix the
> regression against 1.1.x servers.
>
> Steve, thanks!
Hear, hear! :-)
bye,
Erik.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Jul 19 14:26:49 2005