Closed
Bug 140087
Opened 22 years ago
Closed 22 years ago
Avoid the leak of nsPluginStreamListenerPeer obj when NS_OpenURI() failed
Categories
(Core Graveyard :: Plug-ins, defect)
Core Graveyard
Plug-ins
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: srgchrpv, Assigned: srgchrpv)
Details
Attachments
(2 files)
217 bytes,
text/html
|
Details | |
1.25 KB,
patch
|
serhunt
:
review+
|
Details | Diff | Splinter Review |
I suddenly run into this leak while tested bug 139563 the test case & patch are following
Assignee | ||
Comment 1•22 years ago
|
||
because file:///somefile.vaw cannot be found rv = NS_OpenURI() fails in nsPluginHostImpl::NewEmbededPluginStream()
Assignee | ||
Comment 2•22 years ago
|
||
if necko does not addRef on this listener, add & release here will delete the object. Av, could you r= on this, please?
Comment on attachment 81035 [details] [diff] [review] patch v1 r=av, we have NS_ENSURE_ARGUMENT_POINTER macro, can make a use of it here.
Attachment #81035 -
Flags: review+
Comment 4•22 years ago
|
||
this works now
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Updated•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•