Closed
Bug 1360992
Opened 7 years ago
Closed 7 years ago
AddressSanitizer: attempting double-free on 0x603000211990 in thread T33 (DOM Worker)
Categories
(Core :: DOM: Workers, enhancement)
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: MatsPalmgren_bugz, Assigned: baku)
Details
(Whiteboard: [asan])
Attachments
(2 files)
31.94 KB,
text/plain
|
Details | |
977 bytes,
patch
|
qdot
:
review+
|
Details | Diff | Splinter Review |
Found this in one of my Try pushes. I doubt it's related to any of my changes. https://treeherder.mozilla.org/#/jobs?repo=try&revision=939ad5a632afe5db44f30cc624919da2dac4e38e&selectedJob=95587737 m-i rev 154b84a2fd16
Reporter | ||
Comment 1•7 years ago
|
||
https://treeherder.mozilla.org/logviewer.html#?job_id=95587737&repo=try&lineNumber=4310
Comment 2•7 years ago
|
||
Looks like fallout from the blob refactor to me.
Flags: needinfo?(amarchesini)
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → amarchesini
Flags: needinfo?(amarchesini)
Assignee | ||
Comment 3•7 years ago
|
||
Attachment #8864042 -
Flags: review?(kyle)
Comment 4•7 years ago
|
||
Comment on attachment 8864042 [details] [diff] [review] lock.patch Review of attachment 8864042 [details] [diff] [review]: ----------------------------------------------------------------- Can you add a commit message to talk about why this was needed? It's not obvious without the stack track for the crash.
Attachment #8864042 -
Flags: review?(kyle) → review+
Pushed by amarchesini@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/ab9fdee3a6a4 RecvStreamReady() should be protected by mutex as any other method in IPCBlobInputStreamChild, r=qdot
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ab9fdee3a6a4
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Comment 7•7 years ago
|
||
Backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/379b03af76d417ba5998a2fb13b125602540c8cc since I didn't know which cset caused https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&filter-searchStr=ea202f6d70b69a8e8d18cd211ce48e611620bf1d&tochange=ab9fdee3a6a471fd88db5790e8499bdb92a4de58&fromchange=6f1a0340d55386cdf61c7a66d4b0a8f79a13ece9&group_state=expanded
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Backout by kwierso@gmail.com: https://hg.mozilla.org/mozilla-central/rev/ff83fde8be94 Backed out 2 changesets (bug 1360992, bug 1361654) for a 70% failure rate in test_fileReader.html on ASan e10s a=merge
Pushed by amarchesini@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/b59890830f84 RecvStreamReady() should be protected by mutex as any other method in IPCBlobInputStreamChild, r=qdot
Comment 10•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b59890830f84
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•