MLK: File channel or protocol handler?

VERIFIED FIXED in M11

Status

()

Core
Networking
P3
major
VERIFIED FIXED
19 years ago
18 years ago

People

(Reporter: Bruce Mitchener, Assigned: brendan)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

19 years ago
This was from a September 7, 1999 build on NT.

[W] MLK: Memory leak of 20 bytes from 1 block allocated in
nsAutoMonitor::NewMonitor(char const*)
        Distribution of leaked blocks
                20 bytes from 1 block of 20 bytes (0x070116f8) allocation number
106131
        Allocation location
            new(UINT)      [new.cpp:23]
            nsAutoMonitor::NewMonitor(char const*) [nsAutoLock.cpp:143]
            nsFileChannel::Init(nsFileProtocolHandler *,char const*,nsIURI
*,nsILoadGroup *,nsIEventSinkGetter *) [nsFileChannel.cpp:110]
            nsFileProtocolHandler::NewChannel(char const*,nsIURI *,nsILoadGroup
*,nsIEventSinkGetter *,nsIChannel * *) [nsFileProtocolHandler.cpp:162]
            nsIOService::NewChannelFromURI(char const*,nsIURI *,nsILoadGroup
*,nsIEventSinkGetter *,nsIChannel * *) [nsIOService.cpp:229]
            nsIOService::NewChannel(char const*,char const*,nsIURI
*,nsILoadGroup *,nsIEventSinkGetter *,nsIChannel * *) [nsIOService.cpp:247]
            nsResourceProtocolHandler::NewChannel(char const*,nsIURI
*,nsILoadGroup *,nsIEventSinkGetter *,nsIChannel * *)
[nsResourceProtocolHandler.cpp:321]
            nsIOService::NewChannelFromURI(char const*,nsIURI *,nsILoadGroup
*,nsIEventSinkGetter *,nsIChannel * *) [nsIOService.cpp:229]
            nsChromeProtocolHandler::NewChannel(char const*,nsIURI
*,nsILoadGroup *,nsIEventSinkGetter *,nsIChannel * *)
[nsChromeProtocolHandler.cpp:176]
            nsIOService::NewChannelFromURI(char const*,nsIURI *,nsILoadGroup
*,nsIEventSinkGetter *,nsIChannel * *) [nsIOService.cpp:229]
            NS_OpenURI(nsIChannel * *,nsIURI *,nsILoadGroup *,nsIEventSinkGetter
*) [nsNeckoUtil.cpp:63]
            nsUnicharStreamLoader::nsUnicharStreamLoader(nsIURI *,nsILoadGroup
*,(*)(nsIUnicharStreamLoader *,nsString&,void *,UINT),void *,UINT *)
[nsNetStreamLoader.cpp:80]
            NS_NewUnicharStreamLoader(nsIUnicharStreamLoader * *,nsIURI
*,nsILoadGroup *,(*)(nsIUnicharStreamLoader *,nsString&,void *,UINT),void *)
[nsNetStreamLoader.cpp:211]
            XULContentSinkImpl::OpenScript(nsIParserNode const&)
[nsXULContentSink.cpp:1605]
            XULContentSinkImpl::OpenTag(nsIParserNode const&)
[nsXULContentSink.cpp:1417]

Updated

19 years ago
Assignee: gagan → warren

Updated

19 years ago
Status: NEW → ASSIGNED

Comment 1

19 years ago
Maybe the XULContentSinkImpl isn't releasing the nsIUnicharStreamLoader here.

Updated

19 years ago
Assignee: warren → brendan
Status: ASSIGNED → NEW
Target Milestone: M11

Comment 2

19 years ago
This is brendan's.
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED

Comment 3

18 years ago
Bruce, what's the good word with that patch I gave you (attached)?

/be

Comment 4

18 years ago
Created attachment 1703 [details] [diff] [review]
free deadlock detection stuff
(Reporter)

Comment 5

18 years ago
running with that patch drops the leak count on viewer to about 1300 bytes for
startup/load test 0/shutdown from roughly 2000 bytes.
(Reporter)

Comment 6

18 years ago
(that was all of the instances of this leak btw)
(Assignee)

Updated

18 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 7

18 years ago
Fix checked in.  Verify via purify on your own, or from bruce's logs.

/be

Updated

18 years ago
Status: RESOLVED → VERIFIED

Comment 8

18 years ago
Bulk move of all Necko (to be deleted component) bugs to new Networking

component.
You need to log in before you can comment on or make changes to this bug.