Last Comment Bug 723360 - crash nsMsgAccountManager::removeListenerFromFolder
: crash nsMsgAccountManager::removeListenerFromFolder
Status: RESOLVED FIXED
: crash
Product: Thunderbird
Classification: Client Software
Component: Account Manager (show other bugs)
: Trunk
: x86 Windows NT
: -- critical (vote)
: Thunderbird 13.0
Assigned To: Makoto Kato [:m_kato]
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-02-01 17:08 PST by Makoto Kato [:m_kato]
Modified: 2012-02-15 18:06 PST (History)
4 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
fix (1.30 KB, patch)
2012-02-14 20:01 PST, Makoto Kato [:m_kato]
mozilla: review+
Details | Diff | Splinter Review

Description Makoto Kato [:m_kato] 2012-02-01 17:08:14 PST
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, NI for questions) 2012-02-01 18:54:41 PST
pretty rare - only 10 per month for all releases
Comment 2 Makoto Kato [:m_kato] 2012-02-14 20:01:33 PST
Created attachment 597285 [details] [diff] [review]
fix
Comment 3 David :Bienvenu 2012-02-15 15:37:10 PST
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] 2012-02-15 18:06:41 PST
http://hg.mozilla.org/comm-central/rev/81e0d1fa9127

Note You need to log in before you can comment on or make changes to this bug.