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.
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
struct.pack('L', -1) #48478
Comments
struct.pack('L', -1) raises a DeprecationWarning since Python 2.5, as it This might lead the zipfile module of release25-maint (the version >= Given the DeprecationWarning I would classify this as low priority. |
Ah, I should also mention that a fix of zipfile for 2.5 to no longer use |
Can you provide a fix within the next two days? Otherwise, I see little |
FWIW, struct.pack("I", "whatever") produces "\x00\x00\x00\x00" too. |
Attached struct-2.5-fix.diff. The tests still pass (both 32- and 64-bits). |
Thanks for the patch. Committed (along with a test case) as r67733, in Porting to the other branches still needs to happen. Armin, if you want |
Committed to trunk in r68120. |
Was zipfile ever fixed to avoid this deprecated behaviour? If not, is the It would be nice to be able to finally turn these struct deprecation |
As I understand actually the zipfile module possibly creates damaged zip I think the _struct.c should be cleaned in any case. Because we only get But, why don't boundary check the header offset in zipfile module in a |
I don't know the zipfile module very well (i.e., at all), Can anyone who understands the zipfile module better than |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: