Skip to content
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

zip64: fix zip64 extended information issue #265

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

onlyxuyang
Copy link

@onlyxuyang onlyxuyang commented Nov 21, 2024

A updated version of #248 and fix crypto case issue, now it can pass CI.

Fix zip64 extended information issue to fix zip file incorrect if size larger than 4GB.

According zip format 4.5.3:
The order of the fields in the zip64 extended
information record is fixed, but the fields MUST
only appear if the corresponding Local or Central
directory record field is set to 0xFFFF or 0xFFFFFFFF.

@wolfv
Copy link

wolfv commented Dec 10, 2024

@Pr0methean can we run the testts here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants