Open Bug 1508649 Opened Last year Updated 8 months ago

Crash sending mail in shutdownhang | _PR_MD_WAIT_CV | _PR_WaitCondVar | PR_Wait | PR_CWait | nsMsgCopy::DoCopy - Error saving to Sent folder

Categories

(MailNews Core :: Composition, defect, critical)

x86
Windows 8
defect
Not set
critical

Tracking

(thunderbird_esr60? affected)

Tracking Status
thunderbird_esr60 ? affected

People

(Reporter: wsmwk, Unassigned)

Details

(4 keywords)

Crash Data

Attachments

(1 file)

#74 crash for 52.9.1, #40 for 60.3.0. TBD for 60.3.1.
An odd dip in crash rate mid-august through Sept, seen in attached graph.

In beta, the signature seems to have changed to shutdownhang | _PR_MD_WAIT_CV | PR_Wait | PR_CWait | nsMsgCopy::DoCopy

We don't have a testcase but comments say sent message not being saved to Sent folder.

bp-39409cdf-a0e5-49e8-b35f-0b6bc0181119.  60.3.1  no significant add-ons
=============================================================
Top 10 frames of crashing thread:

0 ntdll.dll NtWaitForSingleObject 
1 kernelbase.dll WaitForSingleObjectEx 
2 kernelbase.dll WaitForSingleObject 
3 nss3.dll _PR_MD_WAIT_CV nsprpub/pr/src/md/windows/w95cv.c:248
4 nss3.dll _PR_WaitCondVar nsprpub/pr/src/threads/combined/prucv.c:172
5 nss3.dll PR_WaitCondVar nsprpub/pr/src/threads/combined/prucv.c:525
6 nss3.dll PR_Wait nsprpub/pr/src/threads/prmon.c:294
7 nss3.dll PR_CWait nsprpub/pr/src/threads/prcmon.c:394
8 xul.dll nsMsgCopy::DoCopy comm/mailnews/compose/src/nsMsgCopy.cpp:284
9 xul.dll nsMsgCopy::StartCopyOperation comm/mailnews/compose/src/nsMsgCopy.cpp:226
10 xul.dll nsMsgComposeAndSend::StartMessageCopyOperation comm/mailnews/compose/src/nsMsgSend.cpp:4826
11 xul.dll nsMsgComposeAndSend::MimeDoFCC comm/mailnews/compose/src/nsMsgSend.cpp:4795
12 xul.dll nsMsgComposeAndSend::SendToMagicFolder comm/mailnews/compose/src/nsMsgSend.cpp:4244
13 xul.dll nsMsgComposeAndSend::DeliverMessage  comm/mailnews/compose/src/nsMsgSend.cpp:3203
14 xul.dll nsMsgComposeAndSend::GatherMimeAttachments	comm/mailnews/compose/src/nsMsgSend.cpp:994
=============================================================

bp-d259c4ca-cdd8-4131-82bc-f75d50181006  60.0b11
In the right hand side of that graph we see in mid-November the beginnings of a significant decrease for shutdownhang | _PR_MD_WAIT_CV | _PR_WaitCondVar | PR_Wait | PR_CWait | nsMsgCopy::DoCopy  which in fact has persisted.  The crash rate for v60 appears to be about 1/6 the rate of v52.

Became a solid topcrash in the vicinity of 60.5.0. So 60.5.0 introduces some regressive behavior. https://www.thunderbird.net/en-US/thunderbird/60.5.0/releasenotes/

Attachment related?

A common crash comment "Thunderbird can send, but not copy to SEND folder." bp-7bd0cfae-242b-48ef-a5f4-105f40190208

