diff options
| author | Jim Meyering <meyering@redhat.com> | 2007-12-09 00:56:02 +0100 |
|---|---|---|
| committer | Jim Meyering <meyering@redhat.com> | 2007-12-09 00:57:00 +0100 |
| commit | de5b055bbf6c77246826f1276c2a5b9c7264b2bf (patch) | |
| tree | c01060293b726d419c90875dbb3ff3c65eb7760c /src | |
| parent | f16499e4d5cc4bd50067a7d8acc295ac5446833f (diff) | |
| download | coreutils-de5b055bbf6c77246826f1276c2a5b9c7264b2bf.tar.xz | |
"rm" as root would fail to unlink a non-directory on OS X 10.4.x
* src/remove.c (remove_entry) ["can unlink directories"]: Fix a
mistakenly reversed condition.
* NEWS: Mention this bug fix.
Reported by Pieter Bowman.
Diffstat (limited to 'src')
| -rw-r--r-- | src/remove.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/remove.c b/src/remove.c index 11e2ccb0e..de8f5ffc9 100644 --- a/src/remove.c +++ b/src/remove.c @@ -1150,7 +1150,7 @@ remove_entry (int fd_cwd, Dirstack_state const *ds, char const *filename, dirent_type = DT_DIR; } - if (dirent_type == DT_DIR) + if (dirent_type != DT_DIR) { /* At this point, barring race conditions, FILENAME is known to be a non-directory, so it's ok to try to unlink it. */ |