Closed
Bug 678189
Opened 13 years ago
Closed 13 years ago
Crash [@ RelatedAccIterator::RelatedAccIterator(nsDocAccessible*, nsIContent*, nsIAtom*) ]
Categories
(Core :: Disability Access APIs, defect)
Tracking
()
RESOLVED
FIXED
mozilla8
People
(Reporter: MarcoZ, Assigned: tbsaunde)
References
Details
(Keywords: crash, regression)
Crash Data
Attachments
(1 file, 1 obsolete file)
783 bytes,
patch
|
Details | Diff | Splinter Review |
First appears in the August 11, 2011 nightly build and is a regression from bug 641838 (the AccessibleRelation dexpcom refactor). I can reproduce this at will by: 1. with NVDA running, starting the Nightly. 2. Pressing Ctrl+L to go to the Location bar. 3. Starting to type something like about:buildconfig. After one, two letters, probably as soon as the list appears, I get a crash. Report: https://crash-stats.mozilla.com/report/index/bp-a152e4e3-ebb8-4328-8e23-c42a72110811
Reporter | ||
Updated•13 years ago
|
Crash Signature: [@ RelatedAccIterator::RelatedAccIterator(nsDocAccessible*, nsIContent*, nsIAtom*) ] → RelatedAccIterator::RelatedAccIterator(nsDocAccessible*, nsIContent*, nsIAtom*)
Assignee | ||
Comment 1•13 years ago
|
||
aI'm a little skeptical this is the issue Marco had based on the stack, but that stack isn't really good enough to denie anything, and I'm pretty sure we want this patch anyway so, which ever of you gets to it first or both of you if you like :)
Attachment #552482 -
Flags: review?(surkov.alexander)
Attachment #552482 -
Flags: review?(bolterbugz)
Comment 2•13 years ago
|
||
Comment on attachment 552482 [details] [diff] [review] a thought r=me (code inspection only)
Attachment #552482 -
Flags: review?(bolterbugz) → review+
Assignee | ||
Comment 3•13 years ago
|
||
try builds for that patch should show up at http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/trev.saunders@gmail.com-66a378ab0860.
Comment 4•13 years ago
|
||
Comment on attachment 552482 [details] [diff] [review] a thought yes, this is right. Until we have good stack this is reasonable guess because IsDefunct() check was got rid.
Attachment #552482 -
Flags: review?(surkov.alexander) → review+
Comment 5•13 years ago
|
||
Comment on attachment 552482 [details] [diff] [review] a thought > VariantInit(pvarEndUpAt); >+ nsAccessible *xpAccessibleStart = GetXPAccessibleFor(varStart); >+ if (!xpAccessibleStart || IsDefunct()) >+ return E_FAIL; > initialize variant after you are use in correct accessible
Updated•13 years ago
|
Assignee: nobody → trev.saunders
Assignee | ||
Comment 6•13 years ago
|
||
is this the whitespace you want around variant_init()?
Attachment #552482 -
Attachment is obsolete: true
Assignee | ||
Comment 7•13 years ago
|
||
landed on http://hg.mozilla.org/integration/mozilla-inbound/rev/febd4e26179c
Reporter | ||
Comment 8•13 years ago
|
||
From my testing the try-server build, it appears that this bug fixes the crash.
Comment 9•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/febd4e26179c
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•13 years ago
|
Target Milestone: --- → mozilla8
Updated•13 years ago
|
Crash Signature: RelatedAccIterator::RelatedAccIterator(nsDocAccessible*, nsIContent*, nsIAtom*) → [@ RelatedAccIterator::RelatedAccIterator(nsDocAccessible*, nsIContent*, nsIAtom*) ]
You need to log in
before you can comment on or make changes to this bug.
Description
•