390852 - crash in [@ nsQueryInterface::operator()(nsID const&, void**)]

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[baffclan]

version 3.0a1pre (2007080304)

Steps to Reproduce :
1. getting mails
2. click a Subject column in Thread paine

Crash ID: bp-62fb23c7-41f5-11dc-a7dd-001a4bd43e5c
Crash ID: bp-7007bec2-41f6-11dc-ae5a-001a4bd46e84

---
0 	nsQueryInterface::operator()(nsID const&, void**) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\obj-tb-trunk\xpcom\build\nscomptr.cpp:47
1 	nsCOMPtr_base::assign_from_qi(nsQueryInterface, nsID const&) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\obj-tb-trunk\xpcom\build\nscomptr.cpp:96
2 	nsCOMPtr<nsIMsgFolder>::operator=(nsQueryInterface) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\obj-tb-trunk\dist\include\xpcom\nscomptr.h:731
3 	nsMsgDBView::SecondarySort(unsigned int, nsISupports*, unsigned int, nsISupports*, viewSortInfo*) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\base\src\nsmsgdbview.cpp:3718
4 	nsMsgDBView::FnSortIdKeyPtr(void const*, void const*, void*) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\base\src\nsmsgdbview.cpp:3304
5 	nsMsgDBView::GetInsertIndexHelper(nsIMsgDBHdr*, nsMsgKeyArray*, int, int) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\base\src\nsmsgdbview.cpp:4637
6 	nsMsgDBView::GetInsertIndex(nsIMsgDBHdr*) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\base\src\nsmsgdbview.cpp:4669
7 	nsMsgDBView::AddHdr(nsIMsgDBHdr*) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\base\src\nsmsgdbview.cpp:4701
8 	nsMsgThreadedDBView::OnNewHeader(nsIMsgDBHdr*, unsigned int, int) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\base\src\nsmsgthreadeddbview.cpp:598
9 	nsMsgDBView::OnHdrAdded(nsIMsgDBHdr*, unsigned int, int, nsIDBChangeListener*) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\base\src\nsmsgdbview.cpp:4993
10 	nsMsgDatabase::NotifyHdrAddedAll(nsIMsgDBHdr*, unsigned int, int, nsIDBChangeListener*) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\db\msgdb\src\nsmsgdatabase.cpp:668
11 	nsMsgDatabase::AddNewHdrToDB(nsIMsgDBHdr*, int) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\db\msgdb\src\nsmsgdatabase.cpp:3038
12 	nsParseNewMailState::PublishMsgHeader(nsIMsgWindow*) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\local\src\nsparsemailbox.cpp:1719
13 	nsPop3Sink::IncorporateComplete(nsIMsgWindow*, int) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\local\src\nspop3sink.cpp:900
14 	nsPop3Protocol::HandleLine(char*, unsigned int) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\local\src\nspop3protocol.cpp:3183
15 	nsPop3Protocol::RetrResponse(nsIInputStream*, unsigned int) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\local\src\nspop3protocol.cpp:2966
16 	nsPop3Protocol::ProcessProtocolState(nsIURI*, nsIInputStream*, unsigned int, unsigned int) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\local\src\nspop3protocol.cpp:3591
17 	nsMsgProtocol::OnDataAvailable(nsIRequest*, nsISupports*, nsIInputStream*, unsigned int, unsigned int) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mailnews\base\util\nsmsgprotocol.cpp:345
18 	nsInputStreamPump::OnStateTransfer() 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\netwerk\base\src\nsinputstreampump.cpp:508
19 	nsInputStreamPump::OnInputStreamReady(nsIAsyncInputStream*) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\netwerk\base\src\nsinputstreampump.cpp:398
20 	nsInputStreamReadyEvent::Run() 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\xpcom\io\nsstreamutils.cpp:111
21 	nsThread::ProcessNextEvent(int, int*) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\xpcom\threads\nsthread.cpp:490
22 	NS_ProcessNextEvent_P(nsIThread*, int) 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\obj-tb-trunk\xpcom\build\nsthreadutils.cpp:227
23 	nsBaseAppShell::Run() 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\widget\src\xpwidgets\nsbaseappshell.cpp:154
24 	nsAppStartup::Run() 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\toolkit\components\startup\src\nsappstartup.cpp:170
25 	XRE_main 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\toolkit\xre\nsapprunner.cpp:3057
26 	main 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mail\app\nsmailapp.cpp:87
27 	WinMain 	d:\builds\tinderbox\tb-trunk\winnt_5.2_depend\mozilla\mail\app\nsmailapp.cpp:98
28 	__tmainCRTStartup 	f:\rtm\vctools\crt_bld\self_x86\crt\src\crtexe.c:578
29 	BaseProcessStart
---

Comment 1

[David :Bienvenu]

this isn't a db crash; it's a crash in the view code. I checked in a fix for this on 08/03 - the patch is in bug 57898 - does the crash appear in builds from 08/04?

Comment 2

[David :Bienvenu]

reassigning to me, just in case.

Comment 3

[baffclan]

(In reply to comment #1)
> this isn't a db crash; it's a crash in the view code. I checked in a fix for
> this on 08/03 - the patch is in bug 57898 - does the crash appear in builds
> from 08/04?
> 
reproduce on Thunderbird/2007080807-trunk/WinXP 
version 3.0a1pre (2007080807)

Crash ID: bp-dd5b7223-45ee-11dc-afd2-001a4bd46e84
Crash ID: bp-dcfdbb00-45f0-11dc-b5c2-001a4bd43e5c

Crash ID: bp-2505354c-45ef-11dc-ad45-001a4bd46e84
Crash ID: bp-013a8e8f-45f0-11dc-9b7c-001a4bd43ef6

Comment 4

[Wayne Mery (:wsmwk)]

#2 crash on breakpad

mine 
- bp-3772f987-50aa-11dc-aa13-001a4bd43ef6 at 2007-08-22-12 using (2007082004)
- don't know what I was doing, but quite sure I hadn't clicked any any column
- not the first time it crashed with this stack

Comment 5

[David :Bienvenu]

Attachment: possible fix

you can also get this crash if new mail arrives into a sorted view.

Comment 6

[Wayne Mery (:wsmwk)]

I've been using sorted view (love it)

Comment 7

[David :Bienvenu]

just curious - do you mean with a secondarily sorted view? I'm trying to think how to publicize this feature...

Comment 8

[Wayne Mery (:wsmwk)]

no, not your secondary sort. (haven't acquainted myself with that yet.

what I only recently started using (a month maybe?) for my bugmail folder is sort ascending, date, threaded.  previously had been using sort ascending, date recieved,, unthreaded (which is what I still use for most of my other folders)

Comment 9

[baffclan]

cannot reproduce on Thunderbird/2007091503-trunk/WinXP 

I found check-in at 2007-08-24 11:05 in bonsai.
Thanks!

Comment 10

[David :Bienvenu]

thx, marking fixed.