Last Comment Bug 783383 - crash in nsBaseWidget::GetAccessible
: crash in nsBaseWidget::GetAccessible
: crash, regression, topcrash
Product: Core
Classification: Components
Component: Disability Access APIs (show other bugs)
: 17 Branch
: All All
-- critical (vote)
: mozilla17
Assigned To: Andrew Quartey [:drexler]
: alexander :surkov
Depends on:
Blocks: 743975
  Show dependency treegraph
Reported: 2012-08-16 14:30 PDT by Scoobidiver (away)
Modified: 2012-10-16 15:34 PDT (History)
8 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (1.40 KB, patch)
2012-08-16 16:11 PDT, Andrew Quartey [:drexler]
tnikkel: review+
Details | Diff | Splinter Review

Description User image Scoobidiver (away) 2012-08-16 14:30:37 PDT
It first appeared in 17.0a1/20120816. The regression range is:
It's likely a regression from bug 743975.

Signature 	nsBaseWidget::GetAccessible() More Reports Search
UUID	90181ba8-b252-41f4-b5e3-e2c862120816
Date Processed	2012-08-16 19:53:46
Uptime	3664
Last Crash	1.0 hours before submission
Install Age	3.6 hours since version was first installed.
Install Time	2012-08-16 16:20:07
Product	Firefox
Version	17.0a1
Build ID	20120816030539
Release Channel	nightly
OS	Windows NT
OS Version	6.1.7601 Service Pack 1
Build Architecture	x86
Build Architecture Info	GenuineIntel family 6 model 26 stepping 4
Crash Address	0x0
App Notes 	
AdapterVendorID: 0x1002, AdapterDeviceID: 0x9440, AdapterSubsysID: 20071787, AdapterDriverVersion: 8.970.100.3000
Processor Notes 	WARNING: JSON file missing Add-ons
EMCheckCompatibility	True
Adapter Vendor ID	0x1002
Adapter Device ID	0x9440
Total Virtual Memory	2147352576
Available Virtual Memory	1788301312
System Memory Use Percentage	78
Available Page File	927268864
Available Physical Memory	701054976

Frame 	Module 	Signature 	Source
0 	xul.dll 	nsBaseWidget::GetAccessible 	widget/xpwidgets/nsBaseWidget.cpp:1388
1 	xul.dll 	nsWindow::GetRootAccessible 	widget/windows/nsWindow.cpp:7361
2 	xul.dll 	nsWindow::ProcessMessage 	widget/windows/nsWindow.cpp:5052
3 	xul.dll 	nsWindow::WindowProcInternal 	widget/windows/nsWindow.cpp:4265
4 	xul.dll 	CallWindowProcCrashProtected 	xpcom/base/nsCrashOnException.cpp:32
5 	xul.dll 	nsWindow::WindowProc 	widget/windows/nsWindow.cpp:4207

More reports at:
Comment 1 User image Andrew Quartey [:drexler] 2012-08-16 15:36:15 PDT
The null check in static nsIPresShell* GetPresShell(nsIWidget* aWidget, void* clientData) was removed by this revision: Furthermore, not all places that use mWidgetListener were null-checked thereafter.
Comment 2 User image Andrew Quartey [:drexler] 2012-08-16 16:11:45 PDT
Created attachment 652605 [details] [diff] [review]
Comment 3 User image Trevor Saunders (:tbsaunde) 2012-08-17 08:11:04 PDT
Comment on attachment 652605 [details] [diff] [review]

I think tnikkel is enough here.
Comment 4 User image Andrew Quartey [:drexler] 2012-08-17 08:38:55 PDT
Comment 5 User image David Bolter [:davidb] 2012-08-17 08:40:55 PDT
Thank you!
Comment 6 User image Ryan VanderMeulen [:RyanVM] 2012-08-17 19:23:08 PDT

Note You need to log in before you can comment on or make changes to this bug.