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

Disable Zip Decryption auth check. #3151

Merged
merged 2 commits into from
Dec 7, 2023
Merged

Disable Zip Decryption auth check. #3151

merged 2 commits into from
Dec 7, 2023

Conversation

scudette
Copy link
Contributor

@scudette scudette commented Dec 7, 2023

This default causes zip members to be decompressed into a memory
buffer leading to huge memory consumption.

In practice the embedded zip file has sufficient structure to be
fortified against bit manipulation attacks anyway and there are
checksums.

Optionally report the derived session password. This allows decryption by other tools (e.g. 7zip).

This default causes zip members to be decompressed into a memory
buffer leading to huge memory consumption.

In practice the embedded zip file has sufficient structure to be
fortified against bit manipulation attacks anyway and there are
checksums.
This allows decryption by other tools (e.g. 7zip).
@scudette scudette merged commit c7e3661 into master Dec 7, 2023
1 check passed
@scudette scudette deleted the unzip branch December 7, 2023 06:26
scudette added a commit that referenced this pull request Dec 13, 2023
This default causes zip members to be decompressed into a memory
buffer leading to huge memory consumption.

In practice the embedded zip file has sufficient structure to be
fortified against bit manipulation attacks anyway and there are
checksums.

Optionally report the derived session password. This allows decryption
by other tools (e.g. 7zip).
scudette added a commit that referenced this pull request Dec 13, 2023
This default causes zip members to be decompressed into a memory
buffer leading to huge memory consumption.

In practice the embedded zip file has sufficient structure to be
fortified against bit manipulation attacks anyway and there are
checksums.

Optionally report the derived session password. This allows decryption
by other tools (e.g. 7zip).
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.

1 participant