50863 - MLK: leaking CNavDTD

Status: VERIFIED WORKSFORME

(Core :: DOM: HTML Parser, defect, P1)

Description

[Bruce Mitchener]

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]

Comment 1

[Michael La Guardia]

cc'ing Jud.  Jud, could you make the call on whether this is nsbeta3 + or -?
Thanks.

Comment 2

[Judson Valeski]

we'll suck this into the trunk.

Comment 3

[bsharma]

updated qa contact.

Comment 4

[Christopher Hoess (gone)]

->harishd

Comment 5

[harishd]

I don't see the leak now. Marking WFM.

Comment 6

[Moied]

Marking verified WFM per comment #5 From harishd