Closed
Bug 783383
Opened 12 years ago
Closed 12 years ago
crash in nsBaseWidget::GetAccessible
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
mozilla17
People
(Reporter: scoobidiver, Assigned: drexler)
References
Details
(Keywords: crash, regression, topcrash, Whiteboard: [qa-])
Crash Data
Attachments
(1 file)
(deleted),
patch
|
tnikkel
:
review+
|
Details | Diff | Splinter Review |
It first appeared in 17.0a1/20120816. The regression range is:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=7d4268f8884c&tochange=50e4ff05741e
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 Reason EXCEPTION_ACCESS_VIOLATION_READ
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:
https://crash-stats.mozilla.com/report/list?signature=nsBaseWidget%3A%3AGetAccessible%28%29
https://crash-stats.mozilla.com/report/list?signature=nsBaseWidget%3A%3AGetAccessible
Assignee | ||
Comment 1•12 years ago
|
||
The null check in static nsIPresShell* GetPresShell(nsIWidget* aWidget, void* clientData) was removed by this revision: http://hg.mozilla.org/mozilla-central/rev/4aca82dc0d41. Furthermore, not all places that use mWidgetListener were null-checked thereafter.
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #652605 -
Flags: review?(trev.saunders)
Attachment #652605 -
Flags: review?(tnikkel)
Updated•12 years ago
|
Attachment #652605 -
Flags: review?(tnikkel) → review+
Reporter | ||
Updated•12 years ago
|
tracking-firefox17:
--- → ?
Keywords: topcrash
Comment 3•12 years ago
|
||
Comment on attachment 652605 [details] [diff] [review]
patch
I think tnikkel is enough here.
Attachment #652605 -
Flags: review?(trev.saunders)
Assignee | ||
Comment 4•12 years ago
|
||
Status: NEW → ASSIGNED
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → andrew.quartey
Comment 5•12 years ago
|
||
Thank you!
Comment 6•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
Updated•12 years ago
|
status-firefox17:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•