Add a test to exercise a readlink bug.

* tests/misc/readlink-fp-loop: New file.  Test for the readlink bug
fixed through today's change to Gnulib's canonicalize module.
* tests/misc/Makefile.am (TESTS): Add readlink-fp-loop.
Bug report and a test case from mpb.mail@gmail.com.
This commit is contained in:
Jim Meyering 2007-09-24 09:53:57 +02:00
parent 23d293cbc2
commit beaa39a649
3 changed files with 78 additions and 0 deletions

View File

@ -1,3 +1,11 @@
2007-09-27 Jim Meyering <jim@meyering.net>
Add a test to exercise a readlink bug.
* tests/misc/readlink-fp-loop: New file. Test for the readlink bug
fixed through today's change to Gnulib's canonicalize module.
* tests/misc/Makefile.am (TESTS): Add readlink-fp-loop.
Bug report and a test case from mpb.mail@gmail.com.
2007-09-25 Pádraig Brady <P@draigBrady.com>
* doc/coreutils.texi (date invocation):

View File

@ -86,6 +86,7 @@ TESTS = \
printf \
printf-hex \
pwd-long \
readlink-fp-loop \
runcon-no-reorder \
seq \
sha1sum \

69
tests/misc/readlink-fp-loop Executable file
View File

@ -0,0 +1,69 @@
#!/bin/sh
# readlink from 6.9 would fail with a false-positive symlink loop error
# Copyright (C) 2007 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/>.
if test "$VERBOSE" = yes; then
set -x
readlink --version
fi
. $srcdir/../test-lib.sh
cwd=$("$abs_top_builddir/src/pwd")
# To trigger this bug, we have to construct a name/situation during
# the resolution of which the code dereferences the same symlink (S)
# two different times with no actual loop. In addition, arrange
# so that the second and fourth calls to readlink operate on S.
ln -s s p || framework_failure
ln -s d s || framework_failure
mkdir d || framework_failure
echo 2 > d/2 || framework_failure
ln -s ../s/2 d/1 || framework_failure
fail=0
# With coreutils-6.9, this would fail with ELOOP.
readlink -v -e p/1 > out || fail=1
# readlink -e d/2 > exp || fail=1
echo "$cwd/d/2" > exp || fail=1
compare out exp || fail=1
# Construct a real loop and make sure readlink still detects it.
ln -sf ../s/1 d/2 || framework_failure
readlink -v -e p/1 2> out && fail=1
echo readlink: p/1: Too many levels of symbolic links > exp || framework_failure
compare out exp || fail=1
# Exercise the hash table code.
ln -nsf ../s/3 d/2 || framework_failure
ln -nsf ../p/4 d/3 || framework_failure
ln -nsf ../p/5 d/4 || framework_failure
ln -nsf ../p/6 d/5 || framework_failure
ln -nsf ../p/7 d/6 || framework_failure
ln -nsf ../p/8 d/7 || framework_failure
echo x > p/1 || framework_failure
readlink -v -e p/1 > out || fail=1
echo "$cwd/d/8" > exp || fail=1
compare out exp || fail=1
# A trivial loop
ln -s loop loop
readlink -v -e loop 2> out && fail=1
echo readlink: loop: Too many levels of symbolic links > exp || framework_failure
compare out exp || fail=1
(exit $fail); exit $fail