Re: svn commit: r33617 - trunk/subversion/libsvn_subr

Lieven Govaerts wrote:
> Stefan,
>
> I'm sorry I'm repeating myself, but this change would be so much better
> if you added a testcase demonstrating the issue.
>
> So, can you give an example of a url/path demonstrating this memory
> corruption?

On Windows, any url will do. The parts that are copied there don't have
a slash at the end. Just pass an url to svn_path_canonicalize().

Try for example:
http://testserver
(this will corrupt the memory)

I discovered this because the following url lead to a segfault:
http://testing%20special%20chars%20%c3%a4%c3%b6%c3%bc
with the TSVN static tests.

I'm sorry for the missing test case, but I haven't got around to set up
the svn test suite yet.

Stefan

-- 
       ___
  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.net