Closed Bug 489779 Opened 11 years ago Closed 11 years ago

Crash [@ nsMsgSendLater::DoShutdownTask(nsIUrlListener*, nsIMsgWindow*, int*) ] whilst sending mail on shutdown

Categories

(MailNews Core :: Networking: SMTP, defect, critical)

defect
Not set
critical

Tracking

(Not tracked)

VERIFIED FIXED
Thunderbird 3.0b3

People

(Reporter: standard8, Assigned: standard8)

References

Details

(Keywords: crash)

Crash Data

Attachments

(1 file)

Attached patch The fixSplinter Review
Found during TB's test day:

1) Have a message in the outbox whilst starting up Thunderbird.
2) Thunderbird prompts to send message, select Yes.
3) Quit whilst message is being sent.

Result: We crash.

0  	thunderbird.exe  	nsMsgSendLater::DoShutdownTask  	 mailnews/compose/src/nsMsgSendLater.cpp:1431
1 	thunderbird.exe 	nsMsgShutdownService::ProcessNextTask 	mailnews/base/src/nsMsgMailSession.cpp:562
2 	thunderbird.exe 	nsMsgShutdownService::StartShutdownTasks 	mailnews/base/src/nsMsgMailSession.cpp:697
3 	xpcom_core.dll 	NS_InvokeByIndex_P 	xpcom/reflect/xptcall/src/md/win32/xptcinvoke.cpp:101
4 	thunderbird.exe 	XPCWrappedNative::CallMethod 	js/src/xpconnect/src/xpcwrappednative.cpp:2480
5 	thunderbird.exe 	XPC_WN_CallMethod 	js/src/xpconnect/src/xpcwrappednativejsops.cpp:1587
6 	js3250.dll 	js_Invoke 	js/src/jsinterp.cpp:1380 

This is because mTimer is null at that stage and needs a null check.

Patch attached, I probably didn't need to bullet proof the Observe function, but thought I would do it just in case.
Attachment #374258 - Flags: superreview?(bienvenu)
Attachment #374258 - Flags: review?(bienvenu)
Attachment #374258 - Flags: superreview?(bienvenu)
Attachment #374258 - Flags: superreview+
Attachment #374258 - Flags: review?(bienvenu)
Attachment #374258 - Flags: review+
Patch checked in: http://hg.mozilla.org/comm-central/rev/98ac44647507
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
nice - one more shutdown bug gone!
WFM Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1b5pre) Gecko/20090505 Shredder/3.0b3pre (but I didn't test build before the patch)
v.fixed and bump severity

(In reply to comment #1)
> Breakpad stack URL:
> 
> bp-accd8b6a-f80a-476f-9579-273192090423

that's the only crash on crash-stats, build 20090422
Severity: normal → critical
Status: RESOLVED → VERIFIED
Crash Signature: [@ nsMsgSendLater::DoShutdownTask(nsIUrlListener*, nsIMsgWindow*, int*) ]
You need to log in before you can comment on or make changes to this bug.