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

Input validation observations

From: Julian Foad <julian.foad_at_wandisco.com>
Date: Tue, 30 Nov 2010 11:48:42 +0000

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

  * "svn copy a ^/b c" doesn't detect the mixed source types in cl, only
in lib; should reject them at CLI level?

  * "svn info ^/b" -> "Not a valid URL"; should be "path '/b' not found
in revision REV"?

  * "svn mkdir ^/ a" -> "Illegal repository URL 'a'"; should say "can't
mix URL and local targets"?

  * "svn mkdir a ^/" -> "Can't create directory
'https:/svn.apache.org/repos/asf'"; should not print the URL as if it's
a local path.

  * "svn mv ^/ ^/" -> "Cannot move path
'https:/svn.apache.org/repos/asf' into itself"; should not print the URL
as if it's a local path.

  * "svn update ^/a" -> "Skipped
'https://svn.apache.org/repos/asf/a' ..."; should fail: "'^/a' is not a
WC path"?

- Julian
Received on 2010-11-30 12:51:44 CET

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