loading local message leaks a url spec

VERIFIED FIXED

Status

MailNews Core
Networking
VERIFIED FIXED
16 years ago
10 years ago

People

(Reporter: Bienvenu, Assigned: Bienvenu)

Tracking

({mlk})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

663 bytes, patch
(not reading, please use seth@sspitzer.org instead)
: superreview+
Details | Diff | Splinter Review
(Assignee)

Description

16 years ago
Here's the stack trace:

     =>   rv = GetFileFromURL(aURL, getter_AddRefs(file));
          NS_ENSURE_SUCCESS(rv, rv);
        
          nsCOMPtr<nsIFileTransportService> fts =
    nsMailboxProtocol::Initialize(nsIURI *) [nsMailboxProtocol.cpp:209]
    nsMailboxProtocol::nsMailboxProtocol(nsIURI *) [nsMailboxProtocol.cpp:92]
    nsMailboxService::NewChannel(nsIURI *,nsIChannel * *) [nsMailboxService.cpp:523]
    ???            [ip=0x0955263c]

Fix coming up. GetFileFromUrl does a pr_smprintf and never frees it.
(Assignee)

Updated

16 years ago
Status: NEW → ASSIGNED
Keywords: mlk
QA Contact: huang → stephend
(Assignee)

Comment 1

16 years ago
Created attachment 52581 [details] [diff] [review]
proposed fix
Comment on attachment 52581 [details] [diff] [review]
proposed fix

sr=sspitzer
Attachment #52581 - Flags: superreview+

Comment 3

16 years ago
r=naving
(Assignee)

Comment 4

16 years ago
fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
I didn't see this when I was verifying bug 103023 as FIXED, and that testcase
was a local message.

Verified FIXED.  Latest CVS with Windows 2000/Purify.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.