Skip to content

[2.7] bpo-39503: CVE-2020-8492: Fix AbstractBasicAuthHandler#19302

Closed
tapakund wants to merge 1 commit intopython:2.7from
tapakund:2.7
Closed

[2.7] bpo-39503: CVE-2020-8492: Fix AbstractBasicAuthHandler#19302
tapakund wants to merge 1 commit intopython:2.7from
tapakund:2.7

Conversation

@tapakund
Copy link

@tapakund tapakund commented Apr 2, 2020

The AbstractBasicAuthHandler class of the urllib.request module uses
an inefficient regular expression which can be exploited by an
attacker to cause a denial of service. Fix the regex to prevent the
catastrophic backtracking. Vulnerability reported by Ben Caller
and Matt Schwager.

AbstractBasicAuthHandler of urllib.request now parses all
WWW-Authenticate HTTP headers and accepts multiple challenges per
header: use the realm of the first Basic challenge.

Co-Authored-By: Serhiy Storchaka storchaka@gmail.com
(cherry picked from commit 0b297d4)

Co-authored-by: Victor Stinner vstinner@python.org

Signed-off-by: Tapas Kundu tkundu@vmware.com

https://bugs.python.org/issue39503

The AbstractBasicAuthHandler class of the urllib.request module uses
an inefficient regular expression which can be exploited by an
attacker to cause a denial of service. Fix the regex to prevent the
catastrophic backtracking. Vulnerability reported by Ben Caller
and Matt Schwager.

AbstractBasicAuthHandler of urllib.request now parses all
WWW-Authenticate HTTP headers and accepts multiple challenges per
header: use the realm of the first Basic challenge.

Co-Authored-By: Serhiy Storchaka <storchaka@gmail.com>
(cherry picked from commit 0b297d4)

Co-authored-by: Victor Stinner <vstinner@python.org>

Signed-off-by: Tapas Kundu <tkundu@vmware.com>
@tapakund
Copy link
Author

tapakund commented Apr 2, 2020

Need to rework, will raise fresh.

@tapakund tapakund closed this Apr 2, 2020
@vstinner
Copy link
Member

vstinner commented Apr 2, 2020

Python 2.7 no longer accept security fixes: https://devguide.python.org/#status-of-python-branches

@vstinner vstinner reopened this Apr 2, 2020
@vstinner vstinner closed this Apr 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants