Crash in AccessibleHandler::CleanupDynamicIA2Data with Vispero's ZoomText or Fusion 2021 Public Beta
Categories
(Core :: Disability Access APIs, defect, P1)
Tracking
()
People
(Reporter: MarcoZ, Assigned: Jamie)
References
(Regression)
Details
(Keywords: crash, regression)
Attachments
(2 files)
(deleted),
text/html
|
Details | |
(deleted),
text/x-phabricator-request
|
jcristau
:
approval-mozilla-beta+
jcristau
:
approval-mozilla-esr78+
|
Details |
[Tracking Requested - why for this release]:
Regression introduced in 78, crash with external software.
Steps:
- Download the Fusion Public Beta from this page and install it.
- After restarting your computer, start Fusion first (is normally started automatically the first time after an install), and then Firefox. This order is important.
- Open the attached test case. This was sent to us by an engineer at Vispero, and I was able to reproduce the crash with it.
- Tab to the first input box and start typing.
- Result: A parent process crash with reports like this will be submitted.
The people at Vispero were seeing this crash with Firefox 80, I was able to reproduce this with the 83 Nightly build as well.
This is another regression from bug 1640553.
Reporter | ||
Comment 1•4 years ago
|
||
See steps above.
Assignee | ||
Comment 2•4 years ago
|
||
Marco, does this try build (only just kicked off) help? (I haven't tested myself; this is a speculative fix.)
Reporter | ||
Comment 3•4 years ago
|
||
Unfortunately, this build still crashes with Fusion and the test case. If it is of any use: here is the report for that crash.
Assignee | ||
Comment 4•4 years ago
|
||
Updated•4 years ago
|
Reporter | ||
Comment 5•4 years ago
|
||
The second try build fixes the crash for me.
Assignee | ||
Comment 6•4 years ago
|
||
Updated•4 years ago
|
Comment 8•4 years ago
|
||
bugherder |
Comment 9•4 years ago
|
||
The patch landed in nightly and beta is affected.
:Jamie, is this bug important enough to require an uplift?
If not please set status_beta
to wontfix
.
For more information, please visit auto_nag documentation.
Reporter | ||
Comment 10•4 years ago
|
||
Verified in Nightly 83.0a1 20200929213959.
Assignee | ||
Comment 11•4 years ago
|
||
Comment on attachment 9177517 [details]
Bug 1666720: AccessibleHandler: When refreshing the cache, fetch into a temporary struct rather than into the live cache to avoid problems triggered by COM re-entry.
Beta/Release Uplift Approval Request
- User impact if declined: Crashes with ZoomText or ZoomText Fusion 2021 assistive technology products (currently in public beta).
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Only affects Windows accessibility. Straightforward patch which updates a complete cache in two stages instead of one.
- String changes made/needed: None.
Comment 12•4 years ago
|
||
Comment on attachment 9177517 [details]
Bug 1666720: AccessibleHandler: When refreshing the cache, fetch into a temporary struct rather than into the live cache to avoid problems triggered by COM re-entry.
a11y crash fix, approved for 82.0b6
Comment 13•4 years ago
|
||
bugherder uplift |
Assignee | ||
Comment 15•4 years ago
|
||
Comment on attachment 9177517 [details]
Bug 1666720: AccessibleHandler: When refreshing the cache, fetch into a temporary struct rather than into the live cache to avoid problems triggered by COM re-entry.
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: Crashes.
- User impact if declined: Crashes with ZoomText or ZoomText Fusion 2021 assistive technology products (currently in public beta).
- Fix Landed on Version: 82, 83
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Only affects Windows accessibility. Straightforward patch which updates a complete cache in two stages instead of one.
- String or UUID changes made by this patch: None.
Updated•4 years ago
|
Comment 16•4 years ago
|
||
Comment on attachment 9177517 [details]
Bug 1666720: AccessibleHandler: When refreshing the cache, fetch into a temporary struct rather than into the live cache to avoid problems triggered by COM re-entry.
approved for 78.5esr
Comment 17•4 years ago
|
||
bugherder uplift |
Updated•4 years ago
|
Description
•