From: justdave%bugzilla.org <>
Date: Wed, 26 May 2004 14:21:43 +0000 (+0000)
Subject: Bug 227172: Deal with race condition between a user being removed from the CC on... 
X-Git-Tag: bugzilla-2.18rc1~53
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c5718f3890535bb27a2db35c1f08b22b05e89ac4;p=thirdparty%2Fbugzilla.git

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
---

diff --git a/Bugzilla/BugMail.pm b/Bugzilla/BugMail.pm
index 5975c859ac..40a40dc2be 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;