[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r29663 - trunk/subversion/svnserve

From: David Glasser <glasser_at_davidglasser.net>
Date: Sat, 1 Mar 2008 14:59:54 -0800

On Sat, Mar 1, 2008 at 2:51 PM, David Glasser <glasser_at_davidglasser.net> wrote:
>
> On Sat, Mar 1, 2008 at 2:37 PM, Hyrum K. Wright
> <hyrum_wright_at_mail.utexas.edu> wrote:
> >
> > glasser_at_tigris.org wrote:
> > > Author: glasser
> > > Date: Sat Mar 1 11:04:42 2008
> > > New Revision: 29663
> > >
> > > Log:
> > > Followup to r29659: *really* fix a bunch of error leaks in the
> > > svnserve Cyrus SASL implementation.
> > >
> > > * subversion/svnserve/cyrus_auth.c
> > > (write_failure): New wrapper around svn_ra_svn_write_cmd_failure
> > > which clears its error argument.
> > > (fail_cmd, cyrus_auth_request): Use write_failure.
> > >
> > > Found by: dionisos
> > >
> > >
> > > Modified:
> > > trunk/subversion/svnserve/cyrus_auth.c
> > >
> > > Modified: trunk/subversion/svnserve/cyrus_auth.c
> > > URL: http://svn.collab.net/viewvc/svn/trunk/subversion/svnserve/cyrus_auth.c?pathrev=29663&r1=29662&r2=29663
> > > ==============================================================================
> > > --- trunk/subversion/svnserve/cyrus_auth.c (original)
> > > +++ trunk/subversion/svnserve/cyrus_auth.c Sat Mar 1 11:04:42 2008
> > > @@ -132,14 +132,24 @@
> > > return svn_ra_svn_flush(conn, pool);
> > > }
> > >
> > > +/* Like svn_ra_svn_write_cmd_failure, but also clears the given error
> > > + and sets it to SVN_NO_ERROR. */
> > > +static svn_error_t *
> > > +write_failure(svn_ra_svn_conn_t *conn, apr_pool_t *pool, svn_error_t **err_p)
> > > +{
> > > + svn_error_t *write_err = svn_ra_svn_write_cmd_failure(conn, pool, *err);
> > > + svn_error_clear(*err);
> > > + *err = SVN_NO_ERROR;
> >
> > subversion/svnserve/cyrus_auth.c: In function 'write_failure':
> > subversion/svnserve/cyrus_auth.c:140: error: 'err' undeclared (first use
> > in this function)
> >
> > Did you mean err_p?
>
> Uh, right. I did (and tested) the first change on my work computer
> and the second on my laptop. Oh, and my laptop didn't have SASL, so
> the whole file was ifdefed out :-) Fixing now.

r29664.

--dave

-- 
David Glasser | glasser@davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-03-02 00:00:19 CET

This is an archived mail posted to the Subversion Dev mailing list.