diff options
author | justdave%syndicomm.com <> | 2001-08-18 08:49:02 +0000 |
---|---|---|
committer | justdave%syndicomm.com <> | 2001-08-18 08:49:02 +0000 |
commit | a6f587a48e315d44edee876e9a21d9e5e51e51ee (patch) | |
tree | 63beaa2a1d34487e03e2348af8a4c730830adc97 | |
parent | c409d06992b1af4f3a8ff10c7344046a91e2d5c1 (diff) | |
download | bugs-a6f587a48e315d44edee876e9a21d9e5e51e51ee.tar bugs-a6f587a48e315d44edee876e9a21d9e5e51e51ee.tar.gz bugs-a6f587a48e315d44edee876e9a21d9e5e51e51ee.tar.bz2 bugs-a6f587a48e315d44edee876e9a21d9e5e51e51ee.tar.xz bugs-a6f587a48e315d44edee876e9a21d9e5e51e51ee.zip |
Fix for bug 95857: process_bugs.cgi sees blank address in cc: line (would show up as __UNKNOWN__)
Patch by Myk Melez <myk@mozilla.org>
r= justdave@syndicomm.com
-rwxr-xr-x | process_bug.cgi | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/process_bug.cgi b/process_bug.cgi index 76e52a2df..5bb180c5a 100755 --- a/process_bug.cgi +++ b/process_bug.cgi @@ -582,17 +582,15 @@ if (defined $::FORM{newcc} || defined $::FORM{removecc} || defined $::FORM{massc } if ($cc_add) { - foreach my $person (split(/[ ,]/, $cc_add)) { - # Ignore blanks - next unless $person; + $cc_add =~ s/^[\s,]+//; # Remove leading delimiters. + foreach my $person ( split(/[\s,]+/, $cc_add) ) { my $pid = DBNameToIdAndCheck($person); $cc_add{$pid} = $person; } } if ($cc_remove) { - foreach my $person (split(/[ ,]/, $cc_remove)) { - # Ignore blanks - next unless $person; + $cc_remove =~ s/^[\s,]+//; # Remove leading delimiters. + foreach my $person ( split(/[\s,]+/, $cc_remove) ) { my $pid = DBNameToIdAndCheck($person); $cc_remove{$pid} = $person; } |