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 /collectstats.pl | |
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 'collectstats.pl')
-rwxr-xr-x | collectstats.pl | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/collectstats.pl b/collectstats.pl index 3473c9e71..339e428bc 100755 --- a/collectstats.pl +++ b/collectstats.pl @@ -309,7 +309,7 @@ sub regenerate_stats { return; } - if (open DATA, ">$file") { + if (open DATA, ">", $file) { my $fields = join('|', ('DATE', @statuses, @resolutions)); my $product_name = $product->name; print DATA <<FIN; |