The default bug view has changed. See this FAQ.

crash in Accessible::NativeState

RESOLVED FIXED in mozilla17

Status

()

Core
Disability Access APIs
--
critical
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Scoobidiver (away), Assigned: tbsaunde)

Tracking

({crash, regression})

17 Branch
mozilla17
crash, regression
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(crash signature)

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
It first appeared in 17.0a1/20120722. The regression range is:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=446b788ab99d&tochange=462106f027af
It's likely a regression from bug 762876.

Stack traces are various:
Frame 	Module 	Signature 	Source
0 	xul.dll 	nsIFrame::GetStyleXUL 	layout/style/nsStyleStructList.h:119
1 	xul.dll 	Accessible::NativeState 	accessible/src/generic/Accessible.cpp:687
2 	xul.dll 	mozilla::a11y::XULMenupopupAccessible::NativeState 	accessible/src/xul/XULMenuAccessible.cpp:448
3 	xul.dll 	Accessible::State 	accessible/src/generic/Accessible.cpp:1450
4 	xul.dll 	AccessibleWrap::get_accState 	accessible/src/msaa/AccessibleWrap.cpp:468
5 	rpcrt4.dll 	Invoke 	
...

Frame 	Module 	Signature 	Source
0 	xul.dll 	nsIFrame::GetStyleXUL 	layout/style/nsStyleStructList.h:119
1 	xul.dll 	Accessible::NativeState 	accessible/src/generic/Accessible.cpp:687
2 	xul.dll 	mozilla::a11y::XULButtonAccessible::NativeState 	accessible/src/xul/XULFormControlAccessible.cpp:96
3 	xul.dll 	Accessible::State 	accessible/src/generic/Accessible.cpp:1450
4 	xul.dll 	AccessibleWrap::get_accState 	accessible/src/msaa/AccessibleWrap.cpp:468
5 	oleacc.dll 	AccWrap_Base::get_accState 	
6 	oleacc.dll 	AccWrap_Annotate::get_accState 	
7 	FSDomNodeFirefox.DLL 	FSDomNodeFirefox.DLL@0x109d1
...

Frame 	Module 	Signature 	Source
0 	libxul.so 	Accessible::NativeState 	nsStyleStructList.h:119
1 	libxul.so 	HyperTextAccessible::NativeState 	accessible/src/generic/HyperTextAccessible.cpp:124
2 	libxul.so 	mozilla::a11y::XULLabelAccessible::NativeState 	accessible/src/xul/XULElementAccessibles.cpp:55
3 	libxul.so 	Accessible::State 	accessible/src/generic/Accessible.cpp:1450
4 	libxul.so 	refStateSetCB 	accessible/src/atk/AccessibleWrap.cpp:885
5 	libatk-1.0.so.0.20409.1 	libatk-1.0.so.0.20409.1@0xb255 
...

More reports at:
https://crash-stats.mozilla.com/report/list?signature=nsIFrame%3A%3AGetStyleXUL%28%29
https://crash-stats.mozilla.com/report/list?signature=Accessible%3A%3ANativeState
Assigning based on suspected regression cause.
Assignee: nobody → trev.saunders
(Assignee)

Comment 2

5 years ago
Created attachment 644982 [details] [diff] [review]
patch
Attachment #644982 - Flags: review?(dbolter)
Comment on attachment 644982 [details] [diff] [review]
patch

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

Why not just check the frame before using it (a one line patch)?
(Assignee)

Comment 4

5 years ago
(In reply to David Bolter [:davidb] from comment #3)
> Comment on attachment 644982 [details] [diff] [review]
> patch
> 
> Review of attachment 644982 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Why not just check the frame before using it (a one line patch)?

trying to not check things multiple times, something wrong with it?
(In reply to Trevor Saunders (:tbsaunde) from comment #4)
> (In reply to David Bolter [:davidb] from comment #3)
 
> trying to not check things multiple times, something wrong with it?

I guess it is either two isXUL checks or two frame checks, I'm more used to frame checks but meh.

Updated

5 years ago
Attachment #644982 - Flags: review?(dbolter) → review+
I hit this crash while trying to test another bug - https://crash-stats.mozilla.com/report/index/bp-c3b3de8d-3952-4aab-8fb6-089d12120723. At the time of the crash I was using the back button to navigate back from a site. If I can reproduce I will let you know.
(Assignee)

Comment 7

5 years ago
landed https://hg.mozilla.org/integration/mozilla-inbound/rev/fcc58ba01ef6
(Assignee)

Comment 8

5 years ago
(In reply to David Bolter [:davidb] from comment #5)
> (In reply to Trevor Saunders (:tbsaunde) from comment #4)
> > (In reply to David Bolter [:davidb] from comment #3)
>  
> > trying to not check things multiple times, something wrong with it?
> 
> I guess it is either two isXUL checks or two frame checks, I'm more used to
> frame checks but meh.

true, but I realized I'd have to do the second IsXUL() check after I started changing stuff...
https://hg.mozilla.org/mozilla-central/rev/fcc58ba01ef6
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.