summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorjustdave%bugzilla.org <>2004-05-26 16:21:43 +0200
committerjustdave%bugzilla.org <>2004-05-26 16:21:43 +0200
commitc5718f3890535bb27a2db35c1f08b22b05e89ac4 (patch)
tree27e0380167395dd82c26aea68bdac7a72f90db91
parent807c816305af7ff71b4d243d747e296e6b15fd3f (diff)
downloadbugzilla-c5718f3890535bb27a2db35c1f08b22b05e89ac4.tar.gz
bugzilla-c5718f3890535bb27a2db35c1f08b22b05e89ac4.tar.xz
Bug 227172: Deal with race condition between a user being removed from the CC on a bug and changing their email address before the bugmail is sent.
r=jouni, a=justdave
-rw-r--r--Bugzilla/BugMail.pm3
1 files changed, 3 insertions, 0 deletions
diff --git a/Bugzilla/BugMail.pm b/Bugzilla/BugMail.pm
index 5975c859a..40a40dc2b 100644
--- a/Bugzilla/BugMail.pm
+++ b/Bugzilla/BugMail.pm
@@ -705,6 +705,9 @@ sub NewProcessOnePerson ($$$$$$$$$$$$$) {
# This routine should really get passed a userid
# This rederives groups as a side effect
my $user = Bugzilla::User->new_from_login($person);
+ if (!$user) { # person doesn't exist, probably changed email
+ return;
+ }
my $userid = $user->id;
$seen{$person} = 1;