Opened 5 years ago

Closed 5 years ago

#9959 closed bug (fixed)

removeDirectoryRecursive should not follow symlinks

Reported by: ezyang Owned by:
Priority: high Milestone:
Component: libraries/directory Version: 7.11
Keywords: Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Test Case:
Blocked By: Blocking:
Related Tickets: Differential Rev(s):
Wiki Page:


It's just, like, a really bad idea. And almost never what you want. (Also, it differs from the behavior of rm -rf). Can we just change the behavior?

Change History (3)

comment:1 Changed 5 years ago by rwbarton

Priority: normalhigh

comment:2 Changed 5 years ago by Rufflewind

Turns out the documentation was misleading, though in some odd cases it does traverse symlinked directories.

The former documentation error has been fixed in db8800. The latter bug should be fixed in this pull request.

comment:3 Changed 5 years ago by Rufflewind

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.