Closed
Bug 1306979
Opened 8 years ago
Closed 8 years ago
[e10s] Firefox crashes in [@ mozilla::ipc::FatalError | mozilla::dom::PContentParent::Read ]
Categories
(Core :: IPC, defect)
Tracking
()
VERIFIED
FIXED
mozilla52
Tracking | Status | |
---|---|---|
firefox49 | --- | unaffected |
firefox-esr45 | --- | unaffected |
firefox50 | --- | unaffected |
firefox51 | --- | unaffected |
firefox52 | + | verified |
People
(Reporter: Virtual, Assigned: baku)
References
Details
(4 keywords, Whiteboard: [fixed by updated patch from bug #1288997])
Crash Data
I enabled e10s in Nightly to see how much it improved, browsed for about 10min and it crahed on one of MEGA website pages, when I wanted to download files or maybe when I wanted to install MEGA extension, I don't remember when now. It crashes in[@ mozilla::ipc::FatalError | mozilla::dom::PContentParent::Read ] https://crash-stats.mozilla.com/report/index/d98131b4-a331-4c63-bc34-e50e32161002
Comment 1•8 years ago
|
||
This sounds like the same crash reported in https://bugzilla.mozilla.org/show_bug.cgi?id=1288997#c61 .
Comment 2•8 years ago
|
||
The crash is happening, intermittently, when downloading blobs larger than a certain size (e.g. from mega.nz, or the attachment in bug 1288997). I can also reproduce it by opening https://www.mapbox.com/blog/mapd-taxi/ and scrolling down. I'm not sure what this site is doing.
Comment 3•8 years ago
|
||
All crashes have IPCFatalErrorProtocol=pcontentparent and IPCFatalErrorMsg=Error deserializing Union type. The first build ID with this crash is 20161001030430, so https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=59db3b0781908d9e2145477ff8dc5a85729e25bf&tochange=87cd291d2db621da6b3eb1057027cc0725b6eb1d. It is likely the same crash from bug 1288997, a different stack trace because of bug 1294450.
Updated•8 years ago
|
Crash Signature: [@ mozilla::ipc::FatalError | mozilla::dom::PContentParent::Read ] → [@ mozilla::ipc::FatalError | mozilla::dom::PContentParent::Read ]
[@ mozilla::ipc::FatalError | mozilla::dom::PContentParent::FatalError | mozilla::dom::PContentParent::Read ]
Comment 4•8 years ago
|
||
Note also that before bug 1294450 the crash (moz_crash_reason=MOZ_CRASH(IPC message size is too large) and ipc_message_name=PContent::Msg_PBlobConstructor) was less frequent (probably only happened on mega.nz) with 54 crashes on Nightly during last week, the new signature has instead 184 crashes in less than two days. 1: https://crash-stats.mozilla.com/search/?moz_crash_reason=%3DMOZ_CRASH%28IPC%20message%20size%20is%20too%20large%29&ipc_message_name=%3DPContent%3A%3AMsg_PBlobConstructor&product=Firefox&version=52.0a1&_sort=-date&_facets=signature&_columns=date&_columns=signature&_columns=product&_columns=version&_columns=build_id&_columns=platform#facet-signature 2: https://crash-stats.mozilla.com/search/?signature=%3Dmozilla%3A%3Aipc%3A%3AFatalError%20%7C%20mozilla%3A%3Adom%3A%3APContentParent%3A%3ARead&product=Firefox&version=52.0a1&_sort=-date&_facets=signature&_columns=date&_columns=signature&_columns=product&_columns=version&_columns=build_id&_columns=platform#crash-reports
status-firefox49:
--- → unaffected
status-firefox50:
--- → unaffected
status-firefox51:
--- → unaffected
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•8 years ago
|
status-firefox-esr45:
--- → unaffected
Comment 5•8 years ago
|
||
Currently the top crash on nightly with well over 500 crashes.
Keywords: topcrash
Comment 6•8 years ago
|
||
I was hitting this consistently today trying to load http://www.apple.com/macos/sierra/.
tracking-firefox52:
--- → ?
Comment 7•8 years ago
|
||
I guess for the case when PSendStream is used, BlobDataFromBlobImpl() needs to keep the instances of AutoIPCStream until SendPBlobConstructor() is done. Otherwise the actor in parent may have been destroyed when PBlobConstructor is received.
Comment 9•8 years ago
|
||
Please look at the patch in https://bugzilla.mozilla.org/show_bug.cgi?id=1288997#c66 .
Flags: needinfo?(josh)
Comment 11•8 years ago
|
||
I believe this is fixed now. https://crash-stats.mozilla.com/search/?signature=%3Dmozilla%3A%3Aipc%3A%3AFatalError%20%7C%20mozilla%3A%3Adom%3A%3APContentParent%3A%3ARead&signature=%3Dmozilla%3A%3Aipc%3A%3AFatalError%20%7C%20mozilla%3A%3Adom%3A%3APContentParent%3A%3AFatalError%20%7C%20mozilla%3A%3Adom%3A%3APContentParent%3A%3ARead&product=Firefox&date=%3E%3D2016-10-03T11%3A07%3A00.000Z&date=%3C2016-10-10T11%3A07%3A00.000Z&_sort=-date&_facets=signature&_facets=build_id&_columns=date&_columns=signature&_columns=product&_columns=version&_columns=build_id&_columns=platform#facet-build_id The last affected build ID is 20161004030204.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Comment 12•8 years ago
|
||
Yes, no more crashes starting from Mozilla Firefox Nightly 52.0a1 (2016-10-05), because of updated patch from bug #1288997, like Josh Matthews [:jdm] said in Comment #9, so marking this as VERIFIED. Thank you very much!
Status: RESOLVED → VERIFIED
Has Regression Range: --- → yes
Has STR: --- → yes
Whiteboard: [fixed by updated patch from bug #1288997]
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•8 years ago
|
Target Milestone: --- → mozilla52
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•7 years ago
|
Keywords: nightly-community
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•7 years ago
|
QA Contact: Virtual
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•6 years ago
|
Assignee: nobody → amarchesini
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
||
Updated•5 years ago
|
Keywords: crashreportid
You need to log in
before you can comment on or make changes to this bug.
Description
•