Windows 10 Fission Crash in [@ nsAccessibilityService::GetMarkupMapInfoForNode]
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
People
(Reporter: aryx, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: crash)
Crash Data
All crashes on Windows 10 with Fisison enabled, oldest build ID in the reports is 89.0a1 20210414160838. For last week, we have 7 Nightly crashes from 5 installations.
Crash report: https://crash-stats.mozilla.org/report/index/e0800b37-7337-4764-93ab-046560210605
Reason: EXCEPTION_ACCESS_VIOLATION_READ
Top 10 frames of crashing thread:
0 xul.dll nsAccessibilityService::GetMarkupMapInfoForNode const accessible/base/nsAccessibilityService.h:367
1 xul.dll mozilla::a11y::HyperTextAccessible::NativeRole const accessible/generic/HyperTextAccessible.cpp:176
2 xul.dll mozilla::a11y::LocalAccessible::Role const accessible/generic/LocalAccessible-inl.h:26
3 xul.dll static mozilla::a11y::nsAccUtils::MustPrune accessible/base/nsAccUtils.cpp:443
4 xul.dll mozilla::a11y::MsaaAccessible::get_accChildCount accessible/windows/msaa/MsaaAccessible.cpp:796
5 xul.dll mozilla::a11y::HandlerProvider::BuildDynamicIA2Data accessible/ipc/win/HandlerProvider.cpp:307
6 xul.dll mozilla::detail::RunnableMethodImpl<mozilla::a11y::HandlerProvider*, void xpcom/threads/nsThreadUtils.h:1203
7 xul.dll static mozilla::mscom::MainThreadInvoker::MainThreadAPC ipc/mscom/MainThreadInvoker.cpp:172
8 ntdll.dll RtlDispatchAPC
9 ntdll.dll KiUserApcDispatch
Comment 1•3 years ago
|
||
I'm not sure how this could happen, but based on the fact that these are all Windows 10 Fission, I wonder if we're somehow calling accChildCount on a RemoteIframeDocRemoteAccessibleWrap (used for OOP iframes). That would have a null mContent, which would result in this crash.
If that's the case, this should be fixed by bug 1715230. Let's see what happens when the patches for that land.
Comment 2•3 years ago
|
||
The crash doesn't look like a Fission crash, but let's keep an eye on it. Tracking for Fission MVP.
Comment 3•3 years ago
|
||
I confirmed this is definitely a Fission crash as I suspected in comment 1. The crash dump shows that the accessible we're looking at here has eProxyType, and in a content process, that could only be RemoteIframeDocRemoteAccessibleWrap, which is only used for OOP iframes.
Comment 4•3 years ago
|
||
Jamie, does someone on the Accessibility team have time to investigate this crash?
This crash's volume is pretty low, so maybe this bug doesn't need to block Fission MVP? We saw 40 crashes from 16 installations from Nightly 91, but only 6 crashes from 2 installations in the first week of Beta 91.
Comment 5•3 years ago
|
||
I'm hoping this is fixed by bug 1715230, which has only just landed. I suggest we monitor to see if the crash goes away with this patch.
Comment 6•3 years ago
|
||
No crashes in 92 recently, so I think it's safe to say this is fixed by bug 1715230 (which landed in 92).
Updated•3 years ago
|
Updated•3 years ago
|
Description
•