43561 - objects held past XPCOM shutdown

Status: RESOLVED WORKSFORME

(Core :: XPCOM, defect, P3)

Description

[Chris Waterson]

meta-bug for tracking leaks of objects held past XPCOM shutdown

Comment 1

[Chris Waterson]

reassigning to self

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

A list of suspicious things can be found at:
http://lxr.mozilla.org/seamonkey/search?string=static+nsCOMPtr
although some of these are functions.

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Attachment: nsStr released after XPCOM shutdown (23 of 25 listed as leaked)

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Some of the statics holding nsStr:

xpcom/io/nsLocalFileCommon.cpp:
  nsFSStringConversion gConverter

layout/base/src/nsSelection.cpp:
  static nsStyleColor sTableStyleColor    [2 strings]

layout/mathml/base/src/nsMathMLChar.cpp:
  nsGlyphTable gGlyphTableUNDEFINED
  nsGlyphTable gGlyphTableSymbol
  nsGlyphTable gGlyphTableMTExtra
  nsGlyphTable gGlyphTableCMEX
  nsGlyphTable gGlyphTableCMSY
  nsGlyphTable gGlyphTableMath4

rdf/content/src/nsXULTemplateBuilder.cpp:
  nsString nsXULTemplateBuilder::kTrueStr;
  nsString nsXULTemplateBuilder::kFalseStr;

rdf/content/src/nsXULSortService.cpp:
  nsString XULSortServiceImpl::trueStr;
  nsString XULSortServiceImpl::naturalStr;
  nsString XULSortServiceImpl::ascendingStr;
  nsString XULSortServiceImpl::descendingStr;

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Marking [tind-mlk] since this is the only bug tracking the |static nsString|s.

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

There are also some static nsString and nsAutoString in wallet.cpp.  morse added
one more this morning, so leaks went up.

Comment 7

[rbs]

statics in layout/mathml/base/src/nsMathMLChar.cpp are out of the radar now.

Comment 8

[Peter Lubczynski]

Java causes this too.

Comment 9

[Jesse Ruderman]

The dependencies are fixed. Should this be marked as WFM?

Comment 10

[Asa Dotzler [:asa]]

dependencies all fixed. resolving.