123017 - Crash opening mail [@ nsMsgFolderDataSource::createFolderOpenNode]

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect, P1)

Description

[scottputterman]

This is showing up as a topcrash.  I've never seen it but apparently a bunch of
people have. The first stack is from 1/24 and the most recent is from the 1/31
build.

Here are some talkback id's:

  2373302
  2329687  Attemmpting to start mail via tasks
  2223035  starting Netscape. First go after a new nightly install.

Here's the stack trace:

nsMsgFolderDataSource::createFolderOpenNode
[d:\builds\seamonkey\mozilla\mailnews\base\src\nsMsgFolderDataSource.cpp, line 1325]
nsMsgFolderDataSource::createFolderNode
[d:\builds\seamonkey\mozilla\mailnews\base\src\nsMsgFolderDataSource.cpp, line 1011]
nsMsgFolderDataSource::GetTarget
[d:\builds\seamonkey\mozilla\mailnews\base\src\nsMsgFolderDataSource.cpp, line 384]
GetTargetHasAssertion
[d:\builds\seamonkey\mozilla\mailnews\base\src\nsMsgRDFUtils.cpp, line 103]
nsMsgFolderDataSource::DoFolderHasAssertion
[d:\builds\seamonkey\mozilla\mailnews\base\src\nsMsgFolderDataSource.cpp, line 2086]
nsMsgFolderDataSource::HasAssertion
[d:\builds\seamonkey\mozilla\mailnews\base\src\nsMsgFolderDataSource.cpp, line 525]
nsXULOutlinerBuilder::IsContainerOpen
[d:\builds\seamonkey\mozilla\content\xul\templates\src\nsXULOutlinerBuilder.cpp,
line 1730]
nsXULOutlinerBuilder::OpenSubtreeOf
[d:\builds\seamonkey\mozilla\content\xul\templates\src\nsXULOutlinerBuilder.cpp,
line 1566]
nsXULOutlinerBuilder::OpenContainer
[d:\builds\seamonkey\mozilla\content\xul\templates\src\nsXULOutlinerBuilder.cpp,
line 1481]
nsXULOutlinerBuilder::Rebuild
[d:\builds\seamonkey\mozilla\content\xul\templates\src\nsXULOutlinerBuilder.cpp,
line 987]
nsXULTemplateBuilder::AttributeChanged
[d:\builds\seamonkey\mozilla\content\xul\templates\src\nsXULTemplateBuilder.cpp,
line 359]
nsXULDocument::AttributeChanged
[d:\builds\seamonkey\mozilla\content\xul\document\src\nsXULDocument.cpp, line 2083]
nsXULElement::SetAttr
[d:\builds\seamonkey\mozilla\content\xul\content\src\nsXULElement.cpp, line 2676]
nsXULElement::SetAttribute
[d:\builds\seamonkey\mozilla\content\xul\content\src\nsXULElement.cpp, line 1331]
XPTC_InvokeByIndex
[d:\builds\seamonkey\mozilla\xpcom\reflect\xptcall\src\md\win32\xptcinvoke.cpp,
line 106]
XPCWrappedNative::CallMethod
[d:\builds\seamonkey\mozilla\js\src\xpconnect\src\xpcwrappednative.cpp, line 2000]
XPC_WN_CallMethod
[d:\builds\seamonkey\mozilla\js\src\xpconnect\src\xpcwrappednativejsops.cpp,
line 1267]
js_Invoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 834]
js_Interpret [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 2803]
js_Invoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 850]
js_InternalInvoke [d:\builds\seamonkey\mozilla\js\src\jsinterp.c, line 925]
JS_CallFunctionValue [d:\builds\seamonkey\mozilla\js\src\jsapi.c, line 3407]
nsJSContext::CallEventHandler
[d:\builds\seamonkey\mozilla\dom\src\base\nsJSEnvironment.cpp, line 1019]
nsJSEventListener::HandleEvent
[d:\builds\seamonkey\mozilla\dom\src\events\nsJSEventListener.cpp, line 182]
nsEventListenerManager::HandleEventSubType
[d:\builds\seamonkey\mozilla\content\events\src\nsEventListenerManager.cpp, line
1206]
nsEventListenerManager::HandleEvent
[d:\builds\seamonkey\mozilla\content\events\src\nsEventListenerManager.cpp, line
1881]
GlobalWindowImpl::HandleDOMEvent
[d:\builds\seamonkey\mozilla\dom\src\base\nsGlobalWindow.cpp, line 639]
DocumentViewerImpl::LoadComplete
[d:\builds\seamonkey\mozilla\content\base\src\nsDocumentViewer.cpp, line 1256]
nsDocShell::EndPageLoad
[d:\builds\seamonkey\mozilla\docshell\base\nsDocShell.cpp, line 3449]
nsWebShell::EndPageLoad
[d:\builds\seamonkey\mozilla\docshell\base\nsWebShell.cpp, line 695]
nsDocShell::OnStateChange
[d:\builds\seamonkey\mozilla\docshell\base\nsDocShell.cpp, line 3357]
nsDocLoaderImpl::FireOnStateChange
[d:\builds\seamonkey\mozilla\uriloader\base\nsDocLoader.cpp, line 1110]
nsDocLoaderImpl::doStopDocumentLoad
[d:\builds\seamonkey\mozilla\uriloader\base\nsDocLoader.cpp, line 749]
nsDocLoaderImpl::DocLoaderIsEmpty
[d:\builds\seamonkey\mozilla\uriloader\base\nsDocLoader.cpp, line 647]
nsDocLoaderImpl::OnStopRequest
[d:\builds\seamonkey\mozilla\uriloader\base\nsDocLoader.cpp, line 578]
nsLoadGroup::RemoveRequest
[d:\builds\seamonkey\mozilla\netwerk\base\src\nsLoadGroup.cpp, line 531]
imgRequestProxy::OnStopRequest
[d:\builds\seamonkey\mozilla\modules\libpr0n\src\imgRequestProxy.cpp, line 389]
imgRequest::OnStopRequest
[d:\builds\seamonkey\mozilla\modules\libpr0n\src\imgRequest.cpp, line 620]
ProxyListener::OnStopRequest
[d:\builds\seamonkey\mozilla\modules\libpr0n\src\imgLoader.cpp, line 582]
nsJARChannel::OnStopRequest
[d:\builds\seamonkey\mozilla\netwerk\protocol\jar\src\nsJARChannel.cpp, line 614]
nsOnStopRequestEvent::HandleEvent
[d:\builds\seamonkey\mozilla\netwerk\base\src\nsRequestObserverProxy.cpp, line 213]
PL_HandleEvent [d:\builds\seamonkey\mozilla\xpcom\threads\plevent.c, line 591]
PL_ProcessPendingEvents [d:\builds\seamonkey\mozilla\xpcom\threads\plevent.c,
line 524]
_md_EventReceiverProc [d:\builds\seamonkey\mozilla\xpcom\threads\plevent.c, line
1072]
nsAppShellService::Run
[d:\builds\seamonkey\mozilla\xpfe\appshell\src\nsAppShellService.cpp, line 308]
main1 [d:\builds\seamonkey\mozilla\xpfe\bootstrap\nsAppRunner.cpp, line 1301]
main [d:\builds\seamonkey\mozilla\xpfe\bootstrap\nsAppRunner.cpp, line 1628]
WinMain [d:\builds\seamonkey\mozilla\xpfe\bootstrap\nsAppRunner.cpp, line 1646]
WinMainCRTStartup()
KERNEL32.DLL + 0x17d08 (0x77e87d08)

Comment 1

[Navin Gupta]

taking, I have recently worked in this code. 

Comment 2

[scottputterman]

*** Bug 121652 has been marked as a duplicate of this bug. ***

Comment 3

[Ludovic Hirlimann]

from #2 Platform is all OS should be all !
Why is severity set to normal ? I thought crahers should be Critical.

Comment 4

[Navin Gupta]

Attachment: proposed fix

sorry, my bad. backing out some minor optimizations I checked in. looking
at it again, they are not needed. david can you r/sr.

Comment 5

[David :Bienvenu]

Comment on attachment 67491 [details] [diff] [review]
proposed fix

r=bienvenu

Comment 6

[David :Bienvenu]

Comment on attachment 67491 [details] [diff] [review]
proposed fix

r=bienvenu

Comment 7

[Navin Gupta]

fixed

Comment 8

[[e-mail] (gone - use [e-mail] instead)]

http://climate/reports/searchstacksignature.cfm?stacksig=nsMsgFolderDataSource%3A%3AcreateFolderOpenNode

The last time this happened on the trunk was  2002020110   MozillaTrunk  
Windows 98 4.10 build 67766446

(The day of the fix).

Marking verified FIXED.