Closed Bug 1195098 Opened 7 years ago Closed 7 years ago

crash in CreateBlurMask


(Core :: Graphics, defect)

41 Branch
Windows NT
Not set



Tracking Status
firefox41 --- fixed
firefox42 --- fixed
firefox43 --- fixed


(Reporter: philipp, Assigned: mchang)



(Keywords: crash, regression, Whiteboard: gfx-noted)

Crash Data


(1 file)

This bug was filed from the Socorro interface and is 
report bp-2051eca1-1c7e-4a53-a6c5-72cf52150816.

Crashing Thread
Frame 	Module 	Signature 	Source
0 	xul.dll 	CreateBlurMask 	gfx/thebes/gfxBlur.cpp
1 	xul.dll 	GetBlur(mozilla::gfx::DrawTarget&, mozilla::gfx::IntSizeTyped<mozilla::gfx::UnknownUnits> const&, mozilla::gfx::IntSizeTyped<mozilla::gfx::UnknownUnits> const&, mozilla::gfx::RectCornerRadii*, gfxRGBA const&, mozilla::gfx::IntMarginTyped<mozilla::gfx::UnknownUnits>&, mozilla::gfx::IntMarginTyped<mozilla::gfx::UnknownUnits>&) 	gfx/thebes/gfxBlur.cpp
2 	xul.dll 	gfxAlphaBoxBlur::BlurRectangle(gfxContext*, gfxRect const&, mozilla::gfx::RectCornerRadii*, gfxPoint const&, gfxRGBA const&, gfxRect const&, gfxRect const&) 	gfx/thebes/gfxBlur.cpp
3 	xul.dll 	nsContextBoxBlur::BlurRectangle(gfxContext*, nsRect const&, int, mozilla::gfx::RectCornerRadii*, int, gfxRGBA const&, nsRect const&, gfxRect const&) 	layout/base/nsCSSRendering.cpp
4 	xul.dll 	nsCSSRendering::PaintBoxShadowOuter(nsPresContext*, nsRenderingContext&, nsIFrame*, nsRect const&, nsRect const&, float) 	layout/base/nsCSSRendering.cpp

this graphics related crash signature seems to have been introduced with firefox 41, and is taking place across multiple windows versions (about half of the time under windows 10). it's not exclusive to any particular graphics card vendor...
Component: General → Graphics
presumably regressed by bug 1155828
Blocks: 1155828
Mason can you add some instrumentation to figure out what's crashing and why?
Assignee: nobody → mchang
Flags: needinfo?(mchang)
Flags: needinfo?(mchang)
Whiteboard: gfx-noted
From the crash report in comment 0, looks like [1] can sometimes return null. The previous code in Gecko 40 null checks this. This patch adds the null check.

Attachment #8649437 - Flags: review?(mstange)
Attachment #8649437 - Flags: review?(mstange) → review+
changeset:  e09694f1df5853bd3b0a4d460dead8306023e6cd
user:       Mason Chang <>
date:       Tue Aug 18 20:43:24 2015 -0700
Bug 1195098. Null check for blurred source surface. r=mstange
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
Comment on attachment 8649437 [details] [diff] [review]
Null check blurred surface

Approval Request Comment
[Feature/regressing bug #]: Bug 1155828, speed up box shadow rendering.
[User impact if declined]: The user can crash when visiting a website with box-shadows.
[Describe test coverage new/current, TreeHerder]: Treeherder reftests
[Risks and why]:  Low, this is a null check that was deleted and this patch puts it back in.
[String/UUID change made/needed]: None
Attachment #8649437 - Flags: approval-mozilla-beta?
Attachment #8649437 - Flags: approval-mozilla-aurora?
Comment on attachment 8649437 [details] [diff] [review]
Null check blurred surface

Simple patch, Aurora+ and Beta+
Attachment #8649437 - Flags: approval-mozilla-beta?
Attachment #8649437 - Flags: approval-mozilla-beta+
Attachment #8649437 - Flags: approval-mozilla-aurora?
Attachment #8649437 - Flags: approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.