Open Bug 486311 Opened 15 years ago Updated 4 months ago

nsScriptSecurityManager getService call chain shouldn't rely on intl/stringbundle

Categories

(Core :: Security: CAPS, defect, P5)

defect

Tracking

()

People

(Reporter: timeless, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

i'd like to replace stringbundle w/ a js impl, but i can't, because when i run xpcshell if the contract is registered to my js impl, i get this:

NS_DebugBreak_P("Recursive GetService!"
nsComponentManagerImpl::GetServiceByContractID
CallGetService("@mozilla.org/scriptsecuritymanager;1"
nsGetServiceByContractID::operator()
mozJSComponentLoader::ReallyInit
mozJSComponentLoader::LoadModule
nsFactoryEntry::GetFactory
nsComponentManagerImpl::CreateInstanceByContractID
nsComponentManagerImpl::GetServiceByContractID
CallGetService("@mozilla.org/intl/stringbundle;1"
nsGetServiceByContractIDWithError::operator()
nsScriptSecurityManager::Init
nsScriptSecurityManager::GetScriptSecurityManager
Construct_nsIScriptSecurityManager
nsGenericFactory::CreateInstance
nsComponentManagerImpl::CreateInstanceByContractID
nsComponentManagerImpl::GetServiceByContractID
xpcom_core!CallGetService("@mozilla.org/scriptsecuritymanager;1"
xpcom_core!nsGetServiceByContractID::operator()
nsXPConnect::GetBackstagePass
main
Attachment #370408 - Flags: review?(jst)
Comment on attachment 370408 [details] [diff] [review]
change caps not to be fatalistic

Clearing out old reviews. If this is still relevant, please re-request review for this patch.
Attachment #370408 - Flags: review?(jst)
Severity: normal → major
QA Whiteboard: qa-not-actionable

The bug assignee didn't login in Bugzilla in the last 7 months.
:ckerschb, could you have a look please?
For more information, please visit auto_nag documentation.

Assignee: timeless → nobody
Flags: needinfo?(ckerschb)
Severity: major → S4
Flags: needinfo?(ckerschb)
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: