The default bug view has changed. See this FAQ.

Warning followed by assertion from reftest/tests/modules/libpr0n/test/crashtests/delaytest.html?523528-1.gif

RESOLVED FIXED in mozilla11

Status

()

Core
ImageLib
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: bas, Assigned: mattwoodrow)

Tracking

unspecified
mozilla11
x86
Windows 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
This tests triggers the creation of a very large image surface. This fails because of checks, but the codepaths this triggers then continue to use the (invalid) image surface. This triggers a couple of assertions, but the NULL surface ends up being silently ignored by Cairo.

This doesn't seem like it's something we should rely on though, it seems to be a result of the 'RasterImage' believing it's ridiculously big (40K+).
(Reporter)

Comment 1

6 years ago
The GIF actually seems to have a frame that big (49173x49173), so the fundamental problem seems to be more about how RasterImage deals with the fact it is not allowed to create a new image surface of that size.
(Assignee)

Comment 2

5 years ago
Created attachment 575423 [details] [diff] [review]
Check CairoStatus of surfaces in DrawImage
Attachment #575423 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/6606fa72c881
Assignee: nobody → matt.woodrow
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
You need to log in before you can comment on or make changes to this bug.