-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Bug in inflate.c #274
Comments
See also issue #41. I would recommend never enabling the ASM optimizations until this issue is fixed. |
And what about inffasx64.asm and gvmat64.asm (x64 asm files)? Are they ok or they're buggy too? |
The assembler code is in contrib and not part of the supported zlib. Since that code seems to be causing problems, it will be removed from the distribution. |
perdrix52
added a commit
to deepskystacker/DSS
that referenced
this issue
Apr 24, 2018
…b#274 This was causing access violations in the x86 builds of DSS. The debug builds never used assembler code.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Zlib 1.2.11 supposedly has a bug in inflate.c.
How to reproduce:
You'll get access violation in contrib\masmx86\inffas32.asm, line 669: mov eax, [ebx+eax*4]
x64 works fine.
1.2.8 worked fine for both x86 and x64.
The text was updated successfully, but these errors were encountered: