On Fri, 17 Nov 2006, David Glasser wrote:
> On 11/17/06, Peter Lundblad <plundblad@google.com> wrote:
> >A quick look at the code yields that it shold do so. I haven't tried
> >it and don't know why it complains about the 's', though. So, if the
> >quoted
> >config file is the one that generates the error, then this might be a
> >bugt.
>
> I think this patch fixes it. Also fixes an error leak. Have to run
> now, and have merely determined that it compiles; somebody else should
> sanity-check, run (and write?) tests, and commit.
I've tested this patch, committed it to trunk, and nominated it for
backport to the 1.4.x line. We already had a test for invalid authz
files -- subversion/tests/cmdline/authz_tests.py:broken_authz_file()
-- but it didn't test the specific error message. FWIW, I've add a
comment character to it.
- Dan
- application/pgp-signature attachment: stored
Received on Sat Nov 18 22:18:13 2006