nsCategoryManager is not threadsafe

RESOLVED FIXED in mozilla1.5alpha

Status

()

Core
XPCOM
P3
normal
RESOLVED FIXED
18 years ago
10 years ago

People

(Reporter: Jon Smirl, Assigned: dougt)

Tracking

({helpwanted})

Trunk
mozilla1.5alpha
Other
Other
helpwanted
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

18 years ago
XPConnect standalone is a multithreaded environment. It should be legal to call 
CreateInstance from any thread. CreateInstance uses  nsCategoryManager 
and triggers the non-thread safe assert when called from the non-primary 
thread. NS_IMPL_ISUPPORTS1(nsCategoryManager, nsICategoryManager) Even if the 
decision is to serialize CreateInstance it still needs to be callable from any 
thread.

This component needs to be checked for thread safety and switched to the 
THREADSAFE ISUPPORTS macro after any problems are fixed.

Comment 1

17 years ago
setting Jon Smirl's bugs to New
Status: UNCONFIRMED → NEW
Ever confirmed: true

Comment 2

17 years ago
Transferring to default XPCOM owner.
Assignee: rayw → warren
Summary: nsCategoryManager is not threadsafe → nsCategoryManager is not threadsafe
(Assignee)

Updated

16 years ago
Blocks: 101976
(Assignee)

Comment 3

16 years ago
to default
Assignee: warren → dougt
QA Contact: rayw → dougt
Target Milestone: --- → mozilla1.1
(Assignee)

Comment 4

16 years ago
moving out based on my workload.  Yell, if you have a problem.
Keywords: helpwanted
Target Milestone: mozilla1.1alpha → Future
(Assignee)

Updated

15 years ago
Target Milestone: Future → mozilla1.4alpha
(Assignee)

Comment 5

15 years ago
this is not immediately needed by any client that I am aware of.

alec, if you get to the category manager hash table clean up before me, we
should address this bug at the same time.
Target Milestone: mozilla1.4alpha → Future

Comment 6

15 years ago
Patch is in bug 193031

Comment 7

15 years ago
Fixed with my patch on bug 193031.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED

Updated

15 years ago
Target Milestone: Future → mozilla1.5alpha

Updated

10 years ago
Component: XPCOM Registry → XPCOM
QA Contact: doug.turner → xpcom
You need to log in before you can comment on or make changes to this bug.