Visitar URL original
[3.8] gh-95778: CVE-2020-10735: Prevent DoS by very large int() by gpshead · Pull Request #96503 · python/cpython · GitHub
Skip to content

[3.8] gh-95778: CVE-2020-10735: Prevent DoS by very large int() #96503

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.

Sign up for GitHub

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

Merged
ambv merged 17 commits into from
Sep 5, 2022
Merged

[3.8] gh-95778: CVE-2020-10735: Prevent DoS by very large int() #96503

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
b518238
Backport to 3.8 of psrt/CVE-2020-10735-3.10backport.
gpshead Aug 19, 2022
504e82f
Fix the versionchanged/addeds to 3.8.14.
gpshead Aug 30, 2022
cae5eba
add the attribution to the NEWS entry.
gpshead Aug 30, 2022
cd54fc3
Backport the Parser/pegen.c change for a good SyntaxError to ast.c.
gpshead Aug 30, 2022
eb68f9c
Add Whats New entry.
gpshead Aug 30, 2022
75bbbbf
Manually add new field to abi file
tiran Sep 1, 2022
14467fc
Move the whatsnew text per review.
gpshead Sep 1, 2022
1e39232
Merge branch 'CVE-2020-10735-3.8backport' of github.com:python/psrt i…
gpshead Sep 1, 2022
70b9aef
Make the doctest actually run & fix it.
gpshead Sep 1, 2022
7eb255f
Fix the docs build.
gpshead Sep 2, 2022
0504ecb
Rename the news file to appease the Bedevere bot.
gpshead Sep 2, 2022
8acc891
hexadecimal spelling =)
gpshead Sep 2, 2022
52f2c26
doc typo: limitation
gpshead Sep 4, 2022
510349b
Misc: Fix a typo in the header comment.
gpshead Sep 4, 2022
ac99726
remove unneeded doc note on float.as_integer_ratio
gpshead Sep 4, 2022
17bd053
gh-95778: Correctly pre-check for int-to-str conversion (#96537)
mdickinson Sep 4, 2022
c9212d5
Use the 3.8 Get API for it to compile.
gpshead Sep 4, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Make the doctest actually run & fix it.
  • Loading branch information
@gpshead
gpshead committed Sep 1, 2022
commit 70b9aefc09252c33709de425c4f0837d70cd0286
4 changes: 3 additions & 1 deletion Doc/library/stdtypes.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4903,6 +4903,8 @@ and the sign are not counted towards the limit.

When an operation would exceed the limit, a :exc:`ValueError` is raised::

.. doctest::

>>> import sys
>>> sys.set_int_max_str_digits(4300) # Illustrative, this is the default.
>>> _ = int('2' * 5432)
Expand All @@ -4919,7 +4921,7 @@ When an operation would exceed the limit, a :exc:`ValueError` is raised::
ValueError: Exceeds the limit (4300) for integer string conversion: value has 8599 digits.
>>> len(hex(i_squared))
7144
>>> assert int(hex(i_squared), base=16) == i # Hexidecimal is unlimited.
>>> assert int(hex(i_squared), base=16) == i*i # Hexidecimal is unlimited.

The default limit is 4300 digits as provided in
:data:`sys.int_info.default_max_str_digits <sys.int_info>`.
Expand Down
4 changes: 2 additions & 2 deletions Misc/NEWS.d/next/Security/2022-08-07-16-53.gh-issue-95778.ch010gps.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,5 @@ line flag, or :mod:`sys` APIs. See the :ref:`integer string conversion length
limitation <int_max_str_digits>` documentation. The default limit is 4300
digits in string form.

Patch by Gregory P. Smith [Google] and Christian Heimes [Red Hat] with feedback from
Victor Stinner, Thomas Wouters, and Steve Dower.
Patch by Gregory P. Smith [Google] and Christian Heimes [Red Hat] with feedback
from Victor Stinner, Thomas Wouters, Steve Dower, and Ned Deily.