Closed Bug 1262681 Opened 5 years ago Closed 5 years ago
crash in mozilla::layers::Rotated
Buffer::Draw Buffer Quadrant
This bug was filed from the Socorro interface and is report bp-60244d11-07c4-4cfd-b546-115092160407. ============================================================= ¡Hola! 569 crashes (12 startup) in the past month at https://crash-stats.mozilla.com/report/list?product=Firefox&range_unit=days&range_value=28&signature=mozilla%3A%3Alayers%3A%3ARotatedBuffer%3A%3ADrawBufferQuadrant#tab-sigsummary
All of the ones I've seen call DrawTargetCairo::Snapshot with invalid DrawTarget, returning null , having RotatedBuffer::DrawBufferQuadrant (which got the snapshot through RotatedContentBuffer::GetSourceSurface call) use that snapshot without checking. This perhaps starts with Factory::CreateDrawTargetForData getting called with DIRECT2D1_1 as the BackendType, which is not something we do. Could be an OOM of some type, or a device reset.
Comment on attachment 8739130 [details] [diff] [review] Not necessariy a patch that deals with this issue, but we need to worry about the validity, not just the existence, of a draw target. Review of attachment 8739130 [details] [diff] [review]: ----------------------------------------------------------------- I'm not sure how much this adds, but it can't hurt.
Attachment #8739130 - Flags: feedback?(bas) → feedback+
5 years ago
(In reply to Carsten Book [:Tomcat] from comment #8) > https://hg.mozilla.org/mozilla-central/rev/7ed46b45eef8 No crashes of this type since just before this patch landed (20160413030239 is the last one so far.) I did find a (single, linux) crash that has error reports added in the above patch: https://crash-stats.mozilla.com/report/index/b6e49c8d-677d-4b22-99e0-8074f2160417. I doubt this is new, it just would have crashed elsewhere before. Lets wait for more days and data, and see if we get any crashes with the extra info once we go to dev edition with 48.
I don't see any of these crashes in 48 or 49. Not enough volume to uplift to 47 beta, looks like we're getting two or so crashes per day, so maybe just call this fixed and let it ride 48.
Assignee: nobody → milan
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.