You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Daniel Shahaf <da...@elego.de> on 2012/04/26 06:44:21 UTC
Re: svn commit: r1330571 -
/subversion/trunk/subversion/mod_dav_svn/lock.c
stsp@apache.org wrote on Wed, Apr 25, 2012 at 21:38:08 -0000:
> Author: stsp
> Date: Wed Apr 25 21:38:08 2012
> New Revision: 1330571
>
> URL: http://svn.apache.org/viewvc?rev=1330571&view=rev
> Log:
> Follow-up to 1330516: Restore client-side output expected by some tests.
>
> Should fix failing tests on the neon-bdb buildbot.
>
> * subversion/mod_dav_svn/lock.c
> (append_locks, refresh_locks): Do no sanitize errors that the test suite
> expects to see. These errors will (hopefully!) never expose sensitive
> information to clients.
>
> Modified:
> subversion/trunk/subversion/mod_dav_svn/lock.c
>
> Modified: subversion/trunk/subversion/mod_dav_svn/lock.c
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/mod_dav_svn/lock.c?rev=1330571&r1=1330570&r2=1330571&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/mod_dav_svn/lock.c (original)
> +++ subversion/trunk/subversion/mod_dav_svn/lock.c Wed Apr 25 21:38:08 2012
> @@ -765,7 +765,12 @@ append_locks(dav_lockdb *lockdb,
> DAV_ERR_LOCK_SAVE_LOCK,
> "Anonymous lock creation is not allowed.");
> }
> - else if (serr && serr->apr_err == SVN_ERR_REPOS_HOOK_FAILURE)
> + else if (serr && (serr->apr_err == SVN_ERR_REPOS_HOOK_FAILURE ||
> + serr->apr_err == SVN_ERR_FS_PATH_ALREADY_LOCKED ||
> + serr->apr_err == SVN_ERR_FS_NO_SUCH_LOCK ||
> + serr->apr_err == SVN_ERR_FS_LOCK_EXPIRED ||
> + serr->apr_err == SVN_ERR_FS_BAD_LOCK_TOKEN ||
> + serr->apr_err == SVN_ERR_FS_OUT_OF_DATE))
.oO ( SVN_ERR_IS_LOCK_ERROR() ? )
Re: svn commit: r1330571 -
/subversion/trunk/subversion/mod_dav_svn/lock.c
Posted by Stefan Sperling <st...@elego.de>.
On Thu, Apr 26, 2012 at 07:44:21AM +0300, Daniel Shahaf wrote:
> stsp@apache.org wrote on Wed, Apr 25, 2012 at 21:38:08 -0000:
> > Author: stsp
> > Date: Wed Apr 25 21:38:08 2012
> > New Revision: 1330571
> > - else if (serr && serr->apr_err == SVN_ERR_REPOS_HOOK_FAILURE)
> > + else if (serr && (serr->apr_err == SVN_ERR_REPOS_HOOK_FAILURE ||
> > + serr->apr_err == SVN_ERR_FS_PATH_ALREADY_LOCKED ||
> > + serr->apr_err == SVN_ERR_FS_NO_SUCH_LOCK ||
> > + serr->apr_err == SVN_ERR_FS_LOCK_EXPIRED ||
> > + serr->apr_err == SVN_ERR_FS_BAD_LOCK_TOKEN ||
> > + serr->apr_err == SVN_ERR_FS_OUT_OF_DATE))
>
> .oO ( SVN_ERR_IS_LOCK_ERROR() ? )
Thanks for making this suggestion! r1330690.
Re: svn commit: r1330571 -
/subversion/trunk/subversion/mod_dav_svn/lock.c
Posted by Stefan Sperling <st...@elego.de>.
On Thu, Apr 26, 2012 at 07:44:21AM +0300, Daniel Shahaf wrote:
> stsp@apache.org wrote on Wed, Apr 25, 2012 at 21:38:08 -0000:
> > Author: stsp
> > Date: Wed Apr 25 21:38:08 2012
> > New Revision: 1330571
> > - else if (serr && serr->apr_err == SVN_ERR_REPOS_HOOK_FAILURE)
> > + else if (serr && (serr->apr_err == SVN_ERR_REPOS_HOOK_FAILURE ||
> > + serr->apr_err == SVN_ERR_FS_PATH_ALREADY_LOCKED ||
> > + serr->apr_err == SVN_ERR_FS_NO_SUCH_LOCK ||
> > + serr->apr_err == SVN_ERR_FS_LOCK_EXPIRED ||
> > + serr->apr_err == SVN_ERR_FS_BAD_LOCK_TOKEN ||
> > + serr->apr_err == SVN_ERR_FS_OUT_OF_DATE))
>
> .oO ( SVN_ERR_IS_LOCK_ERROR() ? )
Thanks for making this suggestion! r1330690.