Status

()

Core
HTML: Parser
P1
normal
VERIFIED WORKSFORME
18 years ago
16 years ago

People

(Reporter: Bruce Mitchener, Assigned: harishd)

Tracking

({memory-leak})

Trunk
mozilla1.0
memory-leak
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [need info])

(Reporter)

Description

18 years ago
This happens on www.visa-asia.com.  Build from last night.

This is taking down 2 DTD context objects with it, which are 972 bytes a piece or 
so as well.

         MLK: 620 bytes leaked at 0x143ddd0
         This memory was allocated from:
               malloc         [rtlib.o]
               __bUiLtIn_nEw  [libxpcom.so]
               __builtin_new  [rtlib.o]
               CNavDTDConstructor(nsISupports*,const nsID&,void**) [nsParserModule.cpp:139]
                  NS_GENERIC_FACTORY_CONSTRUCTOR(nsCParserNode)
                  NS_GENERIC_FACTORY_CONSTRUCTOR(nsLoggingSink)
                  NS_GENERIC_FACTORY_CONSTRUCTOR(CWellFormedDTD)
               => NS_GENERIC_FACTORY_CONSTRUCTOR(CNavDTD)
                  NS_GENERIC_FACTORY_CONSTRUCTOR(nsXIFDTD)
                  NS_GENERIC_FACTORY_CONSTRUCTOR(COtherDTD)
                  NS_GENERIC_FACTORY_CONSTRUCTOR(CViewSourceHTML)
               nsGenericFactory::CreateInstance(nsISupports*,const nsID&,void**) [nsGenericFactory.cpp:47]
               nsComponentManagerImpl::CreateInstance(const nsID&,nsISupports*,const nsID&,void**) [nsComponentManager.cpp:1194]
               nsComponentManager::CreateInstance(const nsID&,nsISupports*,const nsID&,void**) [nsRepository.cpp:81]
               NS_NewNavHTMLDTD(nsIDTD**) [CNavDTD.h:543]
               nsHTMLDocument::OpenCommon(nsIURI*) [nsHTMLDocument.cpp:2095]
               nsHTMLDocument::Open(JSContext*,long*,unsigned int) [nsHTMLDocument.cpp:2139]
               NSHTMLDocumentOpen(JSContext*,JSObject*,unsigned int,long*,long*) [nsJSHTMLDocument.cpp:847]
               js_Invoke      [jsinterp.c:716]
               js_Interpret   [jsinterp.c:2517]
               js_Invoke      [jsinterp.c:732]
               js_InternalInvoke [jsinterp.c:805]
               JS_CallFunctionValue [jsapi.c:3147]
               nsJSContext::CallEventHandler(void*,void*,unsigned int,void*,int*,int) [nsJSEnvironment.cpp:860]
               nsJSEventListener::HandleEvent(nsIDOMEvent*) [nsJSEventListener.cpp:154]
               nsEventListenerManager::HandleEventSubType(nsListenerStruct*,nsIDOMEvent*,nsIDOMEventTarget*,unsigned int,unsigned int) [nsEventListenerManager.cpp:788]
               nsEventListenerManager::HandleEvent(nsIPresContext*,nsEvent*,nsIDOMEvent**,nsIDOMEventTarget*,unsigned int,nsEventStatus*) [nsEventListenerManager.cpp:1361]
               GlobalWindowImpl::HandleDOMEvent(nsIPresContext*,nsEvent*,nsIDOMEvent**,unsigned int,nsEventStatus*) [nsGlobalWindow.cpp:453]
               nsWebShell::OnEndDocumentLoad(nsIDocumentLoader*,nsIChannel*,unsigned int) [nsWebShell.cpp:963]
               nsDocLoaderImpl::FireOnEndDocumentLoad(nsDocLoaderImpl*,nsIChannel*,unsigned int) [nsDocLoader.cpp:803]
               nsDocLoaderImpl::DocLoaderIsEmpty(unsigned int) [nsDocLoader.cpp:609]
               nsDocLoaderImpl::DocLoaderIsEmpty(unsigned int) [nsDocLoader.cpp:612]
(Reporter)

Updated

18 years ago
Keywords: mlk, nsbeta3

Comment 1

18 years ago
cc'ing Jud.  Jud, could you make the call on whether this is nsbeta3 + or -?
Thanks.
Whiteboard: [need info]

Comment 2

18 years ago
we'll suck this into the trunk.

Comment 3

17 years ago
updated qa contact.
QA Contact: janc → bsharma

Updated

17 years ago
Blocks: 92580

Updated

17 years ago
QA Contact: bsharma → moied

Updated

17 years ago
No longer blocks: 92580
->harishd
Assignee: rickg → harishd
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED
Keywords: nsbeta3 → nsbeta1
Priority: P3 → P1
Target Milestone: --- → mozilla1.0
Keywords: nsbeta1 → nsbeta1+
(Assignee)

Comment 5

16 years ago
I don't see the leak now. Marking WFM.
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → WORKSFORME

Comment 6

16 years ago
Marking verified WFM per comment #5 From harishd
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.