Skip to content

[3.5] bpo-39503: CVE-2020-8492: Fix AbstractBasicAuthHandler#19301

Closed
tapakund wants to merge 1 commit intopython:3.5from
tapakund:3.5-bpo-39503
Closed

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

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

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

https://bugs.python.org/issue39503

@tapakund tapakund changed the title bpo-39503: CVE-2020-8492: Fix AbstractBasicAuthHandler [3.5] bpo-39503: CVE-2020-8492: Fix AbstractBasicAuthHandler 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>

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 rework. Will reopen later.

@tapakund tapakund 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.

3 participants