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

Buffer length too short #61

Open
alexvolchek opened this issue May 30, 2017 · 3 comments
Open

Buffer length too short #61

alexvolchek opened this issue May 30, 2017 · 3 comments

Comments

@alexvolchek
Copy link

What is the point of case if (len > UINT16_MAX) return UNZ_PARAMERROR; in unzReadCurrentFile?
It fails on any file more than 65kb.

@gianlucabertani
Copy link
Owner

The limit applies to the buffer size, not to the file size. You can always read the file with a loop, using a buffer shorter than 65 KB. The README contains such an example. You can also find, in the source files, a (commented-out) unit test that zips and unzips a 5 GB file.

Anyway, I am trying to understand with the maintainer of MiniZip why this limit has been added. Since it has been introduced only recently, maybe it can be lifted. Will keep you updated.

@flaushi
Copy link

flaushi commented Jun 13, 2017

I can confirm this problem! Until a few weeks ago this was no issue!
EDIT: after reverting to 1.0.3, the problem does not occur anymore. :-/

@AEiosApp
Copy link

Got into the same issue with 1.0.5. Thanks for @flaushi comment, reverting to 1.0.3 resolved the issue.

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

No branches or pull requests

4 participants