Closed Bug 1293569 Opened 4 years ago Closed 3 years ago

Crash in PLDHashTable::Add | nsBaseHashtable<T>::Put | mozilla::a11y::AccessibleWrap::GetChildIDFor

Categories

(Core :: Disability Access APIs, defect, critical)

x86_64
Windows
defect
Not set
critical

Tracking

()

RESOLVED WORKSFORME
Tracking Status
firefox48 --- affected
firefox49 --- affected
firefox50 --- affected
firefox51 --- affected

People

(Reporter: ting, Unassigned)

Details

(Keywords: crash)

Crash Data

This bug was filed from the Socorro interface and is 
report bp-4fe297c3-3d82-47b8-90cd-a87f32160808.
=============================================================

#34 of 0807 Nightly on Windows, 3 crashes from 2 installations.
OS: Windows 10 → Windows
Hardware: Unspecified → x86_64
Based on the minidump, it seems the |doc| [1] is null. Maybe we need another similar check like [2] for |!aAccessible->DocProxyWrapper() && aAccessible->IsProxy()|?

[1] https://hg.mozilla.org/mozilla-central/annotate/d42aacfe34af/accessible/windows/msaa/AccessibleWrap.cpp#l1372
[2] https://hg.mozilla.org/mozilla-central/annotate/d42aacfe34af/accessible/windows/msaa/AccessibleWrap.cpp#l1360
Flags: needinfo?(tbsaunde+mozbugs)
(In reply to Ting-Yu Chou [:ting] from comment #1)
> Based on the minidump, it seems the |doc| [1] is null. Maybe we need another
> similar check like [2] for |!aAccessible->DocProxyWrapper() &&
> aAccessible->IsProxy()|?

I'm not sure how 1 can happen, but I don't think 2 should be possible, so I'd like to understand that before just adding that check.
Flags: needinfo?(tbsaunde+mozbugs)
also note that bug 1268544 and related work will change this around significantly, so I'm not sure how much its worth investigating until that's sorted out.
Crash volume for signature 'PLDHashTable::Add | nsBaseHashtable<T>::Put | mozilla::a11y::AccessibleWrap::GetChildIDFor':
 - nightly (version 51): 66 crashes from 2016-08-01.
 - aurora  (version 50): 27 crashes from 2016-08-01.
 - beta    (version 49): 34 crashes from 2016-08-02.
 - release (version 48): 22 crashes from 2016-07-25.
 - esr     (version 45): 0 crashes from 2016-05-02.

Crash volume on the last weeks (Week N is from 08-22 to 08-28):
            W. N-1  W. N-2  W. N-3
 - nightly      32      25       7
 - aurora        9       7       4
 - beta          6      12       1
 - release       8       5       5
 - esr           0       0       0

Affected platform: Windows

Crash rank on the last 7 days:
           Browser     Content   Plugin
 - nightly #61
 - aurora  #78
 - beta    #862
 - release #2392
 - esr
I've also started to get this crash today. Nothing has changed in terms of hardware or software, so I'm not entirely sure why it has started so suddenly.

Here are the crash reports:
bp-f30dc8f2-62cd-4568-bf56-35a502161017
bp-08609246-9488-4c92-a01b-4cff12161016
bp-bc6c1fdc-652e-4abc-be9e-8a8b62161016
You need to log in before you can comment on or make changes to this bug.