Re: path canonicalization problem (issue #559)

On Mon, Jan 21, 2002 at 05:54:28PM -0500, Garrett Rooney wrote:
> On Mon, Jan 21, 2002 at 10:44:01PM +0000, Philip Martin wrote:
> > Garrett Rooney <rooneg@electricjellyfish.net> writes:
> >
>
> true...
>
> > > + strcpy (tmp, tmp + 1);

Sorry to respond to a response to a response, but I missed this in the
original commit:

strcpy is not guaranteed to work on overlapping regions. It is likely
to work when dest < src and extremely unlikely to if dst > src.

memmove() would probably be more appropriate in this case.

--ben

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:36:57 2006