Thanks to visit codestin.com
Credit goes to github.com

Skip to content

gh-111140: Fix edge case in PyLong_AsNativeBytes where large negative longs may require an extra byte #116053

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
encukou merged 19 commits into from
Apr 5, 2024
Merged

gh-111140: Fix edge case in PyLong_AsNativeBytes where large negative longs may require an extra byte #116053

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
d5b0e98
gh-111140: Fix edge case in PyLong_AsNativeBytes where large negative…
zooba Feb 28, 2024
f162348
Improved edge case testing and fuzz tests
zooba Mar 1, 2024
0efa143
Test comments
zooba Mar 1, 2024
15b3752
Fix for x86
zooba Mar 4, 2024
fbe42e7
Merge remote-tracking branch 'upstream/main' into gh-111140
zooba Mar 19, 2024
8dedad7
More precise check for unsigned non-overflow
zooba Mar 19, 2024
d3deebf
Merge branch 'main' into gh-111140
zooba Mar 28, 2024
621708c
Change endianness arguments to flags to allow additional arguments
zooba Mar 28, 2024
c607f04
Space
zooba Mar 28, 2024
140c011
Fix tests, one result, and docs
zooba Mar 28, 2024
77097bf
Fix test
zooba Mar 28, 2024
a8e2540
Update NEWS
zooba Mar 28, 2024
5ba6451
Missed colon
zooba Mar 28, 2024
68b9d8e
Doc fixes and improved test failure output
zooba Apr 2, 2024
37ad671
Remove my deliberate failure
zooba Apr 2, 2024
1e615cf
Reword PyLong_AsNativeBytes docs (#33)
encukou Apr 4, 2024
d1b886c
Apply PR feedback
zooba Apr 4, 2024
38ecfe2
Merge branch 'gh-111140' of https://github.com/zooba/cpython into gh-…
zooba Apr 4, 2024
614a6ae
Check for new error
zooba Apr 4, 2024
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
Fix test
  • Loading branch information
@zooba
zooba committed Mar 28, 2024
commit 77097bf4cb3966ccbfa1b80a2a2abd236184c4c1
6 changes: 2 additions & 4 deletions Lib/test/test_capi/test_long.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -711,10 +711,8 @@ def test_long_fromnativebytes(self):
self.assertEqual(expect_u, fromnativebytes(v_be, n, -1, 0),
f"PyLong_FromUnsignedNativeBytes(buffer, {n}, <native>)")

# Swap the signed/unsigned request for tests and use the
# Py_ASNATIVEBYTES_UNSIGNED_BUFFER flag to select the opposite
self.assertEqual(expect_s, fromnativebytes(v_be, n, 0, 0),
f"PyLong_FromUnsignedNativeBytes(buffer, {n}, <big|signed>)")
# Swap the unsigned request for tests and use the
# Py_ASNATIVEBYTES_UNSIGNED_BUFFER flag instead
self.assertEqual(expect_u, fromnativebytes(v_be, n, 4, 1),
f"PyLong_FromNativeBytes(buffer, {n}, <big|unsigned>)")

Expand Down