1744150 - Crash in [@ mozilla::a11y::TextAttrsMgr::GetAttributes]

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Andrew McCreight [:mccr8]]

Crash report: https://crash-stats.mozilla.org/report/index/941eb967-e446-406e-8a42-399470211202

Reason: EXC_BAD_ACCESS / KERN_INVALID_ADDRESS

Top 10 frames of crashing thread:

0 XUL mozilla::a11y::TextAttrsMgr::GetAttributes accessible/base/TextAttrs.cpp:71
1 XUL mozilla::a11y::TextLeafPoint::GetTextAttributesLocalAcc const accessible/base/TextLeafRange.cpp:823
2 XUL mozilla::a11y::LocalAccessible::BundleFieldsForCache accessible/generic/LocalAccessible.cpp:3222
3 XUL mozilla::a11y::DocAccessibleChildBase::InsertIntoIpcTree accessible/ipc/DocAccessibleChildBase.cpp:87
4 XUL mozilla::a11y::DocAccessible::DoInitialUpdate accessible/generic/DocAccessible.cpp:1528
5 XUL mozilla::a11y::DocAccessibleWrap::DoInitialUpdate accessible/mac/DocAccessibleWrap.mm:102
6 XUL mozilla::a11y::NotificationController::WillRefresh accessible/base/NotificationController.cpp:653
7 XUL nsRefreshDriver::Tick layout/base/nsRefreshDriver.cpp:2311
8 XUL mozilla::RefreshDriverTimer::TickRefreshDrivers layout/base/nsRefreshDriver.cpp:331
9 XUL mozilla::RefreshDriverTimer::Tick layout/base/nsRefreshDriver.cpp:347

Null deref. Maybe mHyperTextAcc is null?

Comment 1

[James Teh [:Jamie]]

Somehow, a TextLeafAccessible, HTMLListBulletAccessible or HTMLBRAccessible doesn't have a HyperTextAccessible parent. I didn't think that should be possible any more, but apparently it is. I guess we'll have to wallpaper this but leave an assertion there in hopes of catching the reason for this in future.

Comment 2

[James Teh [:Jamie]]

Marking s4 because this is behind a pref which is disabled by default on all channels.

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1730096

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1730096

Comment 5

[Ryan VanderMeulen [:RyanVM]]

Not seeing any crashes on Beta97. Are we thinking bug 1747164 fixed this?

Comment 6

[James Teh [:Jamie]]

I thought bug 1747164 would fix this, but I wasn't 100% certain. Given that the crashes have gone away, I'd say it's almost certain bug 1747164 fixed it.