Last Comment Bug 727796 - crash after getting new mail [@ nsMsgDBView::FetchKeywords(nsIMsgDBHdr*, nsACString_internal&) ]
: crash after getting new mail [@ nsMsgDBView::FetchKeywords(nsIMsgDBHdr*, nsAC...
: crash
Product: MailNews Core
Classification: Components
Component: Backend (show other bugs)
: Trunk
: x86 Windows Vista
-- critical (vote)
: Thunderbird 13.0
Assigned To: David :Bienvenu
Depends on:
  Show dependency treegraph
Reported: 2012-02-16 05:03 PST by Wayne Mery (:wsmwk, NI for questions)
Modified: 2012-02-16 14:25 PST (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

proposed fix (960 bytes, patch)
2012-02-16 12:22 PST, David :Bienvenu
neil: review+
Details | Diff | Splinter Review

Description User image Wayne Mery (:wsmwk, NI for questions) 2012-02-16 05:03:26 PST
crash after getting new mail [@ nsMsgDBView::FetchKeywords(nsIMsgDBHdr*, nsACString_internal&) ]

Did not attempt to reproduce.
Didn't keep imap log :(

- wake computer from sleep
- get mail
- deleted several multi selected items from Inbox
- attempted to click link in bugmail (it never came up in firefox)

0	xul.dll	nsMsgDBView::FetchKeywords	mailnews/base/src/nsMsgDBView.cpp:891
1	xul.dll	nsMsgDBView::FetchRowKeywords	mailnews/base/src/nsMsgDBView.cpp:933
2	xul.dll	nsMsgDBView::GetRowProperties	mailnews/base/src/nsMsgDBView.cpp:1463
3	xul.dll	nsMsgGroupView::GetRowProperties	mailnews/base/src/nsMsgGroupView.cpp:759
4	xul.dll	nsTreeBodyFrame::PaintRow	layout/xul/base/src/tree/src/nsTreeBodyFrame.cpp:2943
5	xul.dll	nsTreeBodyFrame::PaintTreeBody	layout/xul/base/src/tree/src/nsTreeBodyFrame.cpp:2874
6	xul.dll	PaintTreeBody	layout/xul/base/src/tree/src/nsTreeBodyFrame.cpp:2799
7	xul.dll	nsDisplayGeneric::Paint	layout/base/nsDisplayList.h:1309
8	xul.dll	mozilla::FrameLayerBuilder::DrawThebesLayer	layout/base/FrameLayerBuilder.cpp:2178
9	xul.dll	mozilla::layers::BasicThebesLayer::PaintBuffer	gfx/layers/basic/BasicLayers.cpp:563
10	xul.dll	mozilla::layers::BasicShadowableThebesLayer::PaintBuffer	gfx/layers/basic/BasicLayers.cpp:2357
11	xul.dll	mozilla::layers::BasicThebesLayer::PaintThebes	gfx/layers/basic/BasicLayers.cpp:753
12	xul.dll	mozilla::layers::BasicLayerManager::PaintLayer	gfx/layers/basic/BasicLayers.cpp:1920
13	xul.dll	mozilla::layers::BasicLayerManager::PaintLayer	gfx/layers/basic/BasicLayers.cpp:1935
14	xul.dll	mozilla::layers::BasicLayerManager::EndTransactionInternal	gfx/layers/basic/BasicLayers.cpp:1629
15	xul.dll	mozilla::layers::BasicShadowLayerManager::EndTransaction	gfx/layers/basic/BasicLayers.cpp:3356
16	xul.dll	nsDisplayList::PaintForFrame	layout/base/nsDisplayList.cpp:637
17	xul.dll	nsDisplayList::PaintRoot	layout/base/nsDisplayList.cpp:542
18	xul.dll	nsLayoutUtils::PaintFrame	layout/base/nsLayoutUtils.cpp:1809
Comment 1 User image Wayne Mery (:wsmwk, NI for questions) 2012-02-16 07:26:46 PST
someone else's crash from v9  bp-b6c63835-946e-43a5-9382-e01c12120124
crashed when I closed a sheet(that has been opened automatically when I searched a message with the function search all messages
Comment 2 User image David :Bienvenu 2012-02-16 07:52:06 PST
the hdr is null, which is surprising but easy to check for.
Comment 3 User image David :Bienvenu 2012-02-16 12:22:30 PST
Created attachment 597953 [details] [diff] [review]
proposed fix
Comment 4 User image David :Bienvenu 2012-02-16 14:25:00 PST
fixed on trunk -

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