From 576dff038b5b5dfb9bd28c0f76edab43766b1723 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Elan=20Ruusam=C3=A4e?= <glen@pld-linux.org>
Date: Mon, 9 Mar 2009 11:07:31 +0000
Subject: [PATCH] - no brainos in regexps

Changed files:
    file-fetcher.pl -> 1.48
    request-handler.pl -> 1.16
---
 file-fetcher.pl    | 30 +++++++++++++++---------------
 request-handler.pl |  8 ++++----
 2 files changed, 19 insertions(+), 19 deletions(-)

diff --git a/file-fetcher.pl b/file-fetcher.pl
index e5b08fc..6f3c8b8 100644
--- a/file-fetcher.pl
+++ b/file-fetcher.pl
@@ -40,21 +40,21 @@ sub find_file_in_spool()
 }
 
 # read file from spool, and try unlink it. if cannot unlink -- exit
-# sets $requester (email), $problems, @md5 (arrays of md5's) 
+# sets $requester (email), $problems, @md5 (arrays of md5's)
 # and @url (map from md5 to urls)
 sub read_spool_file()
 {
   open(F, "< $file") || exit 0;
   $requester = <F>;
   chomp $requester;
-  $requester =~ /^[a-zA-Z_\-0-9\@\.]+$/ 
+  $requester =~ /^[a-zA-Z_0-9@.-]+$/
 	or die "$file: evil requester: $requester";
   $req_login = "";
   $requester =~ /^([^@]+)\@/ and $req_login = $1;
-  $req_login =~ /^[a-z0-9A-Z_\.]+$/ or die "$file: evil requester $requester";
+  $req_login =~ /^[a-z0-9A-Z_.]+$/ or die "$file: evil requester $requester";
   my $flags = <F>;
   $force_reply++ if ($flags =~ /force-reply/);
-	
+
   while (<F>) {
     if (/^ERROR/) {
       s/^ERROR: //;
@@ -120,7 +120,7 @@ sub move_file($$$)
   my ($md5, $url, $local_copy) = @_;
 
   my $bn = basename($url);
-  
+
   if ($local_copy ne "$tmp_dir/$md5/$bn") {
     if (system("mv -f \"$local_copy\" \"$tmp_dir/$md5/$bn\"")) {
       $problems .= "FATAL: cannot move $local_copy to $tmp_dir\n";
@@ -128,7 +128,7 @@ sub move_file($$$)
     }
     $local_copy = "$tmp_dir/$md5/$bn";
   }
-  
+
   if (open(D, "> $tmp_dir/$md5/$bn.desc")) {
     print D "URL: $url\n";
     print D "Login: $req_login\n";
@@ -143,7 +143,7 @@ sub move_file($$$)
   my $dir = by_md5($md5, $url);
   $dir =~ s|/[^/]+/[^/]+$||;
   if (copy_to_df("$tmp_dir/$md5/", $dir) == 0) {
-    $normal_out .= 
+    $normal_out .=
       "STORED: $url\n" .
       "\t$md5  " . basename($url) . "\n" .
       "\tSize: " .  (-s $local_copy) . " bytes\n";
@@ -154,9 +154,9 @@ sub move_file($$$)
 sub make_src_symlink($$)
 {
   my ($md5, $url) = @_;
-  
+
   return unless ($url =~ /^no-url/);
-  
+
   my $b = basename($url);
   if (open(S, "> $tmp_dir/$b.link")) {
     if ($url =~ /^no-url-copy/) {
@@ -193,7 +193,7 @@ sub md5($)
 sub handle_no_url($$)
 {
   my ($md5, $url) = @_;
-  
+
   unless ($url =~ m#://([^/]+)#) {
     $problems .= "$url: corrupted! (no-url)";
     return;
@@ -227,7 +227,7 @@ sub fetch_file($$)
   push @files, $bn;
 
   if (got_on_distfiles($md5, $url)) {
-    $normal_out .= 
+    $normal_out .=
     	"ALREADY GOT: $url\n" .
     	"\t$md5  " . basename($url) . "\n";
     make_src_symlink($md5, $url);
@@ -235,12 +235,12 @@ sub fetch_file($$)
   }
 
   mkdir("$tmp_dir/$md5");
-  
+
   if ($url =~ /^no-url/) {
     handle_no_url($md5, $url);
     return;
   }
-  
+
   open(W, "$cmd 2>&1 |");
   while (<W>) {
     $all_out .= $_;
@@ -313,8 +313,8 @@ sub send_email()
 
   splice(@files, 10, @files - 10, "...")
     if (@files > 10);
-  
-  print EMAIL 
+
+  print EMAIL
 "From: $req_login <$requester>
 To: $commits_list
 Cc: $requester
diff --git a/request-handler.pl b/request-handler.pl
index d9a2a88..7606586 100644
--- a/request-handler.pl
+++ b/request-handler.pl
@@ -49,10 +49,10 @@ $id = rand if (!defined $id or $id eq "");
 
 while (<STDIN>) {
   chomp;
-  /^X-Spec: ([\+a-zA-Z0-9_\-\.]+\.spec)/i and $spec = $1;
-  /^X-Branch: ([a-zA-Z0-9_\-]+)/i and $branch = $1;
-  /^X-Login: ([a-zA-Z0-9_\.]+)/i and $from = $1;
-  /^X-Flags: ([a-zA-Z0-9\-_ ]+)/i and $flags = $1;
+  /^X-Spec: ([a-z0-9_.+-]+\.spec)/i and $spec = $1;
+  /^X-Branch: ([a-z0-9_-]+)/i and $branch = $1;
+  /^X-Login: ([a-z0-9_.]+)/i and $from = $1;
+  /^X-Flags: ([a-z0-9_ -]+)/i and $flags = $1;
 }
 
 die "ill-formed request" if (!defined $from);
-- 
2.44.0