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

Re: Input validation observations

From: Julian Foad <julian.foad_at_wandisco.com>
Date: Thu, 02 Dec 2010 12:21:45 +0000

On Thu, 2010-12-02 at 14:05 +0530, Noorul Islam K M wrote:
> Julian Foad <julian.foad_at_wandisco.com> writes:
>
> > I tried some potentially invalid inputs to "svn" a week or two ago and
> > made notes on what I found. Just posting here in case anyone wants to
> > do something about one or more of them.
> >
> > Noorul, I'm including you in the "To" addresses because you said you
> > were looking for more small tasks to do, so feel free to pick one of
> > these if you want to.
> >
> > Where I end with a question mark, it means I'm not sure if we want this
> > change, it's just a suggestion.
> >
> > * "svn checkout ^/ ^/y" -> "A asf/cxf, A asf/cxf/utils, ...". (Don't
> > try this without being ready on the Ctrl-C or Ctrl-\!) It seems to
> > ignore "^/y" and create ./(basename(^/)); should fail: "'^/y' is not a
> > WC path".
> >
> > * "svn checkout ^/subversion/trunk/build ^/y" -> "Checked out revision
> > 1040465. URL 'https://svn.apache.org/repos/asf/y' doesn't exist".
> > Bleach - that's just crazy. Should fail: "'^/y' is not a WC path".
> >
>
> In this case code assumes that the user is trying to checkout from both
> source into current directory. i.e, If all source targets are URLs then
> the checkout target becomes "."

Oh yes, you're right - there's no problem there. I didn't notice that
"svn checkout URL URL" was a supported syntax. Thanks.

- Julian
Received on 2010-12-02 13:22:25 CET

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