Closed Bug 1412408 Opened 4 years ago Closed 4 years ago
content process leak of 1 U2FTransaction
45 bytes, text/x-phabricator-request
|Details | Review|
This showed up in the last week, so presumably it is a regression from bug 1403818. Unsurprisingly, this leak shows up while running e10s Mochitests in the dom/u2f/tests/ directory.
Priority: -- → P1
Whiteboard: [MemShrink] → [MemShrink][u2f]
Assignee: nobody → ttaubert
Status: NEW → ASSIGNED
The problem here is that BackgroundChildImpl::DeallocPWebAuthnTransactionChild() always casts the actor to a WebAuthnTransactionChild, even if it's actually a U2FTransactionChild.
Comment on attachment 8924197 [details] Bug 1412408 - Fix content process leak of U2FTransactionChild instances r=jcj J.C. Jones [:jcj] has approved the revision. https://phabricator.services.mozilla.com/D179#4501
Attachment #8924197 - Flags: review+
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/3a5bfa2ffc98 Fix content process leak of U2FTransactionChild instances r=jcj
Thanks for the quick fix.
Sure, thanks for spotting it!
You need to log in before you can comment on or make changes to this bug.