| Subject: | Misleading documentation about default temporary directory |
Version 1.03 states that:
The optional second parameter, $tempdir, is only used if you want
the children to send back a reference to some data (see RETRIEVING
DATASTRUCTURES below). If not provided, it is set to
$File::Spec->tmpdir().
The new method will die if the temporary directory does not exist
or it is not a directory, whether you provided this parameter or
the $File::Spec->tmpdir() is used.
Apart of there is no such thing as "$File::Spec->tmpdir()", the File::Spec->tmpdir method returns the path to the system-wide temporary directory, which on non-windows is /tmp and is writeable by all users on the system, and if really used would lead to the security problems noted in #68298.
Attached is a patch to bring the documentation in line with the code.
Thanks,
dam
| Subject: | pod-errors.patch |
Description: fix tempdir defaults documentation and an encoding issue
Author: Damyan Ivanov <dmn@debian.org>
--- a/lib/Parallel/ForkManager.pm
+++ b/lib/Parallel/ForkManager.pm
@@ -1,3 +1,5 @@
+=encoding latin1
+
=head1 NAME
Parallel::ForkManager - A simple parallel processing fork manager
@@ -83,11 +85,10 @@ will be forked. This is intended for deb
The optional second parameter, $tempdir, is only used if you want the
children to send back a reference to some data (see RETRIEVING DATASTRUCTURES
-below). If not provided, it is set to $L<File::Spec>->tmpdir().
+below). If not provided, it is set via a call to L<File::Temp>::tempdir().
The new method will die if the temporary directory does not exist or it is not
-a directory, whether you provided this parameter or the
-$L<File::Spec>->tmpdir() is used.
+a directory.
=item start [ $process_identifier ]