MLK: 2380 bytes leaked

VERIFIED FIXED in M16

Status

()

Core
CSS Parsing and Computation
P3
critical
VERIFIED FIXED
19 years ago
10 months ago

People

(Reporter: Bruce Mitchener, Assigned: Marc Attinasi)

Tracking

({mlk})

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: 4/24: verification by reporter/assigned engineer requested.)

(Reporter)

Description

19 years ago
Solaris 2.6, build from morning of July 26, 1999.

I'm not sure whether this is a problem in style or a problem with nsString.  I
see this type of pattern showing up all over.  Look in my leak log at
http://www.cybersight.com/~bruce/apprunner-necko19990726.log for
'nsString::operator ='.  cc'ing rickg and scc in case this is nsString madness.

MLK: 2380 bytes leaked in 38 blocks
  * This memory was allocated from:
        malloc         [rtlib.o]
        PR_Malloc      [prmem.c:38]
        nsAllocatorImpl::Alloc(unsigned int) [nsAllocator.cpp:74]
        nsAllocator::Alloc(unsigned int) [nsAllocator.cpp:141]
        nsMemoryAgent::Alloc(nsStr&,unsigned int) [nsStr.h:315]
        nsMemoryAgent::Realloc(nsStr&,unsigned int) [nsStr.h:335]
        nsStr::EnsureCapacity(nsStr&,unsigned int,nsIMemoryAgent*)
[nsStr.cpp:227]
        nsStr::GrowCapacity(nsStr&,unsigned int,nsIMemoryAgent*) [nsStr.cpp:246]
        nsStr::Append(nsStr&,const nsStr&,unsigned int,int,nsIMemoryAgent*)
[nsStr.cpp:289]
        nsStr::Assign(nsStr&,const nsStr&,unsigned int,int,nsIMemoryAgent*)
[nsStr.cpp:271]
        nsString::Assign(const nsStr&,int) [nsString2.cpp:802]
        nsString::operator =(const nsString&) [nsString2.h:423]
        CSSStyleRuleImpl::SetSourceSelectorText(const nsString&)
[nsCSSStyleRule.cpp:916]
        CSSParserImpl::ParseRuleSet(int&) [nsCSSParser.cpp:1156]
        CSSParserImpl::Parse(nsIUnicharInputStream*,nsIURI*,nsICSSStyleSheet*&)
[nsCSSParser.cpp:446]

CSSLoaderImpl::ParseSheet(nsIUnicharInputStream*,SheetLoadData*,int&,nsICSSStyle
Sheet*&) [nsCSSLoader.cpp:739]

CSSLoaderImpl::DidLoadStyle(nsIUnicharStreamLoader*,nsString&,SheetLoadData*,uns
igned int) [nsCSSLoader.cpp:784]
        DoneLoadingStyle(nsIUnicharStreamLoader*,nsString&,void*,unsigned int)
[nsCSSLoader.cpp:585]
        nsUnicharStreamLoader::OnStopRequest(nsIChannel*,nsISupports*,unsigned
int,const unsigned short*) [nsNetStreamLoader.cpp:161]
        nsOnStopRequestEvent::HandleEvent() [nsAsyncStreamListener.cpp:283]
        nsStreamListenerEvent::HandlePLEvent(PLEvent*)
[nsAsyncStreamListener.cpp:149]
        PL_HandleEvent [plevent.c:509]
        PL_ProcessPendingEvents [plevent.c:470]
        nsEventQueueImpl::ProcessPendingEvents() [nsEventQueue.cpp:118]
        event_processor_callback(void*,int,GdkInputCondition)
[nsAppShell.cpp:78]
        gdk_io_invoke  [gdkevents.c:868]
        g_io_unix_dispatch [giounix.c:131]
        g_main_dispatch [gmain.c:647]
        g_main_iterate [gmain.c:854]
        g_main_run     [gmain.c:912]
  * Block of 258 bytes (2 times); last block at 0x1ad2bb0
  * Block of 130 bytes (5 times); last block at 0x1ae93d8
  * Block of 66 bytes (5 times); last block at 0x1ad2428
  * Block of 34 bytes (26 times); last block at 0x1ac2a10

Comment 1

19 years ago
Pushing off non-beta 1 issues
(Reporter)

Updated

19 years ago
Depends on: 13283

Updated

19 years ago
Summary: MLK: → MLK: 2380 bytes leaked

Updated

19 years ago
Blocks: 14516

Comment 2

18 years ago
Reassigning peterl's bugs to myself.

Comment 3

18 years ago
Accepting peterl's bugs that have a Target Milestone

Comment 4

18 years ago
Pushing my M15 bugs to M16

Updated

18 years ago
Keywords: mlk

Comment 5

18 years ago
Block-moved memory leaks to attinasi
Assignee: pierre → attinasi
Status: ASSIGNED → NEW
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 6

18 years ago
I cannot find these leaks in the leak-log. Since this is so old I am assuming it 
is fixed now.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Updated

18 years ago
Keywords: verifyme

Comment 7

18 years ago
Mark or Bruce:

I am unable to verify leaks. Could one of you please marked this as verified? 
Thanks
Whiteboard: 4/24: verification by reporter/assigned engineer requested.

Comment 8

18 years ago
Marc, Bruce, any luck verifying? -ckritzer
(Assignee)

Comment 9

18 years ago
OK, I'll stamp it verified. Sorry to have spaced on it for so long.
Status: RESOLVED → VERIFIED

Updated

9 years ago
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.