diff options
author | Jim Meyering <meyering@redhat.com> | 2009-11-19 10:13:22 +0100 |
---|---|---|
committer | Jim Meyering <meyering@redhat.com> | 2009-11-19 11:12:29 +0100 |
commit | 30eb606ed59da1c3c511f961abfe75a2749a3228 (patch) | |
tree | 0e96dfa7c2f41d2c97174dee607a0522f4df3933 /tests/rm/fail-eperm | |
parent | f18eef32848e2a0d1cc4159799ddc6d652f9ba75 (diff) | |
download | coreutils-30eb606ed59da1c3c511f961abfe75a2749a3228.tar.xz |
tests: avoid spurious failures due to insecure directory in PATH
These tests perform no PATH search, and used to simply delete PATH from
the environment. However, that is not portable, as seen on Cygwin,
where cygwin.dll must be resolvable via PATH when starting a sub-shell.
With commit 0cc04241, we took the alternate approach of untainting the
incoming $ENV{PATH}, but that fails when it contains an other-writable
directory. Instead, now we hard code it to '/bin:/usr/bin'.
* tests/misc/pwd-long: Hard code $ENV{PATH} to a safe value.
* tests/rm/fail-eperm: Likewise.
Reported by Gilles Espinasse, Andreas Schwab, and Bauke Jan Douma.
Diffstat (limited to 'tests/rm/fail-eperm')
-rwxr-xr-x | tests/rm/fail-eperm | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/tests/rm/fail-eperm b/tests/rm/fail-eperm index 36192c2e5..16811c2ff 100755 --- a/tests/rm/fail-eperm +++ b/tests/rm/fail-eperm @@ -34,11 +34,11 @@ $ENV{LC_ALL} = 'C'; # Set up a safe, well-known environment delete @ENV{qw(BASH_ENV CDPATH ENV)}; $ENV{IFS} = ''; -# PATH is tricky - we can't just clear it, or cygwin will fail. But we -# can't use it as-is, or taint checking in `` will stop us. For this -# script, it is enough to scrub the incoming $PATH first. -$ENV{'PATH'} =~ /(.*)/; -$ENV{'PATH'} = "$1"; + +# Taint checking requires a sanitized $PATH. This script performs no $PATH +# search, so on most Unix-based systems, it is fine simply to clear $ENV{PATH}. +# However, on Cygwin, it's used to find cygwin.dll, so set it. +$ENV{PATH} = '/bin:/usr/bin'; my @dir_list = qw(/tmp /var/tmp /usr/tmp); my $rm = "$ENV{abs_top_builddir}/src/rm"; |