33161 - MLK: char* data from ProcessBodyAsAttachment()

Status: VERIFIED FIXED

(MailNews Core :: MIME, defect, P3)

Description

[Bruce Mitchener]

In ProcessBodyAsAttachment(), you need to free the result of
mime_set_url_part().

malloc
PR_Malloc
mime_set_url_part(const char*, char*, int)
ProcessBodyAsAttachment(MimeObject*, nsMsgAttachmentData**)
MimeGetAttachmentList
mime_display_stream_complete
nsStreamConverter::OnStopRequest(nsIChannel*, nsISupports*, unsigned int, const
unsigned short*)
nsDocumentOpenInfo::OnStopRequest(nsIChannel*, nsISupports*, unsigned int, const
unsigned short*)
nsMsgProtocol::OnStopRequest(nsIChannel*, nsISupports*, unsigned int, const
unsigned short*)
nsMailboxProtocol::OnStopRequest(nsIChannel*, nsISupports*, unsigned int, const
unsigned short*)
nsFileChannel::OnStopRequest(nsIChannel*, nsISupports*, unsigned int, const
unsigned short*)
nsOnStopRequestEvent::HandleEvent()
nsStreamListenerEvent::HandlePLEvent(PLEvent*)
PL_HandleEvent
PL_ProcessPendingEvents
nsEventQueueImpl::ProcessPendingEvents()
nsEventQueueImpl::ProcessPendingEvents()
nsEventQueueImpl::ProcessPendingEvents()
nsMacNSPREventQueueHandler::ProcessPLEventQueue()
nsMacNSPREventQueueHandler::RepeatAction(const EventRecord&)
Repeater::DoRepeaters(const EventRecord&)
nsMacMessagePump::DispatchEvent(int, EventRecord*)
nsMacMessagePump::DoMessagePump()
nsAppShell::Run()
nsAppShellService::Run()
main1(int, char**, nsISplashScreen*)
main
__NSInitialize(1F1D7564)

Comment 1

[rhp (gone)]

Thanks for the help bruce. These should be all better now :-)

- rhp

Comment 2

[rhp (gone)]

I know this was one of mine, but since this is a code level review, I figured 
that I could verify that this change was checked in to fix this problem.

- rhp