Closed Bug 39856 Opened 25 years ago Closed 25 years ago

MLK: nsHTMLInputElement

Categories

(Core :: DOM: Core & HTML, defect, P3)

defect

Tracking

()

VERIFIED DUPLICATE of bug 36639

People

(Reporter: bruce, Assigned: nisheeth_mozilla)

References

()

Details

(Keywords: memory-leak)

We leak an nsHTMLInputElement when loading http://www.cnn.com/ I think that's because of http://lxr.mozilla.org/seamonkey/source/layout/html/content/src/nsHTMLFormElement.cpp#1008 and the seemingly insane refcounting in there. The QueryInterface adds a ref unconditionally, yet there is only a conditional release. What's up?
Keywords: mlk
Niseeth, you wrote this code, so I will let figure out whether the lea is coming from here. Thanks.
Assignee: rods → nisheeth
Related to bug 36639? There is an attempt at a fix there, but it causes new leaks for whatever reason...
This is a duplicate of / will be fixed by bug 36639 (see comments there).
Eric's got a fix for 36639. Marking this as a dup... *** This bug has been marked as a duplicate of 36639 ***
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → DUPLICATE
Updating QA contact.
QA Contact: ckritzer → vladimire
Verified dupe based on patch in bug 36639.
Status: RESOLVED → VERIFIED
Component: HTML: Form Submission → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.