723360 - crash nsMsgAccountManager::removeListenerFromFolder

Status: RESOLVED FIXED

(Thunderbird :: Account Manager, defect, critical)

Description

[Makoto Kato [:m_kato]]

rootFolder into nsMsgAccountManager::hashUnloadServe is null...


This bug was filed from the Socorro interface and is 
report bp-a510d98c-09ba-414e-8d5a-93a182120131 .
============================================================= 
0 	xul.dll 	nsMsgAccountManager::removeListenerFromFolder 	mailnews/base/src/nsMsgAccountManager.cpp:631
1 	xul.dll 	nsSupportsArray::EnumerateForwards 	xpcom/ds/nsSupportsArray.cpp:627
2 	xul.dll 	nsMsgAccountManager::hashUnloadServer 	mailnews/base/src/nsMsgAccountManager.cpp:885
3 	xul.dll 	nsBaseHashtable<nsCStringHashKey,nsAutoPtr<mozilla::scache::CacheEntry>,mozilla::scache::CacheEntry*>::s_EnumStub 	objdir-tb/mozilla/dist/include/nsBaseHashtable.h:364
4 	xul.dll 	PL_DHashTableEnumerate 	objdir-tb/mozilla/xpcom/build/pldhash.cpp:755
5 	xul.dll 	nsBaseHashtable<nsStringHashKey,mozJSComponentLoader::ModuleEntry*,mozJSComponentLoader::ModuleEntry*>::Enumerate 	objdir-tb/mozilla/dist/include/nsBaseHashtable.h:239
6 	xul.dll 	nsMsgAccountManager::UnloadAccounts 	mailnews/base/src/nsMsgAccountManager.cpp:1582
7 	xul.dll 	NS_InvokeByIndex_P 	xpcom/reflect/xptcall/src/md/win32/xptcinvoke.cpp:102
8 	xul.dll 	XPCWrappedNative::CallMethod 	js/xpconnect/src/XPCWrappedNative.cpp:2155
9 	xul.dll 	XPC_WN_CallMethod 	js/xpconnect/src/XPCWrappedNativeJSOps.cpp:1554
10 	mozjs.dll 	js::InvokeKernel 	js/src/jsinterp.cpp:629
11 	mozjs.dll 	js::Interpret 	js/src/jsinterp.cpp:3948
12 	mozjs.dll 	js::RunScript 	js/src/jsinterp.cpp:584
13 	mozjs.dll 	js::InvokeKernel 	js/src/jsinterp.cpp:647
14 	mozjs.dll 	js::Invoke 	js/src/jsinterp.cpp:679
15 	mozjs.dll 	JS_CallFunctionValue 	js/src/jsapi.cpp:5199
16 	xul.dll 	nsXPCWrappedJSClass::CallMethod 	js/xpconnect/src/XPCWrappedJSClass.cpp:1530
17 	xul.dll 	nsXPCWrappedJS::CallMethod 	js/xpconnect/src/XPCWrappedJS.cpp:611
18 	xul.dll 	PrepareAndDispatch 	xpcom/reflect/xptcall/src/md/win32/xptcstubs.cpp:117
19 	xul.dll 	SharedStub 	xpcom/reflect/xptcall/src/md/win32/xptcstubs.cpp:144
20 	xul.dll 	nsEventListenerManager::HandleEventSubType 	content/events/src/nsEventListenerManager.cpp:736
21 	xul.dll 	nsEventListenerManager::HandleEventInternal 	content/events/src/nsEventListenerManager.cpp:795

Comment 1

[Wayne Mery (:wsmwk)]

pretty rare - only 10 per month for all releases

Comment 2

[Makoto Kato [:m_kato]]

Attachment: fix

Comment 3

[David :Bienvenu]

Comment on attachment 597285 [details] [diff] [review]
fix

thx for the patch, looks reasonable - can you add a space between if and (NS_SUCCEEDED) ?

Comment 4

[Makoto Kato [:m_kato]]

http://hg.mozilla.org/comm-central/rev/81e0d1fa9127