440185 - crash importing outlook email [@ nsMsgLocalMailFolder::IsChildOfTrash]

Status: RESOLVED FIXED

(MailNews Core :: Import, defect)

Description

[Wayne Mery (:wsmwk)]

nsMsgLocalMailFolder::IsChildOfTrash

at least two such crashes on talkback

TB42862637
Stack Signature	nsMsgLocalMailFolder::IsChildOfTrash 2c61f030
Product ID	Thunderbird2
Build ID	2008021305
\Platform	Win32
Operating System	Windows NT 5.1 build 2600
Module	thunderbird.exe + (0050e67a)
URL visited	
User Comments	Trying to import e-mail from Outlook
Source File, Line No.	e:/builds/tinderbox/Tb-Mozilla1.8-Release/WINNT_5.0_Depend/mozilla/mailnews/local/src/nsLocalMailFolder.cpp
Stack Trace 	
nsMsgLocalMailFolder::IsChildOfTrash  [mozilla/mailnews/local/src/nsLocalMailFolder.cpp, line 1011]
nsMsgLocalMailFolder::CopyFolderLocal  [mozilla/mailnews/local/src/nsLocalMailFolder.cpp, line 1967]
nsMsgLocalMailFolder::CopyFolder  [mozilla/mailnews/local/src/nsLocalMailFolder.cpp, line 1950]
nsMsgFolderDataSource::DoFolderCopyToFolder  [mozilla/mailnews/base/src/nsMsgFolderDataSource.cpp, line 2068]
nsMsgFolderDataSource::DoCommand  [mozilla/mailnews/base/src/nsMsgFolderDataSource.cpp, line 865]
CompositeDataSourceImpl::DoCommand  [mozilla/rdf/base/src/nsCompositeDataSource.cpp, line 1337]
nsMessenger::CopyFolders  [mozilla/mailnews/base/src/nsMessenger.cpp, line 1490]
XPTC_InvokeByIndex  [mozilla/xpcom/reflect/xptcall/src/md/win32/xptcinvoke.cpp, line 102]
XPCWrappedNative::CallMethod  [mozilla/js/src/xpconnect/src/xpcwrappednative.cpp, line 2169]
XPC_WN_CallMethod  [mozilla/js/src/xpconnect/src/xpcwrappednativejsops.cpp, line 1455]
js_Invoke  [mozilla/js/src/jsinterp.c, line 1379]
js_Interpret  [mozilla/js/src/jsinterp.c, line 3950]
js_Invoke  [mozilla/js/src/jsinterp.c, line 1398]
nsXPCWrappedJSClass::CallMethod  [mozilla/js/src/xpconnect/src/xpcwrappedjsclass.cpp, line 1453]

Comment 1

[timeless]

Attachment: parent folder can be null

Comment 2

[David :Bienvenu]

Comment on attachment 325655 [details] [diff] [review]
parent folder can be null

yes, it can be null (though this is worthy of an assertion, I think)

Comment 3

[Magnus Melin [:mkmelin]]

Adding blocking flag so this get's checked in for a2. (timeless: let us know if you want someone to do it for you)

Comment 4

[timeless]

Comment on attachment 325655 [details] [diff] [review]
parent folder can be null

mozilla/mailnews/local/src/nsLocalMailFolder.cpp 	1.595

assertion = crash in the new world, so no go :(