os2Embed CreateBrowserWindow can leak in some failure cases

RESOLVED WONTFIX

Status

()

Core
Embedding: APIs
--
trivial
RESOLVED WONTFIX
13 years ago
9 years ago

People

(Reporter: timeless, Assigned: mkaply)

Tracking

({memory-leak})

Trunk
x86
OS/2
memory-leak
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

13 years ago
I'm sorta auditing impls of CreateChromeWindow, and yours leaks (which is probably better than crashing).

I'm also not certain that the general refcounting is correct for this object. for bonus points, report OOM using NS_ERROR_OUT_OF_MEMORY instead of NS_ERROR_FAILURE
(Assignee)

Comment 1

13 years ago
This code is literally copied straight out of winEmbed.cpp.

Is there a bug for that?

Comment 2

11 years ago
So this is the OS/2 version of bug 321032 for Windows. If anybody ever fixes that we can then take that fix...
Depends on: 321032

Comment 3

9 years ago
We are going to remove os2Embed in bug 507461. --> WONTFIX
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.