612068 - crash [@ nsParseNewMailState::ApplyFilters(int*, nsIMsgWindow*, unsigned int)]

Status: RESOLVED WORKSFORME

(MailNews Core :: Filters, defect)

Description

[Wayne Mery (:wsmwk)]

crash [@ nsParseNewMailState::ApplyFilters(int*, nsIMsgWindow*, unsigned int)]
bp-b4f48027-177e-449c-9a19-a70932101107
0 	thunderbird.exe 	nsParseNewMailState::ApplyFilters 	mailnews/local/src/nsParseMailbox.cpp:1954
1 	thunderbird.exe 	nsPop3Sink::IncorporateComplete 	mailnews/local/src/nsPop3Sink.cpp:870
2 	thunderbird.exe 	nsPop3Protocol::HandleLine 	mailnews/local/src/nsPop3Protocol.cpp:3547
3 	thunderbird.exe 	nsPop3Protocol::RetrResponse 	mailnews/local/src/nsPop3Protocol.cpp:3333
4 	thunderbird.exe 	nsPop3Protocol::ProcessProtocolState 	mailnews/local/src/nsPop3Protocol.cpp:3954
5 	thunderbird.exe 	nsMsgProtocol::OnDataAvailable 	mailnews/base/util/nsMsgProtocol.cpp:359
6 	thunderbird.exe 	nsInputStreamPump::OnStateTransfer 	netwerk/base/src/nsInputStreamPump.cpp:510
7 	thunderbird.exe 	nsInputStreamPump::OnInputStreamReady 	netwerk/base/src/nsInputStreamPump.cpp:400

Comment 1

[Kent James (:rkent)]

I would guess that this is occurring during the destruction of either msgHdr or downloadFolder (with the former more likely). It's not clear why we need this msgHdr variable in the first place, since we could just as easily use m_newMsgHdr  But this section has had previous crash work done on it (CVS version 1.87. "Fix a crash when running a local mail filter --> if you moved the message and then had another filter fire after we moved the message we would crash.") I would be afraid that trying to fix this without a clear understanding of the root cause would be as likely to cause new problems as to fix this one.

Comment 2

[Wayne Mery (:wsmwk)]

no crashes in recent versions, so WFM.  

(one guy had several in TB9 bp-b5f7bf5a-b439-473b-8b84-8af892120125 and a few in TB8 dcc4143f-eb01-4e3d-9a84-9049d2111130 )