Closed Bug 50663 Opened 24 years ago Closed 24 years ago

[MLK] Leaking C string in nsMsgCompose::CheckAndPopulateRecipients()

Categories

(MailNews Core :: Composition, defect, P3)

Product:
MailNews Core
Component:
Composition
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: beard, Assigned: bugzilla)

References

(
URL
)

Details

(Keywords: memory-leak)

Attachments

(3 files)

proposed diff
(deleted), patch
Details | Diff | Splinter Review
newest revision: should fix all possible leaks when using GetDirUri
(deleted), patch
Details | Diff | Splinter Review
implements sspitzer's suggestions
(deleted), patch
Details | Diff | Splinter Review
The line in the URL above is leaking the C string pointed to by the uri variable.
Keywords: mlk
Accepting
Status: NEW → ASSIGNED
Target Milestone: --- → M19
Blocks: 51177
Attached patch proposed diff (deleted) — — Splinter Review
Keywords: patch, review
r=beard
sr=bienvenu
adding myself to the cc list. r=sspitzer.
who will do the check in?
I told Jarrod that I would do the checkin.
fix checked in
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
QA Contact: lchiang → stephend
Verified FIXED using Purify. I did -compose, typed and let a recipient autocomplete, clicked the address button and Cc:ed a recipient from my Personal Address Book, and then sent the message.
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

Creator:
Created:
Updated:
Size: