MLK: Memory leak of 5 bytes -nsBrowserAppCore::OnEndDocumentLoad()

RESOLVED FIXED in M11

Status

SeaMonkey
General
P3
normal
RESOLVED FIXED
19 years ago
13 years ago

People

(Reporter: Bruce Mitchener, Assigned: Radha on family leave (not reading bugmail))

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
1) char* scheme ends up shadowing itself.
2) it owns the memory it gets from the URL, so it needs to free it.

[W] MLK: Memory leak of 5 bytes from 1 block allocated in PL_strdup
        Distribution of leaked blocks
                 5 bytes from 1 block of 5 bytes (0x097b1200) allocation number
210505
        Allocation location
            malloc         [dbgheap.c:129]
            PL_strdup      [strdup.c:30]
            nsCRT::strdup(char const*) [nsCRT.h:151]
            nsStdURL::DupString(char * *,char *) [nsStdURL.cpp:673]
            nsStdURL::GetScheme(char * *) [nsStdURL.h:99]
            nsBrowserInstance::OnEndDocumentLoad(nsIDocumentLoader *,nsIChannel
*,UINT,nsIDocumentLoaderObserver *) [nsBrowserInstance.cpp:1705]
            nsWebShell::OnEndDocumentLoad(nsIDocumentLoader *,nsIChannel
*,UINT,nsIDocumentLoaderObserver *) [nsWebShell.cpp:3515]
            nsDocLoaderImpl::FireOnEndDocumentLoad(nsDocLoaderImpl *,UINT)
[nsDocLoader.cpp:1228]
            nsDocLoaderImpl::OnStopRequest(nsIChannel *,nsISupports *,UINT,WORD
const*) [nsDocLoader.cpp:1098]
            nsLoadGroup::RemoveChannel(nsIChannel *,nsISupports *,UINT,WORD
const*) [nsLoadGroup.cpp:560]
            nsHTTPChannel::ResponseCompleted(nsIChannel *,UINT)
[nsHTTPChannel.cpp:692]
            nsHTTPResponseListener::OnStopRequest(nsIChannel *,nsISupports
*,UINT,WORD const*) [nsHTTPResponseListener.cpp:260]
            nsOnStopRequestEvent::HandleEvent(void)
[nsAsyncStreamListener.cpp:268]
            nsStreamListenerEvent::HandlePLEvent(PLEvent *)
[nsAsyncStreamListener.cpp:144]
            PL_HandleEvent [plevent.c:509]
Assignee: don → radha
Target Milestone: M12

Updated

18 years ago
Summary: MLK: nsBrowserAppCore::OnEndDocumentLoad() → MLK: Memory leak of 5 bytes -nsBrowserAppCore::OnEndDocumentLoad()

Updated

18 years ago
Blocks: 14516
Status: NEW → ASSIGNED
Target Milestone: M12 → M11
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
Code modified to use nsXPIDLCString.

Updated

18 years ago
QA Contact: leger → gerardok

Comment 2

18 years ago
updating QA Contact for verification
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.