Closed
Bug 132126
Opened 23 years ago
Closed 23 years ago
nsFactoryEntry::ReInit leaks memory
Categories
(Core :: XPCOM, defect, P3)
Tracking
()
RESOLVED
FIXED
mozilla1.0
People
(Reporter: Biesinger, Assigned: Biesinger)
References
()
Details
(Keywords: memory-leak)
Attachments
(1 file)
|
691 bytes,
patch
|
dougt
:
review+
hewitt
:
superreview+
scc
:
approval+
|
Details | Diff | Splinter Review |
According to the URL above, there's a memory leak in nsFactoryEntry::ReInit:
430 location = nsCRT::strdup(aLocation);
But it is not checked if location already contains data.
Will attach a patch in a second.
|
Assignee | |
Comment 1•23 years ago
|
||
|
|
Assignee | |
Updated•23 years ago
|
|
|
Assignee | |
Comment 2•23 years ago
|
||
dougt, could you review?
|
||
Comment 3•23 years ago
|
||
Comment on attachment 75032 [details] [diff] [review]
patch
r=dougt
Attachment #75032 -
Flags: review+
|
||
Comment 4•23 years ago
|
||
Comment on attachment 75032 [details] [diff] [review]
patch
sr=hewitt
Attachment #75032 -
Flags: superreview+
|
|
||
Comment 5•23 years ago
|
||
Comment on attachment 75032 [details] [diff] [review]
patch
a=scc
Attachment #75032 -
Flags: approval+
|
|
Assignee | |
Comment 6•23 years ago
|
||
checked in
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
*** Bug 111296 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•