Closed Bug 4346 Opened 25 years ago Closed 25 years ago

MLK: 72 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: 72 bytes leaked in 2 blocks
  * This memory was allocated from:
        malloc         [rtlib.o]
        __bUiLtIn_nEw  [libgcc.a]
        __builtin_new  [rtlib.o]
        NS_NewISupportsArray(nsISupportsArray**) [nsSupportsArray.cpp:352]

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

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::ConstructFrameByDisplayType(nsIPresContext*,const
nsStyleDisplay*,nsIContent*,nsIFrame*,nsIStyleContext*,nsAbsoluteItems&,nsFrameI
tems&,nsAbsoluteItems&,nsAbsoluteItems&) [nsCSSFrameConstructor.cpp:2680]

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

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]
        stub_complete(_NET_StreamClass*) [nsStubContext.cpp:585]
        net_ProcessFile [mkfile.c:1356]
        NET_ProcessNet [mkgeturl.c:3312]
        NET_PollSockets [mkselect.c:298]
        nsNetlibService::NetPollSocketsCallback(nsITimer*,void*)
[nsNetService.cpp:1220]
        TimerImpl::FireTimeout() [nsTimer.cpp:73]
  * Block of 36 bytes (2 times); last block at 0x75e8f0
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: 72 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 are not showing up in the leak logs or in recent Purify runs. 
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.