diff options
| author | Jim Meyering <jim@meyering.net> | 2006-09-29 09:54:24 +0000 |
|---|---|---|
| committer | Jim Meyering <jim@meyering.net> | 2006-09-29 09:54:24 +0000 |
| commit | a4d470409f5590b95d33ab67e5866ab71e4f0e27 (patch) | |
| tree | 063eddc8b35a52abf51ae0d9e99483029015d5cd /src | |
| parent | 344c4467768879d21f90b27bad10dc98c764d349 (diff) | |
| download | coreutils-a4d470409f5590b95d33ab67e5866ab71e4f0e27.tar.xz | |
Work around a readdir bug in Darwin 7.9.0 (MacOS X 10.3.9) on HFS+
and NFS, whereby rm would not remove all files in a directory.
* src/remove.c (CONSECUTIVE_READDIR_UNLINK_THRESHOLD): Reduce to 10.
(NEED_REWIND): New macro, so that we incur the cost of the work-around
rewinddir only on afflicted systems.
* NEWS: Clarify and correct.
* tests/rm/readdir-bug: New file. Test for the above fix.
* tests/rm/Makefile.am (TESTS): Add it.
Prompted by testing and analysis from Bruno Haible:
http://lists.gnu.org/archive/html/bug-coreutils/2006-09/msg00326.html
Diffstat (limited to 'src')
| -rw-r--r-- | src/remove.c | 30 |
1 files changed, 18 insertions, 12 deletions
diff --git a/src/remove.c b/src/remove.c index 24426cff9..1ea7773f7 100644 --- a/src/remove.c +++ b/src/remove.c @@ -50,20 +50,27 @@ #define obstack_chunk_free free /* This is the maximum number of consecutive readdir/unlink calls that - can be made (with no intervening rewinddir or closedir/opendir) - before triggering a bug that makes readdir return NULL even though - some directory entries have not been processed. The bug afflicts - SunOS's readdir when applied to ufs file systems and Darwin 6.5's - (and OSX v.10.3.8's) HFS+. This maximum is conservative in that - demonstrating the problem seems to require a directory containing - at least 254 deletable entries (which doesn't count . and ..). - However, in 2006, we see that Darwin 8.6.1, using NFS has an even - lower limit: 188. */ + can be made (with no intervening rewinddir or closedir/opendir) before + triggering a bug that makes readdir return NULL even though some + directory entries have not been processed. The bug afflicts SunOS's + readdir when applied to ufs file systems and Darwin 6.5's (and OSX + v.10.3.8's) HFS+. This maximum is conservative in that demonstrating + the problem requires a directory containing at least 16 deletable + entries (which doesn't count . and ..). + This problem also affects Darwin 7.9.0 (aka MacOS X 10.3.9) on HFS+ + and NFS-mounted file systems, but not vfat ones. */ enum { - CONSECUTIVE_READDIR_UNLINK_THRESHOLD = 180 + CONSECUTIVE_READDIR_UNLINK_THRESHOLD = 10 }; +#ifdef HAVE_WORKING_READDIR +# define NEED_REWIND(readdir_unlink_count) 0 +#else +# define NEED_REWIND(readdir_unlink_count) \ + (CONSECUTIVE_READDIR_UNLINK_THRESHOLD <= (readdir_unlink_count)) +#endif + enum Ternary { T_UNKNOWN = 2, @@ -1139,8 +1146,7 @@ remove_cwd_entries (DIR **dirp, { /* fall through */ } - else if (CONSECUTIVE_READDIR_UNLINK_THRESHOLD - < n_unlinked_since_opendir_or_last_rewind) + else if (NEED_REWIND (n_unlinked_since_opendir_or_last_rewind)) { /* Call rewinddir if we've called unlink or rmdir so many times (since the opendir or the previous rewinddir) that this |