Fix for bug 21253: removing all single-parameter system() calls from Bugzilla

Patch by Dave Miller <justdave@syndicomm.com>
r= tara@tequilarista.org

1 files changed, 2 insertions, 2 deletions

diff --git a/collectstats.pl b/collectstats.pl
index e36a3ca90..4e69ab9b5 100755
--- a/collectstats.pl
+++ b/collectstats.pl
@@ -131,8 +131,8 @@ sub calculate_dupes {
     # Save % count here in a date-named file
     # so we can read it back in to do changed counters
     # First, delete it if it exists, so we don't add to the contents of an old file
-    if (<data/duplicates/dupes$today*>) {
-        system("rm -f data/duplicates/dupes$today*");
+    if (my @files = <data/duplicates/dupes$today*>) {
+        unlink @files;
     }
    
     dbmopen(%count, "data/duplicates/dupes$today", 0644) || die "Can't open DBM dupes file: $!";