cxt / webp

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

File successfully decoded in 0.4.0 and fails in 0.4.3 #264

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
Running dwebp on the attached file fails.

What version of the product are you using? On what operating system?
Works with 0.4.0 but doesn't work in 0.4.3

Please provide any additional information below.
The file was created via photoshop.

Original issue reported on code.google.com by tal.lev...@gmail.com on 26 Sep 2015 at 9:34

Attachments:

GoogleCodeExporter commented 8 years ago
I confirm that this file decodes OK till 0.4.0 (included) and fails afterward, 
starting at 0.4.1.

Will have a deeper look. Note that 'webpmux -info' fails on it too.

Original comment by pascal.m...@gmail.com on 28 Sep 2015 at 12:38

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
This file is, in fact, invalid:
The RIFF chunk size (stored in bytes 5 to 8 of the file) is 582360. But the 
actual file size (minus RIFF chunk header size = 8) is 582340.
So, decoder is complaining that file is 20 bytes shorter than expected.
This looks like a bug in the Photoshop plugin.

Note: v0.4.3 catches this issue (while 0.4.0 didn't) because it probably has a 
stricter check for this.

Original comment by urv...@google.com on 28 Sep 2015 at 3:55

GoogleCodeExporter commented 8 years ago
FYI:
The patch that added this stricter check: 
https://chromium-review.googlesource.com/#/c/68731/

This was to fix this issue: https://code.google.com/p/webp/issues/detail?id=185

Feel free to file a bug on Photoshop WebP plugin for this.

Original comment by urv...@google.com on 28 Sep 2015 at 3:59