revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-23 14:00:56 +01:00
Code Issues Releases Wiki Activity

fix error where blank email addresses were sneaking in the stack

Browse source 
Summary: array_filter to the rescue

Test Plan: mostly lots of reasoning (as opposed to making a fake user with a blank email address to reproduce)

Reviewers: epriestley, chad

Reviewed By: chad

CC: aran, Korvin

Maniphest Tasks: T2052

Differential Revision: https://secure.phabricator.com/D3927
This commit is contained in:
Bob Trahan 2012-11-08 12:21:50 -08:00
parent 378702e7b4
commit bd1bc6d71a
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/metamta/storage/PhabricatorMetaMTAReceivedMail.php
View file

@ -356,7 +356,7 @@ final class PhabricatorMetaMTAReceivedMail extends PhabricatorMetaMTADAO {
foreach (explode(',', $addresses) as $address) { foreach (explode(',', $addresses) as $address) {
$raw_addresses[] = $this->getRawEmailAddress($address); $raw_addresses[] = $this->getRawEmailAddress($address);
} }
return $raw_addresses; return array_filter($raw_addresses);
} }
private function lookupPublicUser() { private function lookupPublicUser() {