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)
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•12 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•11 years ago
|
Severity: normal → major
Comment 2•2 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•2 years ago
|
Severity: major → S4
Flags: needinfo?(ckerschb)
Priority: -- → P5
Updated•4 months ago
|
Blocks: nss-external
You need to log in
before you can comment on or make changes to this bug.
Description
•