You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Mar 1, 2024. It is now read-only.
Repository navigation
This repository was archived by the owner on Mar 1, 2024. It is now read-only.
The deflate_medium strategy occasionally produces outputs that are corrupt in various ways. The archive http://jsnell.iki.fi/tmp/deflate-medium-testcase.tar.gz has a minimal test case. The result of running it against this zlib at level 5 can't be decompressed, but can be if run at level 9. Likewise decompressing the output works with baseline zlib at all levels.
This test case has been heavily pruned from a much larger one. I'm hoping there's just one bug here, but it's possible there are multiple, since the behavior is occasionally different with the larger test cases. (I can't really direct the automated pruning process in any way, so if there are multiple issues, creating a minimal test case would require that this issue be fixed).
I can reproduce the issue with your testcase. Interestingly, it doesn't occur when decompressing with minigzip or gzip, as opposed to openssl. I'll investigate and get back to you.
Activity
jtkukunas commentedon May 5, 2014
Sorry for the delay in responding ...
I can reproduce the issue with your testcase. Interestingly, it doesn't occur when decompressing with minigzip or gzip, as opposed to openssl. I'll investigate and get back to you.
Thanks.
joshtriplett commentedon Jun 2, 2014
Any progress tracking down the issue?
jtkukunas commentedon Jun 3, 2014
Yeah, I tracked the issue down to an edge case, triggered by the sync flush in the testcase, where we would update the hash with invalid data.
jsnell,
I've pushed a branch, issue2, that should resolve the issue. Can you confirm whether this fixes the bug for you? Specifically, commit ef27907.
Thanks.
jsnell commentedon Jun 3, 2014
Thanks, that fixes my problem.
deflate: add new deflate_medium strategy
deflate: add new deflate_medium strategy
deflate: add new deflate_medium strategy
deflate: add new deflate_medium strategy
deflate: add new deflate_medium strategy
deflate: add new deflate_medium strategy
deflate: add new deflate_medium strategy
deflate: add new deflate_medium strategy
deflate: add new deflate_medium strategy
deflate: add new deflate_medium strategy