Skip to content

bpo-41085: Fixed 'array.array.index' downcasting 'Py_ssize_t' to 'long'.#21071

Merged
vstinner merged 2 commits intopython:masterfrom
WildCard65:bpo-41085
Jun 23, 2020
Merged

bpo-41085: Fixed 'array.array.index' downcasting 'Py_ssize_t' to 'long'.#21071
vstinner merged 2 commits intopython:masterfrom
WildCard65:bpo-41085

Conversation

@WildCard65
Copy link
Copy Markdown
Contributor

@WildCard65 WildCard65 commented Jun 23, 2020 •
edited
Loading

Fix integer overflow in the array.array.index() method on 64-bit Windows for index larger than 2**31.

https://bugs.python.org/issue41085

vstinner
vstinner reviewed Jun 23, 2020
View reviewed changes
Copy link
Copy Markdown
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

Please add a NEWS entry: either run blurb tool locally, or use https://blurb-it.herokuapp.com/ service.

@vstinner
Copy link
Copy Markdown
Member

vstinner commented Jun 23, 2020

The function 'array_array_index' was invoking 'PyLong_FromLong' instead of 'PyLong_FromSsize_t', this was causing the returned index to overflow the signed positive limit, not only that, but on some platforms, 'long' is an alias for 'int'

I suggest to write a more high-level description for end users, like:

"Fix integer overflow in the array.array.index() method on 64-bit Windows for index larger than 2**31."

vstinner
vstinner reviewed Jun 23, 2020
View reviewed changes
Misc/NEWS.d/next/Tests/2020-06-23-12-02-45.bpo-41085.JZKsyz.rst Outdated
Copy link
Copy Markdown
Member

@vstinner vstinner Jun 23, 2020

Choose a reason for hiding this comment

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

(copy/paste of my comment :-))

I suggest to write a more high-level description for end users, like:

"Fix integer overflow in the array.array.index() method on 64-bit Windows for index larger than 2**31."

@WildCard65
Copy link
Copy Markdown
Contributor Author

WildCard65 commented Jun 23, 2020

@vstinner There, I made the changes you requested.

@WildCard65 WildCard65 requested a review from vstinner June 23, 2020 12:09
vstinner
vstinner reviewed Jun 23, 2020
View reviewed changes
@WildCard65 WildCard65 requested a review from vstinner June 23, 2020 12:25
@WildCard65
Copy link
Copy Markdown
Contributor Author

WildCard65 commented Jun 23, 2020

This pull request may need backporting to older Python versions (Found the issue goes as far back as Python 2.7)

@vstinner vstinner merged commit 1d3dad5 into python:master Jun 23, 2020
@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented Jun 23, 2020

Thanks @WildCard65 for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented Jun 23, 2020

Thanks @WildCard65 for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 23, 2020
@WildCard65 @miss-islington
bpo-41085: Fix array.array.index() on 64-bit Windows (pythonGH-21071)
8431d29 
Fix integer overflow in the :meth:`array.array.index` method on 64-bit Windows
for index larger than ``2**31``.
(cherry picked from commit 1d3dad5)

Co-authored-by: WildCard65 <WildCard65@users.noreply.github.com>
@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented Jun 23, 2020

GH-21076 is a backport of this pull request to the 3.9 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 23, 2020
@WildCard65 @miss-islington
bpo-41085: Fix array.array.index() on 64-bit Windows (pythonGH-21071)
160a812 
Fix integer overflow in the :meth:`array.array.index` method on 64-bit Windows
for index larger than ``2**31``.
(cherry picked from commit 1d3dad5)

Co-authored-by: WildCard65 <WildCard65@users.noreply.github.com>
@bedevere-bot
Copy link
Copy Markdown

bedevere-bot commented Jun 23, 2020

GH-21077 is a backport of this pull request to the 3.8 branch.

@vstinner
Copy link
Copy Markdown
Member

vstinner commented Jun 23, 2020

Thanks @WildCard65, I merged your PR!

This pull request may need backporting to older Python versions (Found the issue goes as far back as Python 2.7)

Right. I created backports to 3.8 and 3.9 branches. I don't think that it's worth it to backport it to 3.7 which is very close to move to security-only. Python 2.7 is no longer maintained.

https://devguide.python.org/#status-of-python-branches

miss-islington added a commit that referenced this pull request Jun 23, 2020
@miss-islington @WildCard65
bpo-41085: Fix array.array.index() on 64-bit Windows (GH-21071)
c6e24e7 
Fix integer overflow in the :meth:`array.array.index` method on 64-bit Windows
for index larger than ``2**31``.
(cherry picked from commit 1d3dad5)

Co-authored-by: WildCard65 <WildCard65@users.noreply.github.com>
miss-islington added a commit that referenced this pull request Jun 23, 2020
@miss-islington @WildCard65
bpo-41085: Fix array.array.index() on 64-bit Windows (GH-21071)
92f8b48 
Fix integer overflow in the :meth:`array.array.index` method on 64-bit Windows
for index larger than ``2**31``.
(cherry picked from commit 1d3dad5)

Co-authored-by: WildCard65 <WildCard65@users.noreply.github.com>
@WildCard65 WildCard65 deleted the bpo-41085 branch June 23, 2020 13:48
fasih pushed a commit to fasih/cpython that referenced this pull request Jun 29, 2020
@WildCard65 @fasihahmad
bpo-41085: Fix array.array.index() on 64-bit Windows (pythonGH-21071)
a63a16b 
Fix integer overflow in the :meth:`array.array.index` method on 64-bit Windows
for index larger than ``2**31``.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vstinner vstinner Awaiting requested review from vstinner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@WildCard65 @vstinner @miss-islington @bedevere-bot @the-knights-who-say-ni