0 ntdll.dll NtWaitForSingleObject context
1 kernelbase.dll WaitForSingleObjectEx cfi
2 kernelbase.dll WaitForSingleObject cfi
3 nss3.dll _PR_MD_WAIT_CV nsprpub/pr/src/md/windows/w95cv.c:248 cfi
4 nss3.dll _PR_WaitCondVar nsprpub/pr/src/threads/combined/prucv.c:172 cfi
5 nss3.dll PR_WaitCondVar nsprpub/pr/src/threads/combined/prucv.c:525 cfi
6 nss3.dll PR_Wait nsprpub/pr/src/threads/prmon.c:294 cfi
7 nss3.dll PR_CWait nsprpub/pr/src/threads/prcmon.c:394 cfi
8 xul.dll nsMsgCopy::DoCopy(nsIFile*, nsIMsgFolder*, nsIMsgDBHdr*, bool, nsIMsgWindow*, nsIMsgSend*) comm/mailnews/compose/src/nsMsgCopy.cpp:284 cfi
9 xul.dll nsMsgCopy::StartCopyOperation(nsIMsgIdentity*, nsIFile*, int, nsIMsgSend*, char const*, nsIMsgDBHdr*) comm/mailnews/compose/src/nsMsgCopy.cpp:226 cfi
10 xul.dll nsMsgComposeAndSend::StartMessageCopyOperation(nsIFile*, int, nsTString<char> const&) comm/mailnews/compose/src/nsMsgSend.cpp:4815 cfi
11 xul.dll nsMsgComposeAndSend::MimeDoFCC(nsIFile*, int, char const*, char const*, char const*) comm/mailnews/compose/src/nsMsgSend.cpp:4784 cfi
12 xul.dll nsMsgComposeAndSend::SendToMagicFolder(int) comm/mailnews/compose/src/nsMsgSend.cpp:4233 cfi
13 xul.dll nsMsgComposeAndSend::DeliverMessage() comm/mailnews/compose/src/nsMsgSend.cpp:3192 cfi
14 xul.dll nsMsgComposeAndSend::GatherMimeAttachments() comm/mailnews/compose/src/nsMsgSend.cpp:994 cfi
15 xul.dll nsMsgComposeAndSend::HackAttachments(nsIArray*, nsIArray*) comm/mailnews/compose/src/nsMsgSend.cpp:2533 cfi
16 xul.dll nsMsgComposeAndSend::Init(nsIMsgIdentity*, char const*, nsMsgCompFields*, nsIFile*, bool, bool, int, nsIMsgDBHdr*, char const*, nsTSubstring<char> const&, nsIArray*, nsIArray*, nsTSubstring<char16_t> const&, nsTSubstring<char> const&, int) comm/mailnews/compose/src/nsMsgSend.cpp:3115 cfi
17 xul.dll nsMsgComposeAndSend::CreateAndSendMessage(nsIEditor*, nsIMsgIdentity*, char const*, nsIMsgCompFields*, bool, bool, int, nsIMsgDBHdr*, char const*, nsTSubstring<char> const&, nsIArray*, nsIArray*, mozIDOMWindowProxy*, nsIMsgProgress*, nsIMsgSendListener*, nsTSubstring<char16_t> const&, nsTSubstring<char> const&, int) comm/mailnews/compose/src/nsMsgSend.cpp:4118 cfi
18 xul.dll nsMsgCompose::SendMsgToServer(int, nsIMsgIdentity*, char const*) comm/mailnews/compose/src/nsMsgCompose.cpp:1267 cfi
19 xul.dll nsMsgCompose::SendMsg(int, nsIMsgIdentity*, char const*, nsIMsgWindow*, nsIMsgProgress*) comm/mailnews/compose/src/nsMsgCompose.cpp:1486 cfi
20 xul.dll NS_InvokeByIndex xpcom/reflect/xptcall/md/win32/xptcinvoke_asm_x86_msvc.asm:54 cfi

Flags: needinfo?(mkmelin+mozilla)
Summary: Crash sending mail in shutdownhang | _PR_MD_WAIT_CV | _PR_WaitCondVar | PR_Wait | PR_CWait | nsMsgCopy::DoCopy → Crash sending mail in shutdownhang | _PR_MD_WAIT_CV | _PR_WaitCondVar | PR_Wait | PR_CWait | nsMsgCopy::DoCopy - Error saving to Sent folder

Continuing comment 0:
a) I haven't done a full numerical analysis so this is mostly eyeball and
b) update rate for various v60.x increased over time (Sept-Jan), so reading the tea leaves and the conclusion might be wrong that some newer version has a regression.

Unfortunately the crash rate or non-release builds is very low, so we lack data => and so can't determine whether or when things got worse https://crash-stats.mozilla.com/search/?signature=%3Dshutdownhang%20%7C%20_PR_MD_WAIT_CV%20%7C%20_PR_WaitCondVar%20%7C%20PR_Wait%20%7C%20PR_CWait%20%7C%20nsMsgCopy%3A%3ADoCopy&release_channel=%21release&product=Thunderbird&date=%3E%3D2018-10-18T11%3A40%3A00.000Z&date=%3C2019-01-18T11%3A40%3A00.000Z&_facets=signature&_facets=build_id&_sort=-date&_columns=date&_columns=signature&_columns=product&_columns=version&_columns=build_id&_columns=platform#facet-build_id

Therefore another possible conclusion is the code went bad in the 60.0b* cycle, and the topcrash emerged only when we significantly increased the v60 update rate. (I'm not totally sold on this idea)

6.5.2 bp-2c6378aa-bf95-4ecb-8203-9b1410190317 Thunderbird claimed an email was sent but not saved to the Sent Folder giving me the options to Retry/Save/Not Save. I selected Save but could not verify the mail had been saved. I checked for Update and founs that an Update was claimed to be be running

52.9.1 bp-b609baa5-f127-452d-a9a7-04b830181206 1.Thunderbird won't save emails as always. 2. I'm getting no new emails since yesterday at Oh!Emails just arrived and the 'waiting for something' circling signal is gone! Hurrah
52.9.1 bp-46f6bc91-9be7-40e1-8a71-9b86b0181102 Unable to send emails - states outgoing server(SMPT) smpt.gmail is unknown. Server may be incorrectly configured. verify outgoing sever settings are correct and try again. Don't understand any of this

https://crash-stats.mozilla.com/search/?signature=%3Dshutdownhang%20%7C%20_PR_MD_WAIT_CV%20%7C%20_PR_WaitCondVar%20%7C%20PR_Wait%20%7C%20PR_CWait%20%7C%20nsMsgCopy%3A%3ADoCopy&product=Thunderbird&date=%3E%3D2018-09-18T10%3A53%3A00.000Z&date=%3C2019-01-31T10%3A53%3A00.000Z&_facets=signature&_facets=build_id&_sort=-date&_columns=date&_columns=signature&_columns=product&_columns=version&_columns=build_id&_columns=platform#facet-build_id

build version
20180709124824 52.9.1
20181217101353 60.4.0 spike?
20181204174145 60.3.3 spike?
20181030083531 60.3.0

(I don't have any clue here.)

Flags: needinfo?(mkmelin+mozilla)
You need to log in before you can comment on or make changes to this bug.