bug 204659 - Security manager check needs adjusting. r=adamlock, sr=alecf, a=sspitzer

js/src/xpconnect/src/XPCIDispatchExtension.cpp

@@ -54,7 +54,7 @@ CommonConstructor(JSContext *cx, int name, JSObject *obj, uintN argc,
     nsIXPCSecurityManager* sm = ccx.GetXPCContext()
         ->GetAppropriateSecurityManager(nsIXPCSecurityManager::HOOK_CALL_METHOD);
     XPCWrappedNative * wrapper = ccx.GetWrapper();
-    if(!sm || NS_FAILED(sm->CanAccess(nsIXPCSecurityManager::ACCESS_CALL_METHOD,
+    if(sm && NS_FAILED(sm->CanAccess(nsIXPCSecurityManager::ACCESS_CALL_METHOD,
                                       &ccx, ccx, ccx.GetFlattenedJSObject(),
                                       wrapper->GetIdentityObject(),
                                       wrapper->GetClassInfo(),

js/src/xpconnect/src/nsDispatchSupport.cpp

@@ -203,7 +203,7 @@ NS_IMETHODIMP nsDispatchSupport::IsClassSafeToHost(JSContext * cx,
     nsIXPCSecurityManager* sm =
             ccx.GetXPCContext()->GetAppropriateSecurityManager(
                         nsIXPCSecurityManager::HOOK_CREATE_INSTANCE);
-    *aResult = sm && 
+    *aResult = !sm ||
         NS_SUCCEEDED(sm->CanCreateInstance(ccx, cid));
 
     if(!*aResult)