Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3.5] bpo-37461: Fix infinite loop in parsing of specially crafted email headers (GH-14794) #15446

Merged
merged 3 commits into from Sep 7, 2019

Conversation

@maxking
Copy link
Contributor

maxking commented Aug 24, 2019

  • bpo-37461: Fix infinite loop in parsing of specially crafted email headers.

Some crafted email header would cause the get_parameter method to run in an
infinite loop causing a DoS attack surface when parsing those headers. This
patch fixes that by making sure the DQUOTE character is handled to prevent
going into an infinite loop.
(cherry picked from commit a4a994b)

Co-authored-by: Abhilash Raj maxking@users.noreply.github.com

https://bugs.python.org/issue37461

…ail headers (GH-14794)

* bpo-37461: Fix infinite loop in parsing of specially crafted email headers.

Some crafted email header would cause the get_parameter method to run in an
infinite loop causing a DoS attack surface when parsing those headers. This
patch fixes that by making sure the DQUOTE character is handled to prevent
going into an infinite loop.
(cherry picked from commit a4a994b)

Co-authored-by: Abhilash Raj <maxking@users.noreply.github.com>
@maxking maxking requested a review from larryhastings Aug 24, 2019
@maxking maxking changed the title [3.5] bpo-37461: Fix infinite loop in parsing of specially crafted em… [3.5] bpo-37461: Fix infinite loop in parsing of specially crafted email headers (GH-14794) Aug 24, 2019
maxking and others added 2 commits Sep 5, 2019
….rst

Co-Authored-By: Ashwin Ramaswami <aramaswamis@gmail.com>
@larryhastings larryhastings merged commit c28e4a5 into python:3.5 Sep 7, 2019
5 checks passed
5 checks passed
bedevere/issue-number Issue number 37461 found
Details
bedevere/maintenance-branch-pr Valid maintenance branch PR title.
bedevere/news News entry found in Misc/NEWS.d
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@bedevere-bot

This comment has been minimized.

Copy link

bedevere-bot commented Sep 7, 2019

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

@larryhastings

This comment has been minimized.

Copy link
Contributor

larryhastings commented Sep 7, 2019

Thanks for the 3.5 backport love!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants
You can’t perform that action at this time.