Closed Bug 240533 Opened 20 years ago Closed 20 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: 20 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: