-
Notifications
You must be signed in to change notification settings - Fork 16
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
fix: Decrypt the read bytes in ZipCrypto instead of entire buffer #39
Conversation
Fixes `corrupt deflate stream` panic when extracting a file from encrypted archive (zip-rs#280).
Sorry for all the updates; I've been trying to troubleshoot https://github.com/zip-rs/zip2/actions/workflows/auto_merge_prs.yml, which somehow broke when I transferred this repo to be organization-owned in order to enable merge queues (issue now tracked at #40). |
Understandable, good old CI whack-a-mole. Looked at the Should I rebase to clean up the merge branch commits? |
You can if you like; meanwhile I'll look into the fuzz failure today. |
Found the cause of the fuzz failure; 4297229 should fix it. |
Update: merge-queue checks may fail again because of a missing DevSkim check, but fd0a113 should fix that. |
Merge-queue checks timed out again; decreased the amount of fuzz testing so that the GitHub Actions workers can catch up with the backlog. |
Resend of zip-rs#431.
Fixes
corrupt deflate stream
error when decompressing an encrypted archive with buffered reader (zip-rs#280).