Last Comment Bug 772760 - crash in nsMsgLocalMailFolder::EndMessage
: crash in nsMsgLocalMailFolder::EndMessage
Status: RESOLVED FIXED
[GS]
: crash, regression
Product: MailNews Core
Classification: Components
Component: Backend (show other bugs)
: 13
: x86 All
: -- critical (vote)
: Thunderbird 17.0
Assigned To: Makoto Kato [:m_kato]
:
:
Mentors:
https://getsatisfaction.com/mozilla_m...
Depends on:
Blocks: 402392
  Show dependency treegraph
 
Reported: 2012-07-11 00:36 PDT by Vincent (caméléon)
Modified: 2012-07-24 03:59 PDT (History)
5 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
possible fix (1.29 KB, patch)
2012-07-17 03:28 PDT, Makoto Kato [:m_kato]
mozilla: review+
Details | Diff | Splinter Review

Description Vincent (caméléon) 2012-07-11 00:36:42 PDT
This bug was filed from the Socorro interface and is 
report bp-b26ee583-e11d-42d4-8318-03f9a2120704 .
============================================================= 
User report on GetSatisfaction:
"When I try to move emails from from my Inbox, etc. to various local folders...Tbird crashes. Anything over 10-15 emails at a time and I have this problem.

Any ideas why? I am running 13.0.1. PC running Win 7"

Source: https://getsatisfaction.com/mozilla_messaging/topics/thunderbird_crashes_when_trying_to_move_emails_between_folders
Comment 1 Ludovic Hirlimann [:Usul] 2012-07-11 00:44:25 PDT
0 	xul.dll 	nsMsgLocalMailFolder::EndMessage 	mailnews/local/src/nsLocalMailFolder.cpp:2607
1 	xul.dll 	nsCopyMessageStreamListener::EndMessage 	mailnews/base/src/nsCopyMessageStreamListener.cpp:117
2 	xul.dll 	nsImapMailFolder::EndMessage 	mailnews/imap/src/nsImapMailFolder.cpp:5674
3 	xul.dll 	`anonymous namespace'::SyncRunnable2<nsIImapMailFolderSink,nsIMsgMailNewsUrl*,ch 	mailnews/imap/src/nsSyncRunnableHelpers.cpp:178
4 	xul.dll 	nsThread::ProcessNextEvent 	xpcom/threads/nsThread.cpp:657
5 	xul.dll 	NS_ProcessNextEvent_P 	objdir-tb/mozilla/xpcom/build/nsThreadUtils.cpp:245
6 	xul.dll 	mozilla::ipc::MessagePump::Run 	ipc/glue/MessagePump.cpp:134
7 	xul.dll 	MessageLoop::RunInternal 	ipc/chromium/src/base/message_loop.cc:208
8 	xul.dll 	MessageLoop::RunHandler 	ipc/chromium/src/base/message_loop.cc:201
9 	xul.dll 	MessageLoop::Run 	ipc/chromium/src/base/message_loop.cc:175
10 	xul.dll 	nsBaseAppShell::Run 	widget/xpwidgets/nsBaseAppShell.cpp:189
11 	xul.dll 	nsAppShell::Run 	widget/windows/nsAppShell.cpp:252
12 	xul.dll 	nsAppStartup::Run 	toolkit/components/startup/nsAppStartup.cpp:295
13 	xul.dll 	XRE_main 	toolkit/xre/nsAppRunner.cpp:3703
14 	thunderbird.exe 	do_main 	mail/app/nsMailApp.cpp:144
15 	thunderbird.exe 	NS_internal_main 	mail/app/nsMailApp.cpp:233
16 	thunderbird.exe 	wmain 	toolkit/xre/nsWindowsWMain.cpp:107
17 	thunderbird.exe 	__tmainCRTStartup 	crtexe.c:594
18 	kernel32.dll 	BaseThreadInitThunk 	
19 	ntdll.dll 	__RtlUserThreadStart 	
20 	ntdll.dll 	_RtlUserThreadStart
Comment 2 Makoto Kato [:m_kato] 2012-07-11 01:06:30 PDT
seekableStream (and mCopyState->m_fileStream) is null.
Comment 3 Ludovic Hirlimann [:Usul] 2012-07-11 01:47:59 PDT
Makoto-San could you cook up a patch ?
Comment 5 Makoto Kato [:m_kato] 2012-07-17 03:28:15 PDT
Created attachment 642898 [details] [diff] [review]
possible fix
Comment 6 David :Bienvenu 2012-07-20 13:00:19 PDT
Comment on attachment 642898 [details] [diff] [review]
possible fix

thx for the patch, you can remove the braces, right?
Comment 7 Wayne Mery (:wsmwk, NI for questions) 2012-07-23 08:34:48 PDT
regression from bug 675407 ?  But crash didn't start until version 12, and 675407 landed in version 10
Comment 8 Makoto Kato [:m_kato] 2012-07-23 18:19:25 PDT
(In reply to Wayne Mery (:wsmwk) from comment #7)
> regression from bug 675407 ?  But crash didn't start until version 12, and
> 675407 landed in version 10

No, I think that this is regression by bug 402392.
Comment 9 Makoto Kato [:m_kato] 2012-07-23 22:56:04 PDT
http://hg.mozilla.org/comm-central/rev/d0c32b433cea
Comment 10 Wayne Mery (:wsmwk, NI for questions) 2012-07-24 03:59:23 PDT
Thanks Makoto-San

posted info in gsfn topic

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