aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorjustdave%syndicomm.com <>2001-08-18 08:49:02 +0000
committerjustdave%syndicomm.com <>2001-08-18 08:49:02 +0000
commita6f587a48e315d44edee876e9a21d9e5e51e51ee (patch)
tree63beaa2a1d34487e03e2348af8a4c730830adc97
parentc409d06992b1af4f3a8ff10c7344046a91e2d5c1 (diff)
downloadbugs-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-xprocess_bug.cgi10
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;
}