diff options
| author | justdave%bugzilla.org <> | 2004-05-26 16:21:43 +0200 |
|---|---|---|
| committer | justdave%bugzilla.org <> | 2004-05-26 16:21:43 +0200 |
| commit | c5718f3890535bb27a2db35c1f08b22b05e89ac4 (patch) | |
| tree | 27e0380167395dd82c26aea68bdac7a72f90db91 /Bugzilla/BugMail.pm | |
| parent | 807c816305af7ff71b4d243d747e296e6b15fd3f (diff) | |
| download | bugzilla-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
Diffstat (limited to 'Bugzilla/BugMail.pm')
| -rw-r--r-- | Bugzilla/BugMail.pm | 3 |
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; |