diff options
| author | Gervase Markham <gerv@mozilla.org> | 2015-01-21 20:49:57 +0100 |
|---|---|---|
| committer | David Lawrence <dkl@mozilla.com> | 2015-01-21 20:49:57 +0100 |
| commit | 4dabf1a9c679f06b3637d3c76e1e05aa83a6d259 (patch) | |
| tree | 93ec377d81b16ad7caccc28c4954048358aa431b /Bugzilla/Install/CPAN.pm | |
| parent | 367d9c2f6efd2cc53b773f0c1cc9e19a8d82c5be (diff) | |
| download | bugzilla-4dabf1a9c679f06b3637d3c76e1e05aa83a6d259.tar.gz bugzilla-4dabf1a9c679f06b3637d3c76e1e05aa83a6d259.tar.xz | |
Bug 1079065: [SECURITY] Always use the 3 arguments form for open() to prevent shell code injection
r=dylan,a=simon
Diffstat (limited to 'Bugzilla/Install/CPAN.pm')
| -rw-r--r-- | Bugzilla/Install/CPAN.pm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Bugzilla/Install/CPAN.pm b/Bugzilla/Install/CPAN.pm index 19f143190..094784e1a 100644 --- a/Bugzilla/Install/CPAN.pm +++ b/Bugzilla/Install/CPAN.pm @@ -196,8 +196,8 @@ sub set_cpan_config { # Calling a senseless autoload that does nothing makes us # automatically load any existing configuration. # We want to avoid the "invalid command" message. - open(my $saveout, ">&STDOUT"); - open(STDOUT, '>/dev/null'); + open(my $saveout, ">&", "STDOUT"); + open(STDOUT, '>', '/dev/null'); eval { CPAN->ignore_this_error_message_from_bugzilla; }; undef $@; close(STDOUT); |