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

Re: Issue #1901 (double slashes screw things): Patch, and some strategy questions

From: Peter N. Lundblad <peter_at_famlundblad.se>
Date: 2004-07-03 22:51:40 CEST

On Sat, 3 Jul 2004, Josh Pieper wrote:

> Klaus Rennecke wrote:
> > Ben Reser wrote:
> >
> > >[...]
> > >
> > >We need to accept double slashes. See Issue #1779 and my comments on it
> > >in:
> > ><20040311072104.GN13621@occipital.brain.org>
> > >http://www.contactor.se/~dast/svn/archive-2004-03/0558.shtml
> > >
> > >The correct fix is to fix whatever is segfault when it encouters
> > >multiple slashes...
> > >
> >
> > As stated in an earlier followup
> > http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=69409 I am
> > pretty convinced that this issue is actually part of a more general
> > problem. There should be an algorithm floting around on my desktop
> > somewhere, but I don't have the time to port it from Java and check
> > legal at the moment.
> >
> > Sorry if I'm repeating myself, but I think this may turn ugly if you fix
> > the issue in small increments every time someone finds a new angle to
> > this. See <http://rfc.net/rfc2396.html#s5.2.>
>
> Yes, this is a symptom of a larger problem, namely that
> svn_path_canonicalize doesn't return a canonicalized path. Canonical

Hmmm... I should read my mail inorder... Don't forget Ben Reser's post
aobut (see the reference in his mail). Maybe it is better to learn the
other path routines about multiple slashes then? Still, normalizing away .
and .. might be a good idea.

//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Jul 3 22:40:12 2004

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.