Last Comment Bug 694196 - crash [@ MsgGetHeadersFromKeys(nsIMsgDatabase*, nsTArray<unsigned int, nsTArrayDefaultAllocator> const&, nsIMutableArray*) ]
: crash [@ MsgGetHeadersFromKeys(nsIMsgDatabase*, nsTArray<unsigned int, nsTArr...
: crash
Product: MailNews Core
Classification: Components
Component: Backend (show other bugs)
: Trunk
: x86 All
-- critical (vote)
: Thunderbird 23.0
Assigned To: Makoto Kato [:m_kato]
Depends on: 688135
  Show dependency treegraph
Reported: 2011-10-12 16:53 PDT by Wayne Mery (:wsmwk, NI for questions)
Modified: 2013-05-22 10:36 PDT (History)
5 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

fix (1.92 KB, patch)
2013-04-30 23:14 PDT, Makoto Kato [:m_kato]
rkent: review+
Details | Diff | Splinter Review

Description User image Wayne Mery (:wsmwk, NI for questions) 2011-10-12 16:53:38 PDT
crash [@ MsgGetHeadersFromKeys(nsIMsgDatabase*, nsTArray<unsigned int, nsTArrayDefaultAllocator> const&, nsIMutableArray*) ]

no useful crash comments
I think crashes prior to version 5 are MsgGetHeadersFromKeys(nsIMsgDatabase*, nsTArray<unsigned int> const&, nsIMutableArray*)

bp-78986e19-c53b-4f3d-be07-c35842110928 version 7 (lrtow)
0	xul.dll	MsgGetHeadersFromKeys	mailnews/base/util/nsMsgUtils.cpp:1989
1	xul.dll	nsMsgDBFolder::NotifyHdrsNotBeingClassified	mailnews/base/util/nsMsgDBFolder.cpp:2797
2	xul.dll	nsMsgDBFolder::CallFilterPlugins	mailnews/base/util/nsMsgDBFolder.cpp:2729
3	xul.dll	nsPop3Sink::EndMailDelivery	mailnews/local/src/nsPop3Sink.cpp:437
4	xul.dll	nsPop3Protocol::GetMsg	mailnews/local/src/nsPop3Protocol.cpp:2912
5	xul.dll	nsPop3Protocol::ProcessProtocolState	mailnews/local/src/nsPop3Protocol.cpp:3987
6	xul.dll	nsMsgProtocol::OnDataAvailable	mailnews/base/util/nsMsgProtocol.cpp:387
7	xul.dll	nsInputStreamPump::OnStateTransfer	netwerk/base/src/nsInputStreamPump.cpp:510 

Mac bp-80a5eb15-0a85-4821-81fc-0e40e2111011 MsgGetHeadersFromKeys version 8 (hfiguiere)
Comment 1 User image Makoto Kato [:m_kato] 2011-10-23 19:27:37 PDT
It seems that mDatabase is null.  This is related to bug 688135...
Comment 2 User image Makoto Kato [:m_kato] 2013-04-30 23:14:30 PDT
Created attachment 744043 [details] [diff] [review]
Comment 3 User image Kent James (:rkent) 2013-05-03 10:33:27 PDT
Comment on attachment 744043 [details] [diff] [review]

Review of attachment 744043 [details] [diff] [review]:

r=me with the one change.

::: mailnews/base/util/nsMsgUtils.cpp
@@ +2014,5 @@
>  NS_MSG_BASE nsresult MsgGetHeadersFromKeys(nsIMsgDatabase *aDB, const nsTArray<nsMsgKey> &aMsgKeys,
>                                             nsIMutableArray *aHeaders)
>  {
> +  NS_ASSERTION(aDB, "aDB is null");

Yes I know NS_ASSERTION is frequently used in utils code, but what it means is "Halt the program if in debug mode, ignore in release mode". This is not really what you want, in particular you do not want to ignore in release mode. Please replace this with NS_ENSURE_SUCCESS(rv, rv);
Comment 4 User image Kent James (:rkent) 2013-05-03 10:34:39 PDT
Oops, I mean replace with:

Comment 5 User image Makoto Kato [:m_kato] 2013-05-06 18:10:56 PDT
Comment 6 User image Wayne Mery (:wsmwk, NI for questions) 2013-05-22 10:36:50 PDT
need to recheck when TB23 goes beta or release.  unable to verify fixed via crash-stats because crash rate, though not zero, is too low in development builds to statistically get a read.

bp-84f4859b-ee01-4893-8fc7-ab4122130226 TB19.0 2013-02-26 03:52:55
bp-50f88988-6590-40d9-b43d-ee0ef2130116 TB18
bp-f5021d64-d0f5-4cf9-b48e-de2e82121228 TB18

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