[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: Noorul Islam K M <noorul_at_collab.net>
Date: Thu, 02 Dec 2010 14:05:47 +0530

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 "."

Thanks and Regards
Noorul
Received on 2010-12-02 09:38:26 CET

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.