Closed Bug 4347 Opened 25 years ago Closed 25 years ago

MLK: 36 bytes leaked CSS leak - startup/shutdown

Categories

(Core :: CSS Parsing and Computation, defect, P3)

Product:
Core
Component:
CSS Parsing and Computation
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: bruce, Assigned: attinasi)

References

Details

(Keywords: memory-leak, verifyme, Whiteboard: 4/24: requested verification by reporter or assigned engineer. waiting for apprunner document leaks to be fixed) 

Build, pull from March 28, 1999.  Happens in apprunner startup/shutdown process
with only default chrome loading.

MLK: 36 bytes leaked in 2 blocks
  * This memory was allocated from:
        malloc         [rtlib.o]
        __bUiLtIn_nEw  [libgcc.a]
        __builtin_new  [rtlib.o]
        __bUiLtIn_vEc_nEw [libgcc.a]
        __builtin_vec_new [rtlib.o]
        nsString::EnsureCapacityFor(int) [nsString.cpp:314]
        nsString::nsString(const nsString&) [nsString.cpp:210]
        nsFont::nsFont(const nsFont&) [nsFont.cpp:45]
        nsStyleFont::nsStyleFont(const nsFont&,const nsFont&)
[nsStyleContext.cpp:49]
        StyleFontImpl::StyleFontImpl(const nsFont&,const nsFont&)
[nsStyleContext.cpp:57]

StyleContextImpl::StyleContextImpl(nsIStyleContext*,nsIAtom*,nsISupportsArray*,n
sIPresContext*) [nsStyleContext.cpp:913]

NS_NewStyleContext(nsIStyleContext**,nsIStyleContext*,nsIAtom*,nsISupportsArray*
,nsIPresContext*) [nsStyleContext.cpp:1514]

StyleSetImpl::GetContext(nsIPresContext*,nsIStyleContext*,nsIAtom*,nsISupportsAr
ray*,int,int&) [nsStyleSet.cpp:453]

StyleSetImpl::ResolveStyleFor(nsIPresContext*,nsIContent*,nsIStyleContext*,int)
[nsStyleSet.cpp:546]

nsPresContext::ResolveStyleContextFor(nsIContent*,nsIStyleContext*,int,nsIStyleC
ontext**) [nsPresContext.cpp:384]

nsCSSFrameConstructor::ResolveStyleContext(nsIPresContext*,nsIFrame*,nsIContent*
,nsIAtom*,nsIStyleContext**) [nsCSSFrameConstructor.cpp:2838]

nsCSSFrameConstructor::ConstructFrame(nsIPresContext*,nsIContent*,nsIFrame*,nsAb
soluteItems&,nsFrameItems&,nsAbsoluteItems&,nsAbsoluteItems&)
[nsCSSFrameConstructor.cpp:2861]

nsCSSFrameConstructor::ProcessChildren(nsIPresContext*,nsIContent*,nsIFrame*,nsA
bsoluteItems&,nsFrameItems&,nsAbsoluteItems&,nsAbsoluteItems&)
[nsCSSFrameConstructor.cpp:275]

nsCSSFrameConstructor::ConstructDocElementFrame(nsIPresContext*,nsIContent*,nsIF
rame*,nsIStyleContext*,nsIFrame*&,nsAbsoluteItems&)
[nsCSSFrameConstructor.cpp:1456]

nsCSSFrameConstructor::ContentInserted(nsIPresContext*,nsIContent*,nsIContent*,i
nt) [nsCSSFrameConstructor.cpp:3318]

StyleSetImpl::ContentInserted(nsIPresContext*,nsIContent*,nsIContent*,int)
[nsStyleSet.cpp:747]
        PresShell::InitialReflow(int,int) [nsPresShell.cpp:850]
        XULDocumentImpl::StartLayout() [nsXULDocument.cpp:3088]
        XULDocumentImpl::EndLoad() [nsXULDocument.cpp:1413]
        XULContentSinkImpl::DidBuildModel(int) [nsXULContentSink.cpp:426]
        CWellFormedDTD::DidBuildModel(unsigned
int,int,nsIParser*,nsIContentSink*) [nsWellFormedDTD.cpp:288]
        nsParser::DidBuildModel(unsigned int) [nsParser.cpp:443]
        nsParser::ResumeParse(nsIDTD*) [nsParser.cpp:760]
        nsParser::OnStopBinding(nsIURL*,unsigned int,const unsigned short*)
[nsParser.cpp:984]
        nsDocumentBindInfo::OnStopBinding(nsIURL*,unsigned int,const unsigned
short*) [nsDocLoader.cpp:1806]
  * Block of 18 bytes (2 times); last block at 0x7641a0
Probably caused by 4336.
Status: NEW → ASSIGNED
Target Milestone: M5
Apprunner is currently leaking the entire document. Deferring until I can even
see these leaks again...
Whiteboard: suspect leaks have been fixed
It seems that RDF generated documents are *still* leaking entirely, not
suprising that all the style data leaks with them. Deferring again until major
leaks are corrected in apprunner.
Whiteboard: suspect leaks have been fixed → waiting for apprunner document leaks to be fixed
Target Milestone: M10 → M11
Pushing off non-beta 1 issues
Depends on: 13283
Summary: MLK: CSS leak → MLK: 36 bytes leaked CSS leak - startup/shutdown
Blocks: 14516
Reassigning peterl's bugs to myself.
Accepting peterl's bugs that have a Target Milestone
Pushing my M15 bugs to M16
Keywords: mlk
Block-moved memory leaks to attinasi
Assignee: pierre → attinasi
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
These leaks ae not showing up in latest leak logs or recent Purify run. Marking 
fixed.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Keywords: verifyme
Mark or Bruce:

I am unable to verify leaks. Could one of you please marked this as verified? 
Thanks
Whiteboard: waiting for apprunner document leaks to be fixed → 4/24: requested verification by reporter or assigned engineer. waiting for apprunner document leaks to be fixed
Verified per Marc's comments.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.