Closed
Bug 59261
Opened 25 years ago
Closed 24 years ago
leak string in nsMsgComposeAndSend::GetBodyFromEditor
Categories
(MailNews Core :: Composition, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: dbaron, Assigned: bugzilla)
Details
(Keywords: memory-leak)
Attachments
(2 files)
|
13.21 KB,
text/html
|
Details | |
|
2.60 KB,
patch
|
Details | Diff | Splinter Review |
Running using the Boehm GC, I sent a message to myself by clicking on a mailto:
link to myself (so it's a text/plain message, although the string leaked was
HTML). This showed that the object in the attached snippet was leaked. The
leak seems to me to be in nsMsgComposeAndSend::GetBodyFromEditor, where
nsMsgI18NSaveAsCharset is called on line 1301, and outCString is never freed
after SnarfAndCopyBody at the end of the message.
|
Reporter | |
Comment 1•25 years ago
|
||
|
Assignee | |
Comment 3•24 years ago
|
||
|
|
Assignee | |
Updated•24 years ago
|
Whiteboard: Fix in hand
|
||
Comment 4•24 years ago
|
||
sr=mscott
|
|
||
Comment 5•24 years ago
|
||
r=alecf
|
|
Assignee | |
Comment 6•24 years ago
|
||
Fixed and checked in
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Whiteboard: Fix in hand
David, could you possibly check to see if this is still happening and verify
this bug if fixed. This would be greatly appreciated. Thanks!
QA Contact: esther → stephend
I saw no leaks here when clicking on my own mailto: link on
http://people.netscape.com/stephend/, using Purify on Windows 2000 (saw a bunch
of what I think is Hyatt's new style code leaking TextRun, but that's diff.)
Trunk CVS pull/opt build.
Status: RESOLVED → VERIFIED
|
||
Updated•21 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
•