Closed Bug 844684 Opened 11 years ago Closed 11 years ago

GIF resource not displayed

Categories

(Core :: Graphics: ImageLib, defect)

Product:
Core
Component:
Graphics: ImageLib
Version:
18 Branch
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla22

People

(Reporter: sl.ostapenko, Assigned: jdm)

References

Details

Attachments

(2 files)

sample.gif
1.45 KB, image/gif
Details
Decode GIFs that include an application extension string shorter than 11 bytes.
8.08 KB, patch
joe
: review+
Details | Diff | Splinter Review
Attached image sample.gif — 
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux i686; rv:18.0) Gecko/20100101 Firefox/18.0
Build ID: 20130201190421

Steps to reproduce:

Open attached sample.gif .


Actual results:

Image is not displayed, some generic icon is shown instead.


Expected results:

Image should be displayed.

This particular - and many other similary 'broken' GIFs - image is served (in data:base64 format) by Amazon Cloud Reader, specifically when viewing 'Programming in Scala, Second Edition' by Odersky, Spoon, Venners.

Similar bug also have reported for chromium: http://code.google.com/p/chromium/issues/detail?id=177561
Webkit fix is here: http://code.google.com/p/chromium/issues/detail?id=177561

Opera and IE are able to dispaly those gifs.
Status: UNCONFIRMED → NEW
Component: Untriaged → ImageLib
Ever confirmed: true
Product: Firefox → Core
We could use the same patch if we are still in sync with the chromium code
The WebKit version of this code was recently refactored to comply with the WebKit style guide, so there will be stylistic differences.  The semantics of the necessary patch are likely to be identical, however.
Assignee: nobody → josh
Comment on attachment 717901 [details] [diff] [review]
Decode GIFs that include an application extension string shorter than 11 bytes.

Review of attachment 717901 [details] [diff] [review]:
-----------------------------------------------------------------

Make sure bug 789046 isn't broken by this
Attachment #717901 - Flags: review?(joe) → review+
Tested an ASAN build with this patch, and loading both gifs from bug 789046 did not trigger any problems.
https://hg.mozilla.org/mozilla-central/rev/ca131cc6fdac
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.