-
-
Notifications
You must be signed in to change notification settings - Fork 52
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
precomp segfaults for this file. #59
Comments
It's giflib. Disabling it completes the precomp. |
try on Win7 64bit.
|
Regression from 0.4.5 (where everything works according to a quick test), one of the culprits is a GIF stream at position 1260117 (60.09%), but there seem to be other problems, as the following gives no segfault on my machine (Win7 64bit, Precomp 0.4.6 commit d9f61c6): precomp -cn -intense -zl18,68,69,82,95 -v -i1254601 -i1244353 -i1237086 -i1162009 -i1146093 -i1138392 precomp_segfault.tar (it basically ignores several streams before the GIF stream, but not the GIF stream itself) OTOH, undefined behaviour leads to undefined behaviour, so you never know... |
Mine is a Linux machine. |
Segfaults 4.5 also. |
Debugging with Visual Studio worked good. This way, the error could be isolated to testing a small GIF file (see attachment). The image is 1x1 pixel, but has no colormap, so that the function EGifPutImageDesc returns an error code - this code was unhandled and led to accessing uninitialized memory. |
Thanks. |
pre compressing this file --
precomp_segfault.zip
with command line --
-cn -oprecomp_tar
segfaults precomp on LInux 64
I tried to get the BT, but the debugging symbols where not there in the binary version and the one that I built from git. I tried changing the CFLAGS and added -ggdb, but the debugging symbols still got stripped. On changing the optimization levels to -O0 for all libraries, solved the problem but of course it's a bad solution.
The text was updated successfully, but these errors were encountered: