Closed Bug 240533 Opened 20 years ago Closed 20 years ago

memory leak in nsImapUrl::AddOnlineDirectoryIfNecessary

Categories

(MailNews Core :: Networking: IMAP, defect)

Product:
MailNews Core
Component:
Networking: IMAP
Platform:
x86
Windows 2000
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Details

(Keywords: fixed1.7, memory-leak)

Attachments

(1 file)

proposed fix
905 bytes, patch
mscott
: superreview+
asa
: approval1.7+
Details | Diff | Splinter Review 
If you specify an imap online directory, we'll leak that string for every imap
url we run.
Attached patch proposed fix — — Splinter 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.