Last Comment Bug 522233 - Crash [@ nsImapMailFolder::ParseAdoptedMsgLine(char const*, unsigned int, int, nsIImapUrl*)]
: Crash [@ nsImapMailFolder::ParseAdoptedMsgLine(char const*, unsigned int, int...
: crash, fixed-seamonkey2.0.1, regression
Product: MailNews Core
Classification: Components
Component: Networking: IMAP (show other bugs)
: 1.9.1 Branch
: All All
-- critical (vote)
: Thunderbird 3.0rc1
Assigned To: David :Bienvenu
Depends on:
  Show dependency treegraph
Reported: 2009-10-14 05:57 PDT by Ludovic Hirlimann [:Usul]
Modified: 2011-06-09 14:58 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

possible fix (992 bytes, patch)
2009-10-14 08:14 PDT, David :Bienvenu
standard8: review+
standard8: superreview+
Details | Diff | Splinter Review

Description User image Ludovic Hirlimann [:Usul] 2009-10-14 05:57:53 PDT
Crashed while shutting down to upgrade Lightning to latest snapshot.

The stack is small and just looks bad : 
0 thunderbird.exe nsImapMailFolder::ParseAdoptedMsgLine mailnews/imap/src/nsImapMailFolder.cpp:4409
1 xpcom_core.dll NS_InvokeByIndex_P xpcom/reflect/xptcall/src/md/win32/xptcinvoke.cpp:101
2 nspr4.dll PR_Unlock nsprpub/pr/src/threads/combined/prulock.c:347
3 xpcom_core.dll nsProxyObjectCallInfo::Run xpcom/proxy/src/nsProxyEvent.cpp:181
4 xpcom_core.dll nsThread::ProcessNextEvent xpcom/threads/nsThread.cpp:521
Comment 1 User image Wayne Mery (:wsmwk, NI for questions) 2009-10-14 06:27:45 PDT
Possibly a regression because it first appears 20091001032040 build.
It's even possible it's gone - because crash was happening steadily with every daily build until last crash with build 20091009032155 on 2009-10-12 00:00 - suddenly stopped
Comment 2 User image David :Bienvenu 2009-10-14 08:14:44 PDT
Created attachment 406228 [details] [diff] [review]
possible fix

depending on how far we are into shutdown, we may not be able to get the message header, so we should really check the rv.
Comment 3 User image David :Bienvenu 2009-10-14 08:25:40 PDT
I suspect this has to do with receiving imap headers at shutdown time
Comment 4 User image David :Bienvenu 2009-10-15 07:53:55 PDT
fix checked in, a=me, since it's a top crash.
Comment 5 User image Wayne Mery (:wsmwk, NI for questions) 2009-11-10 12:59:50 PST
last build we see crash is 2009101500

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