Philip Rowlands <[EMAIL PROTECTED]> wrote:
> On Tue, 22 Jul 2008, Ralf Wildenhues wrote:
>
>> * tests/CuTmpdir.pm (chmod_tree): Do not run chmod on undefined
>> argument, can happen when the build path contains spaces.
>
> That sounds wrong - there's no magic to unusual characters in
> filenames other than avoiding passing them unquoted through an
> IFS-splitting shell.
Hi Phil,
Ralf's change is correct.
Since the script with taint checking enabled, and mkdir requires that
its argument not be tainted, I opted to skip the test when the absolute
temporary directory name contains any unusual characters.
>> sub chmod_tree
>> {
>> - if (chdir $dir)
>> + if (defined $dir && chdir $dir)
>> {
>
> Surely skipping the test is going to give a misleading impression to
> the tester? Tracing back up, why is $dir not defined?
Because the script exits (via the skip/die) before $dir is defined,
and the on-exit (END) handler calls chmod_tree which then uses $dir.
_______________________________________________
Bug-coreutils mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/bug-coreutils
