summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPádraig Brady <P@draigBrady.com>2014-11-22 03:41:55 +0000
committerPádraig Brady <P@draigBrady.com>2014-11-22 03:46:34 +0000
commita78819af2873ea7b104366bf72af7c3e5b30782b (patch)
tree78d69e4f3306ce613e62c519bbeb74351e49cd05
parent57c8196ef6848962145a70dcc465c239a9998eea (diff)
downloadcoreutils-a78819af2873ea7b104366bf72af7c3e5b30782b.tar.xz
tests: add a case verifying mv on case insensitive file systems
* NEWS: Update the recent entry to also mention the avoidance of incorrectly unlinking a multi-hardlinked "source" file when presented with source and dest that only differ in case. * src/copy.c (same_file_ok): Mention the case issue with same_name(). * tests/mv/hardlink-case.sh: Test the issue on HFS+. * tests/local.mk: Reference the new test case. * tests/mv/vfat: Remove an old related but unused test case.
-rw-r--r--NEWS9
-rw-r--r--src/copy.c1
-rw-r--r--tests/local.mk1
-rwxr-xr-xtests/mv/hardlink-case.sh37
-rw-r--r--tests/mv/vfat64
5 files changed, 44 insertions, 68 deletions
diff --git a/NEWS b/NEWS
index 6a84c4892..5d3bc58bd 100644
--- a/NEWS
+++ b/NEWS
@@ -32,10 +32,11 @@ GNU coreutils NEWS -*- outline -*-
** Changes in behavior
- mv no longer supports moving a file to a hardlink, as currently that
- is only supported through unlinking the source file. That's racy
- in the presence of multiple mv instances, which could result in both
- hardlinks being deleted. This feature was added in coreutils-5.0.1.
+ mv no longer supports moving a file to a hardlink, instead issuing an error.
+ The implementation was susceptible to races in the presence of multiple mv
+ instances, which could result in both hardlinks being deleted. Also on case
+ insensitive file systems like HFS, mv would just remove a hardlinked 'file'
+ if called like `mv file File`. The feature was added in coreutils-5.0.1.
** Improvements
diff --git a/src/copy.c b/src/copy.c
index 01cee3214..f316f0522 100644
--- a/src/copy.c
+++ b/src/copy.c
@@ -1526,6 +1526,7 @@ same_file_ok (char const *src_name, struct stat const *src_sb,
return true;
}
+ /* FIXME: What about case insensitive file systems ? */
return ! same_name (src_name, dst_name);
}
diff --git a/tests/local.mk b/tests/local.mk
index 22e8b866e..653c984d1 100644
--- a/tests/local.mk
+++ b/tests/local.mk
@@ -127,6 +127,7 @@ all_root_tests = \
tests/misc/truncate-owned-by-other.sh \
tests/mkdir/writable-under-readonly.sh \
tests/mkdir/smack-root.sh \
+ tests/mv/hardlink-case.sh \
tests/mv/sticky-to-xpart.sh \
tests/rm/fail-2eperm.sh \
tests/rm/no-give-up.sh \
diff --git a/tests/mv/hardlink-case.sh b/tests/mv/hardlink-case.sh
new file mode 100755
index 000000000..5ab8c9543
--- /dev/null
+++ b/tests/mv/hardlink-case.sh
@@ -0,0 +1,37 @@
+#!/bin/sh
+# Ensure multi-hardlinked files are not lost on case insensitive file systems
+
+# Copyright (C) 2014 Free Software Foundation, Inc.
+
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+
+# You should have received a copy of the GNU General Public License
+# along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+. "${srcdir=.}/tests/init.sh"; path_prepend_ ./src
+print_ver_ mv
+require_root_
+
+cwd=$(pwd)
+cleanup_() { cd /; umount "$cwd/mnt"; }
+
+truncate -s100M hfs.img || framework_failure_
+mkfs -t hfsplus hfs.img || skip_ 'failed to create hfs file system'
+mkdir mnt || framework_failure_
+mount hfs.img mnt || skip_ 'failed to mount hfs file system'
+
+cd mnt
+touch foo
+ln foo whatever
+mv foo Foo && fail=1
+test -r foo || fail=1
+
+Exit $fail
diff --git a/tests/mv/vfat b/tests/mv/vfat
deleted file mode 100644
index cd2f8b4d0..000000000
--- a/tests/mv/vfat
+++ /dev/null
@@ -1,64 +0,0 @@
-#!/bin/sh
-# This is just for the record.
-# This test is not run.
-
-# Copyright (C) 2003-2014 Free Software Foundation, Inc.
-
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-# GNU General Public License for more details.
-
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-
-exit 0
-
-cat <<\EOF
-
-Prior to 5.0.91, ...
-
-The problem:
- On a VFAT file system with coreutils-5.0.90, 'mv FOO foo' removes the
- sole copy of the file named by both the source and destination arguments.
-
-Demonstrate the problem, as root:
-
-cd /tmp \
- && dd if=/dev/zero of=1 bs=8192 count=50 \
- && mkdir mnt && mkfs -t vfat 1 \
- && mount -oloop 1 mnt && cd mnt \
- && printf something important > foo \
- && mv foo FOO
-test -f FOO && echo PASS-1 || echo FAIL-1
-ln foo bar
-mv foo FOO
-test -f FOO && echo PASS-2 || echo FAIL-2
-
-And in case you actually do the above, you can do this to clean up:
-
- cd /tmp && umount /tmp/mnt && rm -r 1 mnt
-
-Hey! Can't create hard links on vfat.
-The above 'ln' evokes an 'operation not permitted' failure.
-
-This demonstrates the same thing with file system type 'umsdos'
-No hard links:
-
-cd /tmp \
- && dd if=/dev/zero of=1 bs=8192 count=50 \
- && mkdir mnt && mkfs -t msdos 1 \
- && mount -t umsdos -oloop 1 mnt && cd mnt \
- && printf something important > foo \
- && mv foo FOO
-test -f FOO && echo PASS-1 || echo FAIL-1
-ln foo bar
-mv foo FOO
-test -f FOO && echo PASS-2 || echo FAIL-2
-
-EOF