Found while triaging crash stats. Happens with beta 5. http://crash-stats.mozilla.com/report/list?range_unit=weeks&query_search=signature&query_type=contains&product=Firefox&platform=windows%2Cmac%2Clinux&version=Firefox%3A3.0b5%2CFirefox%3A3.0pre&branch=1.9&signature=nsRootAccessible%3A%3AGetState(unsigned+int*%2C+unsigned+int*)&query=Access&range_value=1 Looks like we're using a focusController without checking if it is not NULL first. Will submit a patch soon.
Assignee: aaronleventhal → marco.zehe
Status: NEW → ASSIGNED
Attachment #316385 - Flags: review?(aaronleventhal)
Comment on attachment 316385 [details] [diff] [review] Simple NULL check How can it happen that there is no focus controller? That seems like a possible bad regression outside of a11y.
Comment on attachment 316385 [details] [diff] [review] Simple NULL check It's a safe "wallpaper fix"
Attachment #316385 - Flags: review?(aaronleventhal) → review+
Comment on attachment 316385 [details] [diff] [review] Simple NULL check a1.9=beltzner
Attachment #316385 - Flags: approval1.9? → approval1.9+
Wouldn't hold the release, but seems fixed anyway.
Flags: blocking1.9? → blocking1.9-
Checking in accessible/src/base/nsRootAccessible.cpp; /cvsroot/mozilla/accessible/src/base/nsRootAccessible.cpp,v <-- nsRootAccessible.cpp new revision: 1.268; previous revision: 1.267 done
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Crash Signature: [@ nsRootAccessible::GetState(unsigned int*, unsigned int*) ]
You need to log in before you can comment on or make changes to this bug.