diff options
| author | Jim Meyering <jim@meyering.net> | 2006-07-05 09:08:42 +0000 |
|---|---|---|
| committer | Jim Meyering <jim@meyering.net> | 2006-07-05 09:08:42 +0000 |
| commit | 36a288e41a6534c816090dce1a6ed139a60e87b7 (patch) | |
| tree | 3793eee5eef96a014ea95df1fe9ca3a8b0e291e8 /src | |
| parent | 7669f3998d2adb0618548dcf95b5ef01a51fac9e (diff) | |
| download | coreutils-36a288e41a6534c816090dce1a6ed139a60e87b7.tar.xz | |
* src/copy.c (copy_internal): Don't work around old NFS clients like
SunOS-4.1.4 and Irix 5.3 that set errno to values like EIO and
ENOTEMPTY upon failed rename. Otherwise, we risk misinterpreting
a banal failure as a recursive move-into-self failure.
Reported by Florent Bayle in <http://bugs.debian.org/376749>.
Diffstat (limited to 'src')
| -rw-r--r-- | src/copy.c | 13 |
1 files changed, 1 insertions, 12 deletions
diff --git a/src/copy.c b/src/copy.c index a6ea3e6b6..c958eb687 100644 --- a/src/copy.c +++ b/src/copy.c @@ -1385,18 +1385,7 @@ copy_internal (char const *src_name, char const *dst_name, /* This happens when attempting to rename a directory to a subdirectory of itself. */ - if (errno == EINVAL - - /* When src_name is on an NFS file system, some types of - clients, e.g., SunOS4.1.4 and IRIX-5.3, set errno to EIO - instead. Testing for this here risks misinterpreting a real - I/O error as an attempt to move a directory into itself, so - FIXME: consider not doing this. */ - || errno == EIO - - /* And with SunOS-4.1.4 client and OpenBSD-2.3 server, - we get ENOTEMPTY. */ - || errno == ENOTEMPTY) + if (errno == EINVAL) { /* FIXME: this is a little fragile in that it relies on rename(2) failing with a specific errno value. Expect problems on |