Closed Bug 240533 Opened 21 years ago Closed 21 years ago

memory leak in nsImapUrl::AddOnlineDirectoryIfNecessary

Categories

(MailNews Core :: Networking: IMAP, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Details

(Keywords: fixed1.7, memory-leak)

Attachments

(1 file)

If you specify an imap online directory, we'll leak that string for every imap url we run.
Attached patch proposed fixSplinter Review
use an nsCAutoString to fix leak.
Attachment #146121 - Flags: superreview?(mscott)
Attachment #146121 - Flags: superreview?(mscott) → superreview+
Attachment #146121 - Flags: approval1.7?
fixed on trunk.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Comment on attachment 146121 [details] [diff] [review] proposed fix a=asa (on behalf of drivers) for checkin to 1.7
Attachment #146121 - Flags: approval1.7? → approval1.7+
Keywords: fixed1.7
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: