[MLK] Leaking PRUnichar in DIR_GetLocalizedStringPref()

VERIFIED FIXED

Status

SeaMonkey
MailNews: Address Book & Contacts
P3
normal
VERIFIED FIXED
18 years ago
13 years ago

People

(Reporter: Patrick C. Beard, Assigned: chuang)

Tracking

({mlk})

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

18 years ago
Look at the URL above, the string in wvalue is leaking.
(Reporter)

Updated

18 years ago
Keywords: mlk

Comment 1

18 years ago
You probably want to use an |nsXPIDLString| for this, and wrap it with a
|getter_Copies| when you pass it as a result parameter to be filled in.

Comment 2

17 years ago
reassigning to chuang.
Assignee: putterman → chuang
Keywords: mail1
(Assignee)

Comment 3

17 years ago
Created attachment 19219 [details] [diff] [review]
patch file for fix

Comment 4

17 years ago
sr=bienvenu
(Assignee)

Comment 5

17 years ago
Fix checked in
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 6

17 years ago
Per Esther, assign this bug to her.
QA Contact: lchiang → esther
QA Contact: esther → stephend
Didn't see a leak reported against this when running Purify under Win2k on an
opt CVS trunk build.  (Saw lots of UMR's in TextRun, but I think that's Hyatt's
CSS style-foo).
Status: RESOLVED → VERIFIED

Comment 8

17 years ago
Stephen, UMR's are uninitialized memory reads, not memory leaks
Right, I'm just saying that's all I saw in the output ;-)
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.