Closed Bug 93087 Opened 18 years ago Closed 18 years ago

nsPref's constructor leaks mDefaultBranch

Categories

(Core :: Preferences: Backend, defect, P2)

x86
Linux
defect

Tracking

()

VERIFIED FIXED
mozilla0.9.4

People

(Reporter: dbaron, Assigned: dbaron)

Details

(Keywords: memory-leak)

Attachments

(1 file)

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.
Status: NEW → ASSIGNED
Keywords: mlk
Priority: -- → P2
Target Milestone: --- → mozilla0.9.4
We really need to get rid of nsPref...

r=bnesse.
sr=waterson
rodney, feel free to punt back if you don't want this one!
QA Contact: sairuh → rvelasco
Fix checked in 2001-08-02 19:36 PDT.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
david, brian.  How would one verify this bug?
It's a code only change. There's really no QA quantifiable to validate it.
okay then --rubberstamp. :)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.