610533 - crash [@ nsMsgDatabase::GetSearchResultsTable(char const*, int, nsIMdbTable**)] with virtual folder

Status: RESOLVED FIXED

(MailNews Core :: Database, defect)

Description

[Wayne Mery (:wsmwk)]

crash [@ nsMsgDatabase::GetSearchResultsTable(char const*, int, nsIMdbTable**)]

bp-9d7721a6-6f9c-4019-842a-c1a542101026 v3.0.6

bp-5daa9521-c0b9-409f-ad8a-b8f402101027 v3.1.5
0	thunderbird.exe	nsMsgDatabase::GetSearchResultsTable	mailnews/db/msgdb/src/nsMsgDatabase.cpp:5499
1	thunderbird.exe	nsMsgDatabase::GetCachedHits	mailnews/db/msgdb/src/nsMsgDatabase.cpp:5512
2	thunderbird.exe	nsMsgXFVirtualFolderDBView::OnNewSearch	mailnews/base/src/nsMsgXFVirtualFolderDBView.cpp:455
3	thunderbird.exe	nsMsgSearchSession::Search	mailnews/base/search/src/nsMsgSearchSession.cpp:283
4	xpcom_core.dll	NS_InvokeByIndex_P	xpcom/reflect/xptcall/src/md/win32/xptcinvoke.cpp:102
5	thunderbird.exe	XPCWrappedNative::CallMethod	js/src/xpconnect/src/xpcwrappednative.cpp:2722
6	thunderbird.exe	XPC_WN_CallMethod	js/src/xpconnect/src/xpcwrappednativejsops.cpp:1740
7	js3250.dll	js_Invoke	js/src/jsinterp.cpp:1360
8	js3250.dll	js_Interpret	js/src/jsops.cpp:2240
9	js3250.dll	js_Invoke	js/src/jsinterp.cpp:1368
10	js3250.dll	js_InternalInvoke	js/src/jsinterp.cpp:1423
11	js3250.dll	JS_CallFunctionValue	js/src/jsapi.cpp:5114
12	thunderbird.exe	nsJSContext::CallEventHandler	dom/base/nsJSEnvironment.cpp:2197

Comment 1

[David :Bienvenu]

m_mdbStore is probably null...

Comment 2

[timeless]

Crash Address	0x0 - so it is a null pointer...

Comment 3

[Wayne Mery (:wsmwk)]

(Note: bug# doesn't show up in socorro crash reports)

Comment 4

[Wayne Mery (:wsmwk)]

signature  for current versions is nsMsgDatabase::GetSearchResultsTable(char const*, bool, nsIMdbTable**)

Comment 5

[Wayne Mery (:wsmwk)]

"Tried to open an individual email while compacting"
bp-fb63445d-8812-430e-848f-248e32121203

mwu@9538 5677 nsresult nsMsgDatabase::GetSearchResultsTable(const char *searchFolderUri, bool createIfMissing, nsIMdbTable **table)
hg@0 5678{
hg@0 5679  mdb_kind kindToken;
hg@0 5680  mdb_count numTables;
hg@0 5681  mdb_bool mustBeUnique;
hg@0 5682  mdb_err err = m_mdbStore->StringToToken(GetEnv(), searchFolderUri, &kindToken);

Comment 6

[Wayne Mery (:wsmwk)]

This still appears. Is it easy to fix?

bp-5c2bbdb6-8266-4c41-a27e-992f42150417 (jeorg)
bp-13cadb64-1424-4925-a6b5-53bd32150507

Comment 7

[Kent James (:rkent)]

Attachment: Null check

Comment 8

[[e-mail]]

Comment on attachment 8603417 [details] [diff] [review]
Null check

nsMsgDatabase is nicely inconsistent about which error code to return when m_mdbStore is null...

Comment 9

[Kent James (:rkent)]

http://hg.mozilla.org/comm-central/rev/a05b10d2a3f7

Comment 10

[Wayne Mery (:wsmwk)]

I'm not sure we should take this in beta unless there is high reward

Comment 11

[Kent James (:rkent)]

Comment on attachment 8603417 [details] [diff] [review]
Null check

On Wayne's recommendation, we won't uplift this at the moment.