qiqian / webp

Automatically exported from code.google.com/p/webp
0 stars 0 forks source link

Floating point exception with cwebp -progress #121

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
'cwebp all_white.png -o out.webp -progress' crashes with a 'Floating point 
exception'. See following stacktrace :

Program received signal SIGFPE, Arithmetic exception.
0x00007ffff799baf0 in VP8IteratorProgress () from 
/home/even/libwebp/install/lib/libwebp.so.2
(gdb) bt
#0  0x00007ffff799baf0 in VP8IteratorProgress () from 
/home/even/libwebp/install/lib/libwebp.so.2
#1  0x00007ffff7993236 in VP8EncAnalyze () from 
/home/even/libwebp/install/lib/libwebp.so.2
#2  0x00007ffff79a73f5 in WebPEncode () from 
/home/even/libwebp/install/lib/libwebp.so.2
#3  0x0000000000404a8e in main ()

System: Linux amd64
libwebp version: latest GIT master as of today (May 16 2012)

Image to reproduce attached

Original issue reported on code.google.com by even.rou...@gmail.com on 16 Jun 2012 at 3:59

Attachments:

GoogleCodeExporter commented 8 years ago
(Actually the image is all red, not all white ;-))

Original comment by even.rou...@gmail.com on 16 Jun 2012 at 4:00

GoogleCodeExporter commented 8 years ago
Arg, and GIT version is June 16 2012

Original comment by even.rou...@gmail.com on 16 Jun 2012 at 4:00

GoogleCodeExporter commented 8 years ago
The attached patch fixes the crash (not sure this is completely correct however)

Original comment by even.rou...@gmail.com on 16 Jun 2012 at 4:23

Attachments:

GoogleCodeExporter commented 8 years ago
I can reproduce this and the fix looks reasonable. I'll post a patch with this 
change, thanks!

Original comment by jz...@google.com on 18 Jun 2012 at 10:45

GoogleCodeExporter commented 8 years ago
https://gerrit.chromium.org/gerrit/25570

Original comment by jz...@google.com on 18 Jun 2012 at 10:48

GoogleCodeExporter commented 8 years ago

Original comment by jz...@google.com on 18 Jun 2012 at 10:48

GoogleCodeExporter commented 8 years ago
This has been submitted.

Original comment by jz...@google.com on 19 Jun 2012 at 12:16