Closed
Bug 93087
Opened 24 years ago
Closed 24 years ago
nsPref's constructor leaks mDefaultBranch
Categories
(Core :: Preferences: Backend, defect, P2)
Tracking
()
VERIFIED
FIXED
mozilla0.9.4
People
(Reporter: dbaron, Assigned: dbaron)
Details
(Keywords: memory-leak)
Attachments
(1 file)
|
1003 bytes,
patch
|
Details | Diff | Splinter Review |
nsPref's constructor leaks mDefaultBranch by never releasing the reference
obtained by the |GetDefaultBranch| call (since the nsCOMPtr mDefaultBranch
|AddRef|s again when assigned into).
Patch coming.
|
Assignee | |
Updated•24 years ago
|
|
|
Assignee | |
Comment 1•24 years ago
|
||
|
||
Comment 2•24 years ago
|
||
We really need to get rid of nsPref...
r=bnesse.
|
||
Comment 3•24 years ago
|
||
sr=waterson
|
||
Comment 4•24 years ago
|
||
rodney, feel free to punt back if you don't want this one!
QA Contact: sairuh → rvelasco
|
|
Assignee | |
Comment 5•24 years ago
|
||
Fix checked in 2001-08-02 19:36 PDT.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
|
||
Comment 6•23 years ago
|
||
david, brian. How would one verify this bug?
|
|
||
Comment 7•23 years ago
|
||
It's a code only change. There's really no QA quantifiable to validate it.
You need to log in
before you can comment on or make changes to this bug.
Description
•