mailnews backend has a few memory leaks

RESOLVED FIXED

Status

--
minor
RESOLVED FIXED
15 years ago
11 years ago

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Tracking

({memory-leak})

Trunk
x86
Windows 2000
memory-leak

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

15 years ago
When running Purify, I encountered a few small memory leaks. Fix upcoming.
(Assignee)

Comment 1

15 years ago
Created attachment 137458 [details] [diff] [review]
proposed fix

I'm not particularly happy about the charset fix (the code should be rewritten
to avoid allocating charsets it's just going to overwrite) but this fix is
safer for 1.6 final
(Assignee)

Updated

15 years ago
Attachment #137458 - Flags: superreview?(mscott)
Attachment #137458 - Flags: review?(sspitzer)

Updated

15 years ago
Attachment #137458 - Flags: superreview?(mscott) → superreview+
Comment on attachment 137458 [details] [diff] [review]
proposed fix

r/a=sspitzer for 1.6 final
Attachment #137458 - Flags: review?(sspitzer)
Attachment #137458 - Flags: review+
Attachment #137458 - Flags: approval1.6+

Updated

15 years ago
Severity: normal → critical
(Assignee)

Comment 3

15 years ago
actually, the severity is pretty minor.  They're not big memory leaks, nor frequent.
Severity: critical → minor
(Assignee)

Comment 4

15 years ago
fix checked in.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.