Closed Bug 1473943 Opened 6 years ago Closed 6 years ago

Robustify blob replay against bad data

Categories

(Core :: Graphics: WebRender, enhancement, P1)

Other Branch
enhancement

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: kats, Assigned: jrmuizel)

References

Details

(Whiteboard: [gfx-noted])

Attachments

(1 file)

When I was working on bug 1469528 I noticed that there is little sanity-checking of the blob images in the compositor. We just take the blob and run it through the replay mechanism, and that provides a lot of opportunities for a malicious content process to corrupt rendering, crash the compositor, or even potentially write to arbitrary memory [1], which would be Really Bad. This needs to be fixed before we ship. [1] https://searchfox.org/mozilla-central/rev/1193ef6a61cb6e350460eb2e8468184d3cb0321d/gfx/webrender_bindings/Moz2DImageRenderer.cpp#303
Comment on attachment 9003344 [details] Bug 1473943. Make blob bounds checks safe. r=mstange Markus Stange [:mstange] has approved the revision.
Attachment #9003344 - Flags: review+
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Assignee: nobody → jmuizelaar
Depends on: 1486198
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: