The default bug view has changed. See this FAQ.

oauth related crash in nsMsgAsyncWriteProtocol::SendData(char const*, bool)

RESOLVED FIXED in Thunderbird 41.0

Status

MailNews Core
Networking
--
critical
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: wsmwk, Assigned: Magnus Melin)

Tracking

({crash})

Trunk
Thunderbird 41.0
x86
Windows NT
crash

Thunderbird Tracking Flags

(thunderbird39 fixed, thunderbird40 fixed, thunderbird41 fixed, thunderbird_esr38+ fixed, seamonkey2.35+ fixed, seamonkey2.36 fixed, seamonkey2.37 fixed)

Details

(crash signature)

Attachments

(1 attachment)

regression?

crash does not appear prior to 38.0b6
bp-bdc4b915-90e7-4c4e-8c90-552932150611 38.0.1
bp-fb669388-3495-41cd-a42b-189762150609 38.0b6

Does "crashed verifying sms code on new Aouth2 accaont" sound familiar? 
belongs to bp-cb15f59c-8766-462b-9b53-2a8642150426. 40.0a1
(Assignee)

Comment 1

2 years ago
Created attachment 8623870 [details] [diff] [review]
bug1175348_mAsyncOutStream_crash.patch

I don't know the flow, but as I read this: if we try to SendData after CloseSocket we crash.
Assignee: nobody → mkmelin+mozilla
Status: NEW → ASSIGNED
Attachment #8623870 - Flags: review?(rkent)

Comment 2

2 years ago
Comment on attachment 8623870 [details] [diff] [review]
bug1175348_mAsyncOutStream_crash.patch

LGTM

We'll push to branches as well.
Attachment #8623870 - Flags: review?(rkent)
Attachment #8623870 - Flags: review+
Attachment #8623870 - Flags: approval-comm-esr38+
Attachment #8623870 - Flags: approval-comm-beta+
Attachment #8623870 - Flags: approval-comm-aurora+

Comment 3

2 years ago
https://hg.mozilla.org/comm-central/rev/6139912cfaa4
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-thunderbird39: --- → affected
status-thunderbird40: --- → affected
status-thunderbird41: --- → fixed
status-thunderbird_esr38: --- → affected
tracking-thunderbird_esr38: --- → +
Component: General → Networking
Product: Thunderbird → MailNews Core
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 41.0

Comment 4

2 years ago
Comment on attachment 8623870 [details] [diff] [review]
bug1175348_mAsyncOutStream_crash.patch

http://hg.mozilla.org/releases/comm-beta/rev/cbb9b580434c
http://hg.mozilla.org/releases/comm-aurora/rev/e0a6c871c33e

Updated

2 years ago
status-thunderbird39: affected → fixed
status-thunderbird40: affected → fixed

Comment 5

2 years ago
http://hg.mozilla.org/releases/comm-esr38/rev/5e4b9696b493
status-seamonkey2.35: --- → affected
status-seamonkey2.36: --- → affected
status-seamonkey2.37: --- → fixed
status-thunderbird_esr38: affected → fixed

Updated

2 years ago
status-seamonkey2.36: affected → fixed

Updated

2 years ago
Blocks: 1177041

Comment 6

2 years ago
Uplift to comm-release+SEAMONKEY_2_35_RELEASE_BRANCH CLOSED TREE
http://hg.mozilla.org/releases/comm-release/rev/0b3c0bc49ac9
status-seamonkey2.35: affected → fixed
tracking-seamonkey2.35: --- → +
You need to log in before you can comment on or make changes to this bug.