Carsten Book [:Tomcat] Reporter
	Description • 11 years ago

WINNT 6.2 mozilla-inbound talos other on 2013-09-25 22:55:02 PDT for push 87b26cec9ee1

slave: t-w864-ix-030

https://tbpl.mozilla.org/php/getParsedLog.php?id=28389661&tree=Mozilla-Inbound

PROCESS-CRASH | a11yr | application crashed [@ mozilla::a11y::DocManager::AddListeners(nsIDocument *,bool)]

23:01:51     INFO -  PROCESS-CRASH | a11yr | application crashed [@ mozilla::a11y::DocManager::AddListeners(nsIDocument *,bool)]
23:01:51     INFO -  Crash dump filename: c:\users\cltbld~1.t-w\appdata\local\temp\tmpok5rrf\profile\minidumps\4ceb25fc-64f6-45c5-be61-e2eb6621a345.dmp
23:01:51     INFO -  Operating system: Windows NT
23:01:51     INFO -                    6.2.9200
23:01:51     INFO -  CPU: x86
23:01:51     INFO -       GenuineIntel family 6 model 30 stepping 5
23:01:51     INFO -       8 CPUs
23:01:51     INFO -  Crash reason:  EXCEPTION_ACCESS_VIOLATION_READ
23:01:51     INFO -  Crash address: 0x4
23:01:51     INFO -  Assertion: Unknown assertion type 0x00000000
23:01:51     INFO -  Thread 0 (crashed)
23:01:51     INFO -   0  xul.dll!mozilla::a11y::DocManager::AddListeners(nsIDocument *,bool) [DocManager.cpp:87b26cec9ee1 : 331 + 0x0]
23:01:51     INFO -      eip = 0x71f31dc0   esp = 0x0030f224   ebp = 0x0030f23c   ebx = 0x041fb8e0
23:01:51     INFO -      esi = 0x012275e0   edi = 0x041fb8e0   eax = 0x00000000   ecx = 0x00000000
23:01:51     INFO -      edx = 0x72cf1134   efl = 0x00010246
23:01:51     INFO -      Found by: given as instruction pointer in context
23:01:51     INFO -   1  xul.dll!mozilla::a11y::DocManager::CreateDocOrRootAccessible(nsIDocument *) [DocManager.cpp:87b26cec9ee1 : 439 + 0xa]
23:01:51     INFO -      eip = 0x71f321b4   esp = 0x0030f244   ebp = 0x0030f264
23:01:51     INFO -      Found by: call frame info
23:01:51     INFO -   2  xul.dll!mozilla::a11y::DocManager::GetDocAccessible(nsIDocument *) [DocManager.cpp:87b26cec9ee1 : 60 + 0x9]
23:01:51     INFO -      eip = 0x71f31bc6   esp = 0x0030f26c   ebp = 0x0030f274
23:01:51     INFO -      Found by: call frame info
23:01:51     INFO -   3  xul.dll!mozilla::a11y::DocManager::GetDocAccessible(nsIPresShell const *) [DocManager.h:87b26cec9ee1 : 52 + 0x7]
23:01:51     INFO -      eip = 0x71f34be8   esp = 0x0030f27c   ebp = 0x0030f280
23:01:51     INFO -      Found by: call frame info
23:01:51     INFO -   4  xul.dll!nsAccessibilityService::ContentRangeInserted(nsIPresShell *,nsIContent *,nsIContent *,nsIContent *) [nsAccessibilityService.cpp:87b26cec9ee1 : 381 + 0x9]
23:01:51     INFO -      eip = 0x71f350f3   esp = 0x0030f288   ebp = 0x0030f294
23:01:51     INFO -      Found by: call frame info
23:01:51     INFO -   5  xul.dll!nsCSSFrameConstructor::ContentRangeInserted(nsIContent *,nsIContent *,nsIContent *,nsILayoutHistoryState *,bool) [nsCSSFrameConstructor.cpp:87b26cec9ee1 : 6902 + 0x11]
23:01:51     INFO -      eip = 0x720e9969   esp = 0x0030f29c   ebp = 0x0030f458
23:01:51     INFO -      Found by: call frame info
23:01:51     INFO -   6  xul.dll!nsCSSFrameConstructor::ContentInserted(nsIContent *,nsIContent *,nsILayoutHistoryState *,bool) [nsCSSFrameConstructor.cpp:87b26cec9ee1 : 6766 + 0x15]
23:01:51     INFO -      eip = 0x720ea054   esp = 0x0030f460   ebp = 0x0030f474
23:01:51     INFO -      Found by: call frame info
23:01:51     INFO -   7  xul.dll!PresShell::Initialize(int,int) [nsPresShell.cpp:87b26cec9ee1 : 1675 + 0xd]
23:01:51     INFO -      eip = 0x7211bc3d   esp = 0x0030f47c   ebp = 0x0030f4c8
23:01:51     INFO -      Found by: call frame info
23:01:51     INFO -   8  xul.dll!nsContentSink::StartLayout(bool) [nsContentSink.cpp:87b26cec9ee1 : 1179 + 0xf]
23:01:51     INFO -      eip = 0x7224158c   esp = 0x0030f4d

	alexander :surkov (:asurkov)
	Comment 1 • 11 years ago

opposite to bug 881504 and bug 849496

seems the problem is we get called inside PresShell::Initialize(), the DOM document doesn't have window yet.

cc'ing Boris and Olli for ideas, just in case if layout does something that it's not supposed to do

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 2 • 11 years ago

Well, at least DocManager::AddListeners is wrong. It misses null checks for the various Get* methods
which may return null. (GetWindow() and GetChromeEventHandler(), GetListenerManager(true) will be renamed to ListenerManager() since it should always return non-null )

	alexander :surkov (:asurkov)
	Comment 3 • 11 years ago

right, you made same point in bug 881504 comment #3 (and we need it to fix same way), just I'm curious whether we should worry it returns null in this particular case.

	Ed Morley [:emorley]
	Comment 6 • 10 years ago

Closing bugs where TBPLbot has previously commented, but have now not been modified for >3 months & do not contain the whiteboard strings for disabled/annotated tests or use the keyword leave-open. Filter on: mass-intermittent-bug-closure-2014-07