memory leak in nsImapUrl::AddOnlineDirectoryIfNecessary

RESOLVED FIXED

Status

RESOLVED FIXED
15 years ago
10 years ago

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Tracking

({fixed1.7, memory-leak})

Trunk
x86
Windows 2000
fixed1.7, memory-leak

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

15 years ago
If you specify an imap online directory, we'll leak that string for every imap
url we run.
(Assignee)

Comment 1

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

use an nsCAutoString to fix leak.
(Assignee)

Updated

15 years ago
Attachment #146121 - Flags: superreview?(mscott)

Updated

15 years ago
Attachment #146121 - Flags: superreview?(mscott) → superreview+
(Assignee)

Updated

15 years ago
Attachment #146121 - Flags: approval1.7?
(Assignee)

Comment 2

15 years ago
fixed on trunk.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED

Comment 3

15 years ago
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+
(Assignee)

Updated

15 years ago
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.