nsPref's constructor leaks mDefaultBranch

VERIFIED FIXED in mozilla0.9.4

Status

()

P2
normal
VERIFIED FIXED
18 years ago
18 years ago

People

(Reporter: dbaron, Assigned: dbaron)

Tracking

({memory-leak})

Trunk
mozilla0.9.4
x86
Linux
memory-leak
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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

Comment 2

18 years ago
We really need to get rid of nsPref...

r=bnesse.

Comment 3

18 years ago
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
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 6

18 years ago
david, brian.  How would one verify this bug?

Comment 7

18 years ago
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.