From 2b8f033bcbd5581ff7b27b396f308b77058c09dc Mon Sep 17 00:00:00 2001 From: Aleksander Machniak <alec@alec.pl> Date: Sun, 23 Feb 2014 08:20:43 -0500 Subject: [PATCH] Improvement in handling invalid email address strings --- tests/Framework/Mime.php | 3 +++ program/lib/Roundcube/rcube_mime.php | 4 ++++ 2 files changed, 7 insertions(+), 0 deletions(-) diff --git a/program/lib/Roundcube/rcube_mime.php b/program/lib/Roundcube/rcube_mime.php index 55b70f6..091b2fa 100644 --- a/program/lib/Roundcube/rcube_mime.php +++ b/program/lib/Roundcube/rcube_mime.php @@ -366,6 +366,9 @@ $address = 'MAILER-DAEMON'; $name = substr($val, 0, -strlen($m[1])); } + else if (preg_match('/('.$email_rx.')/', $val, $m)) { + $name = $m[1]; + } else { $name = $val; } @@ -387,6 +390,7 @@ if (!$address && $name) { $address = $name; + $name = ''; } if ($address) { diff --git a/tests/Framework/Mime.php b/tests/Framework/Mime.php index d767905..d47eba8 100644 --- a/tests/Framework/Mime.php +++ b/tests/Framework/Mime.php @@ -42,6 +42,8 @@ // invalid (#1489092) 22 => '"John Doe @ SomeBusinessName" <MAILER-DAEMON>', 23 => '=?UTF-8?B?IlRlc3QsVGVzdCI=?= <test@domain.tld>', + // invalid, but we do our best to parse correctly + 24 => '"email@test.com" <>', ); $results = array( @@ -70,6 +72,7 @@ // invalid (#1489092) 22 => array(1, 'John Doe @ SomeBusinessName', 'MAILER-DAEMON'), 23 => array(1, 'Test,Test', 'test@domain.tld'), + 24 => array(1, '', 'email@test.com'), ); foreach ($headers as $idx => $header) { -- Gitblit v1.9.1