[PATCH] svn_io_remove_dir mishandles links

svn_io_remove_dir fails to remove directory trees containing symbolic links.

There seem to be two problems: first anything other than dirs and
regular files were ignored (there was already a comment to this
effect). Second, it never checked for symbolic links. A symbolic
link to a directory would be walked like a regular directory entry
(possibly causing things outside the tree to be deleted).

There is no test case, because I can't find anything already testing
svn_io_remove_dir.

Oh, svn_walk_dir has similar problems (that was my first attempt at
working around the problems in svn_io_remove_dir). It should be
getting link info and handling them. I haven't worked on a patch for
this yet.

-Dan Christian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org