56738 - bcJavaComponentLoader not thread-safe

Status: RESOLVED FIXED

(Core Graveyard :: Java to XPCOM Bridge, defect, P3)

Description

[Jon Smirl]

Ignore the other failures about bcjavastubs.dll I caused them.

--bcJavaComponentLoader::bcJavaComponentLoader
--bcJavaComponentLoader::Init
--bcJavaComponentLoader::GetFactory
--JavaModule::JavaModule d:\testdll\components\bcFOP.jar.info
--JavaModule::GetClassObject
--bcJavaComponentFactory::CreateInstance
**************************************************
nsNativeComponentLoader: GetFactory(d:\testdll\components\bcjavastubs.dll) Load
FAILED with error: <unknown; can't get error from NSPR>
**************************************************
--bcJavaComponentFactory::CreateInstance javaStubsAndProxies failed
--bcJavaComponentFactory::CreateInstance
**************************************************
nsNativeComponentLoader: GetFactory(d:\testdll\components\bcjavastubs.dll) Load
FAILED with error: <unknown; can't get error from NSPR>
**************************************************
--bcJavaComponentFactory::CreateInstance javaStubsAndProxies failed
###!!! ASSERTION: bcJavaComponentLoader not thread-safe: 'owningThread == NS_Cur
rentThread()', file d:\cvs\mozilla\xpcom\base\nsDebug.cpp, line 490

Comment 1

[Asa Dotzler [:asa]]

setting Jon Smirl's bugs to New

Comment 2

[Igor Kushnirskiy]

bcJavaComponentLoader is threadsafe.
the only thing to change is to 
change NS_IMPL_ISUPPORTS
to 
NS_IMPL_THREADSAFE_ISUPPORTS
for bcJavaComponentLoader.cpp

I am closing this bug as fixed.