Closed Bug 1354198 Opened 8 years ago Closed 8 years ago

Crash in mozilla::ipc::IPCResult::Fail

Categories

(Core :: Graphics: WebRender, defect)

55 Branch
Unspecified
Windows 8
defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox-esr52 --- unaffected
firefox53 --- unaffected
firefox54 --- unaffected
firefox55 --- fixed

People

(Reporter: marcia, Assigned: sotaro)

References

(Depends on 1 open bug)

Details

(Keywords: crash)

Crash Data

Attachments

(1 file)

This bug was filed from the Socorro interface and is report bp-b4bed85c-59c4-4786-882e-244162170324. ============================================================= Seen while looking at crash stats - crashes started using 20170324030205 build: http://bit.ly/2nOSOrd Possible regression range based on Build ID: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=7513b3f42058e9bcf9950d4acf4647d4ad2240f0&tochange=01d1dedf400d4be413b1a0d48090dca7acf29637
This happens sometimes if I close tabs with Ctrl+W when webrender is enabled. Can't tell precise STR yet :/
Assignee: nobody → sotaro.ikeda.g
I could reproduce the crash with the following STR on my laptop. [1] Open Many Tabs [2] Close Tabs in sequence.
Implementation of WebRenderBridgeParent::RecvShutdown() seems wrong. IPC_FAIL_NO_REASON(this) is called after Send__delete__(this), but the Send__delete__(this) could make this pointer invalid.
It seems better to change WebRenderBridgeParent::RecvShutdown() like LayerTransactionParent::RecvShutdown().
attachment 8868033 [details] [diff] [review] addressed the problem for me.
Blocks: webrender
Depends on: 1360701
Attachment #8868033 - Attachment description: patch - WebRenderBridgeParent::RecvShutdown() → patch - Fix WebRenderBridgeParent::RecvShutdown()
Attachment #8868033 - Flags: review?(sotaro.ikeda.g)
Attachment #8868033 - Flags: review?(sotaro.ikeda.g) → review?(nical.bugzilla)
Comment on attachment 8868033 [details] [diff] [review] patch - Fix WebRenderBridgeParent::RecvShutdown() Review of attachment 8868033 [details] [diff] [review]: ----------------------------------------------------------------- Makes sense.
Attachment #8868033 - Flags: review?(nical.bugzilla) → review+
Pushed by sikeda@mozilla.com: https://hg.mozilla.org/projects/graphics/rev/04109e4818e5 Fix WebRenderBridgeParent::RecvShutdown() r=nical
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: