Legacy ImageMagick Discussions Archive

correct path for 6.8.6-9 to fix this problem look so: --- a/coders/gif.c +++ b/coders/gif.c @@ -397,9 +397,6 @@ static MagickBooleanType DecodeImage(Image *image,const ssize_t opacity) int c; - InterlaceType - interlace; - LZWInfo *lzw_info; @@ -428,9 +425,6 @@ static MagickBooleanType DecodeImage ...

Thanks for the support. Result is not great not because rounding issue, but because fix is incorrect. If height of interlaced image less then 8px, missing rows (pass) should be skipped instead of disabling interlacing at all.

thanks for the fix, but now these images are incorrect after resize

looks like bug in following code in DecodeImage() and EncodeImage() functions in coders/gif.c switch (pass) { case 0: default: { offset+=8; if (offset >= (ssize_t) image->rows) { pass++; offset=4; } break; so decode will fail with all interlaced gifs containing frames with height less then 5

We have 2 animated GIFs: https://docs.google.com/file/d/0B9AgH7Lxh7dIUE9FSm5RQ0pPbk0/edit?usp=sharing https://docs.google.com/file/d/0B9AgH7Lxh7dIWXJQcjZucDQ3NXM/edit?usp=sharing This GIFs are correct but can't be opened in ImageMagic. It throws the following exception: $ ./convert /tmp/1.gif -debug ...