Closed Bug 1381365 Opened 8 years ago Closed 8 years ago

Crash in nsNPAPIPluginInstance::~nsNPAPIPluginInstance

Categories

(Core Graveyard :: Plug-ins, defect)

Product:
Core Graveyard
Component:
Plug-ins
Platform:
Unspecified
Android
Type:
defect
Priority:
Not set
Severity:
critical

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: n.nethercote, Unassigned)

Details

(Keywords: crash)

Crash Data

This bug was filed from the Socorro interface and is report bp-6ad3cd7b-95c9-4e49-8831-931630170715. ============================================================= This crash has been around for a while, and is showing up in Fennec 55 and 56. The crash is in ~nsNPAPIPluginInstance(), but the stack trace includes xpcom/string/nsTSubstring.h:347, which suggests the problem is with destruction of a string object within nsNPAPIPluginInstance. The only such string object is mFakeURL, which isn't used much. Although there is this line: http://searchfox.org/mozilla-central/rev/01d27fdd3946f7210da91b18fcccca01d7324fe2/dom/plugins/base/nsNPAPIPlugin.cpp#1375 which doesn't seem likely to be the cause of this crash, but I don't understand how it's supposed to work or even how it compiles. jimm, any ideas about this one?
Flags: needinfo?(jmathies)
We're going to disable plugins in Fennec maybe as soon as 55. I'd suggest not spending time on actually figuring this out.
Crash Signature: [@ nsNPAPIPluginInstance::~nsNPAPIPluginInstance] → [@ nsNPAPIPluginInstance::~nsNPAPIPluginInstance] [@ 0x0 | nsNPAPIPluginInstance::~nsNPAPIPluginInstance]
Crash Signature: [@ nsNPAPIPluginInstance::~nsNPAPIPluginInstance] [@ 0x0 | nsNPAPIPluginInstance::~nsNPAPIPluginInstance] → [@ nsNPAPIPluginInstance::~nsNPAPIPluginInstance] [@ 0x0 | nsNPAPIPluginInstance::~nsNPAPIPluginInstance] [@ @0x0 | nsNPAPIPluginInstance::~nsNPAPIPluginInstance]
Hey James, Benjamin suggested ignoring this but the crash looks pretty bad. 100% on android. Can you help triage this?
Flags: needinfo?(jmathies) → needinfo?(snorp)
Plugins are disabled in 56, so this crash is fixed there.
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(snorp)
Resolution: --- → WONTFIX
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.