crash [@ nsMsgDBFolder::OnMessageClassified via bayesian analysis, with aMsgURI null?

NEW
Unassigned

Status

--
critical
8 years ago
20 days ago

People

(Reporter: wsmwk, Unassigned)

Tracking

({crash})

1.9.2 Branch
x86
All
crash

Firefox Tracking Flags

(Not tracked)

Details

(crash signature)

(Reporter)

Description

8 years ago
crash [@ nsMsgDBFolder::OnMessageClassified(char const*, unsigned int, unsigned int)]

bp-801df54b-0f78-4738-aee0-73bb02101126
EXCEPTION_ACCESS_VIOLATION_READ
0x0
0	thunderbird.exe	nsMsgDBFolder::OnMessageClassified	mailnews/base/util/nsMsgDBFolder.cpp:2247
1	thunderbird.exe	nsMsgLocalMailFolder::OnMessageClassified	mailnews/local/src/nsLocalMailFolder.cpp:3669
2	thunderbird.exe	nsBayesianFilter::classifyMessage	mailnews/extensions/bayesian-spam-filter/src/nsBayesianFilter.cpp:1787
3	thunderbird.exe	MessageClassifier::analyzeTokens	mailnews/extensions/bayesian-spam-filter/src/nsBayesianFilter.cpp:1367
4	thunderbird.exe	TokenStreamListener::OnStopRequest	mailnews/extensions/bayesian-spam-filter/src/nsBayesianFilter.cpp:1208
5	thunderbird.exe	nsStreamConverter::OnStopRequest	mailnews/mime/src/nsStreamConverter.cpp:1116
6	thunderbird.exe	nsMsgProtocol::OnStopRequest	mailnews/base/util/nsMsgProtocol.cpp:401
7	thunderbird.exe	nsMailboxProtocol::OnStopRequest	mailnews/local/src/nsMailboxProtocol.cpp:381
8	thunderbird.exe	nsInputStreamPump::OnStateStop	netwerk/base/src/nsInputStreamPump.cpp:578
9	thunderbird.exe	nsInputStreamPump::OnInputStreamReady	netwerk/base/src/nsInputStreamPump.cpp:403
10	xpcom_core.dll	nsOutputStreamReadyEvent::Run	xpcom/io/nsStreamUtils.cpp:112

Comment 1

8 years ago
a bunch of the methods here check if (mDatabase), this one doesn't. it's likely that aMsgURI is null which is why we wouldn't crash earlier
OS: Windows Vista → All
Crash Signature: [@ nsMsgDBFolder::OnMessageClassified(char const*, unsigned int, unsigned int)]
(Reporter)

Comment 2

6 years ago
these crashes might all be coming through nsBayesianFilter::classifyMessage

(In reply to timeless from comment #1)
> a bunch of the methods here check if (mDatabase), this one doesn't. it's
> likely that aMsgURI is null which is why we wouldn't crash earlier

recent tb16 crashes bp-4cb09a4f-f746-4dd7-ae73-3dd6c2121102 bp-4f2b6036-b8aa-4bff-84cb-17d072121018
Crash Signature: [@ nsMsgDBFolder::OnMessageClassified(char const*, unsigned int, unsigned int)] → [@ nsMsgDBFolder::OnMessageClassified(char const*, unsigned int, unsigned int)] [@ nsMsgDBFolder::OnMessageClassified ]
(Reporter)

Updated

4 years ago
Depends on: 797710
(Reporter)

Updated

3 years ago
No longer depends on: 797710
(Reporter)

Comment 3

20 days ago
Still a solid crash in TB60, in fact some 60.3.1 which was only recently released.

bp-c957c658-708b-4e36-a2f0-6c59a0181123 60.3.1
 0 	xul.dll	nsMsgDBFolder::OnMessageClassified(char const*, unsigned int, unsigned int)	comm/mailnews/base/util/nsMsgDBFolder.cpp:2372
1 	xul.dll	nsImapMailFolder::OnMessageClassified(char const*, unsigned int, unsigned int)	comm/mailnews/imap/src/nsImapMailFolder.cpp:9199
2 	xul.dll	MessageClassifier::classifyNextMessage()	comm/mailnews/extensions/bayesian-spam-filter/src/nsBayesianFilter.cpp:1327
3 	xul.dll	MessageClassifier::analyzeTokens(Tokenizer&)	comm/mailnews/extensions/bayesian-spam-filter/src/nsBayesianFilter.cpp:1313
4 	xul.dll	TokenStreamListener::OnStopRequest(nsIRequest*, nsISupports*, nsresult)	comm/mailnews/extensions/bayesian-spam-filter/src/nsBayesianFilter.cpp:1145
5 	xul.dll	nsStreamConverter::OnStopRequest(nsIRequest*, nsISupports*, nsresult)	comm/mailnews/mime/src/nsStreamConverter.cpp:1079
Summary: crash [@ nsMsgDBFolder::OnMessageClassified(char const*, unsigned int, unsigned int)] → crash [@ nsMsgDBFolder::OnMessageClassified via bayesian analysis, with aMsgURI null?
You need to log in before you can comment on or make changes to this bug.