Skip to content

Conversation

@maxking
Copy link
Contributor

@maxkingmaxking commented Aug 17, 2019

Before:

 >>> email.message_from_string('From: [email protected]@important.com', policy=email.policy.default)['from'].addresses (Address(display_name='', username='a', domain='malicious.org'),) >>> parseaddr('[email protected]@important.com') ('', '[email protected]') After: >>> email.message_from_string('From: [email protected]@important.com', policy=email.policy.default)['from'].addresses (Address(display_name='', username='', domain=''),) >>> parseaddr('[email protected]@important.com') ('', 'a@') 

https://bugs.python.org/issue34155
(cherry picked from commit 8cb65d1)

Co-authored-by: jpic [email protected]

https://bugs.python.org/issue34155

Before: >>> email.message_from_string('From: [email protected]@important.com', policy=email.policy.default)['from'].addresses (Address(display_name='', username='a', domain='malicious.org'),) >>> parseaddr('[email protected]@important.com') ('', '[email protected]') After: >>> email.message_from_string('From: [email protected]@important.com', policy=email.policy.default)['from'].addresses (Address(display_name='', username='', domain=''),) >>> parseaddr('[email protected]@important.com') ('', 'a@') https://bugs.python.org/issue34155 (cherry picked from commit 8cb65d1) Co-authored-by: jpic <[email protected]>
@alex
Copy link
Member

alex commented Aug 17, 2019

This needs a review from the 3.5 RM if this is going to be backported.

Copy link
Member

@vstinnervstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM: same change than 8cb65d1 already merged into master.

@vstinner
Copy link
Member

@larryhastings: Would you mind to merge this security fix?

@larryhastingslarryhastings merged commit 063eba2 into python:3.5Sep 7, 2019
@bedevere-bot
Copy link

@larryhastings: Please replace # with GH- in the commit message next time. Thanks!

@larryhastings
Copy link
Contributor

Thanks for the backport and the 3.5 love!

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-securityA security issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@maxking@alex@vstinner@bedevere-bot@larryhastings@the-knights-who-say-ni@jpic