Global destructors shouldn't fire after xpcom is unloaded

RESOLVED INVALID

Status

()

P3
normal
RESOLVED INVALID
19 years ago
16 years ago

People

(Reporter: samir_bugzilla, Assigned: dougt)

Tracking

({topembed+})

Trunk
mozilla1.1alpha
x86
Linux
topembed+
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
This bug is best explained by example:

The linux installer loads xpistub which in turn is linked into xpcom and 
requisite libs.  After working with xpistub (libxpistub.so on Unix), the 
installer unloads it which in turn causes xpcom and the linked libs to unload 
too.  When the standalone installer app quits, the global destructors fire but 
the xpcom dll is no longer around so we core dump.  The bug to fix here is to 
make sure we don't have global classes upon which destructors fire on exit.

Per dp:
XPCOM wasn't intended to be used by loading and unloading it manually. Hence we 
were lax about creating classes globally. This would be something we should fix 
for XPCOM 1.0.

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → M18

Updated

18 years ago
Blocks: 38671

Updated

18 years ago
Target Milestone: M18 → M20

Comment 1

18 years ago
Edward: Welcome to xpcom!
Status: ASSIGNED → NEW
QA Contact: leger → rayw
Target Milestone: M20 → mozilla1.0

Comment 2

18 years ago
Once again... attempting to reassign from Ray to Edward.
Assignee: rayw → kandrot

Updated

18 years ago
Status: NEW → ASSIGNED
QA Contact: rayw → scc
Target Milestone: mozilla1.0 → Future
(Assignee)

Comment 3

17 years ago
reassign all kandrot xpcom bug.
Assignee: kandrot → dougt
Status: ASSIGNED → NEW
Target Milestone: Future → ---
(Assignee)

Updated

17 years ago
Blocks: 98283
(Assignee)

Updated

17 years ago
Target Milestone: --- → mozilla1.1
(Assignee)

Comment 4

16 years ago
shouldn't fire global destructors?  there is nothing we can do.  when C++
objects go out of scope, they fire.  The real problem is that we have static
global data.

Marking invalid.  
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → INVALID

Updated

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