Issue 16550: pickletools.py treats 32bit lengths as signed, but pickle.py as unsigned (original) (raw)

This issue has been migrated to GitHub: https://github.com/python/cpython/issues/60754

classification

Title: pickletools.py treats 32bit lengths as signed, but pickle.py as unsigned
Type: behavior Stage: resolved
Components: Library (Lib) Versions: Python 3.2, Python 3.3, Python 3.4

process

Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: alexandre.vassalotti Nosy List: alexandre.vassalotti, jcea, pitrou, python-dev, serhiy.storchaka
Priority: normal Keywords: patch

Created on 2012-11-25 12:17 by serhiy.storchaka, last changed 2022-04-11 14:57 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
pickletools_uint4.patch serhiy.storchaka,2012-11-25 12:17 review
Messages (4)
msg176344 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2012-11-25 12:17
Since some lengths in pickle module treated as unsigned, however pyckletools module treats all 32-bit lengths as signed. The proposed patch adds support of unsigned 32-bit lengths.
msg177552 - (view) Author: Antoine Pitrou (pitrou) * (Python committer) Date: 2012-12-15 19:28
Does it need additional tests?
msg177563 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2012-12-15 20:54
Lib/test/pickletester.py contains some bigmem tests, they can be modified to cover and this issue. But I can't run this tests.
msg186907 - (view) Author: Roundup Robot (python-dev) (Python triager) Date: 2013-04-14 10:45
New changeset 4ced30417300 by Alexandre Vassalotti in branch '3.3': Issue #16550: Update the opcode descriptions of pickletools to use unsigned http://hg.python.org/cpython/rev/4ced30417300
History
Date User Action Args
2022-04-11 14:57:38 admin set github: 60754
2013-04-15 19:43:59 serhiy.storchaka set status: open -> closedassignee: serhiy.storchaka -> alexandre.vassalottiresolution: fixedstage: patch review -> resolved
2013-04-14 10:45:22 python-dev set nosy: + python-devmessages: +
2012-12-29 21:56:22 serhiy.storchaka set assignee: serhiy.storchaka
2012-12-15 20:54:54 serhiy.storchaka set messages: +
2012-12-15 19:28:58 pitrou set messages: +
2012-11-26 17🔞00 jcea set nosy: + jcea
2012-11-25 12:17:16 serhiy.storchaka create