Closed
Bug 228561
Opened 21 years ago
Closed 21 years ago
mailnews backend has a few memory leaks
Categories
(MailNews Core :: Backend, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Bienvenu, Assigned: Bienvenu)
Details
(Keywords: memory-leak)
Attachments
(1 file)
|
2.50 KB,
patch
|
sspitzer
:
review+
mscott
:
superreview+
sspitzer
:
approval1.6+
|
Details | Diff | Splinter Review |
When running Purify, I encountered a few small memory leaks. Fix upcoming.
|
Assignee | |
Comment 1•21 years ago
|
||
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•21 years ago
|
Attachment #137458 -
Flags: superreview?(mscott)
Attachment #137458 -
Flags: review?(sspitzer)
|
||
Updated•21 years ago
|
Attachment #137458 -
Flags: superreview?(mscott) → superreview+
|
||
Comment 2•21 years ago
|
||
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•21 years ago
|
Severity: normal → critical
|
|
Assignee | |
Comment 3•21 years ago
|
||
actually, the severity is pretty minor. They're not big memory leaks, nor frequent.
Severity: critical → minor
|
|
Assignee | |
Comment 4•21 years ago
|
||
fix checked in.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
|
||
Updated•20 years ago
|
Product: MailNews → Core
|
||
Updated•17 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•