Open
Bug 486311
Opened 17 years ago
Updated 2 years ago
nsScriptSecurityManager getService call chain shouldn't rely on intl/stringbundle
Categories
(Core :: Security: CAPS, defect, P5)
Core
Security: CAPS
Tracking
()
NEW
People
(Reporter: timeless, Unassigned)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
|
14.57 KB,
patch
|
Details | Diff | Splinter Review |
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 1•13 years ago
|
||
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)
Updated•12 years ago
|
Severity: normal → major
Comment 2•4 years ago
|
||
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)
Updated•4 years ago
|
Severity: major → S4
Flags: needinfo?(ckerschb)
Priority: -- → P5
Updated•2 years ago
|
Blocks: nss-external
You need to log in
before you can comment on or make changes to this bug.
Description
•