From 79b77adb8fef08386ee640cd7fe41e845f34093d Mon Sep 17 00:00:00 2001 From: lresende Date: Fri, 22 May 2009 16:19:10 +0000 Subject: Merging changes from 1.5 release branch to avoid NPE when running some samples git-svn-id: http://svn.us.apache.org/repos/asf/tuscany@777599 13f79535-47bb-0310-9956-ffa450edef68 --- .../JaasAuthenticationImplementationPolicyProvider.java | 2 +- .../jaas/JaasAuthenticationServicePolicyProvider.java | 14 ++++++-------- 2 files changed, 7 insertions(+), 9 deletions(-) (limited to 'branches/sca-java-1.x/modules/policy-security/src/main/java/org/apache/tuscany') diff --git a/branches/sca-java-1.x/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/JaasAuthenticationImplementationPolicyProvider.java b/branches/sca-java-1.x/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/JaasAuthenticationImplementationPolicyProvider.java index 185982a749..981c29df6a 100644 --- a/branches/sca-java-1.x/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/JaasAuthenticationImplementationPolicyProvider.java +++ b/branches/sca-java-1.x/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/JaasAuthenticationImplementationPolicyProvider.java @@ -52,7 +52,7 @@ public class JaasAuthenticationImplementationPolicyProvider implements PolicyPro if (implementation instanceof OperationsConfigurator) { OperationsConfigurator operationsConfigurator = (OperationsConfigurator)implementation; for (ConfiguredOperation cop : operationsConfigurator.getConfiguredOperations()) { - if (cop.getName().equals(op.getName())) { + if (cop != null && cop.getName() != null && cop.getName().equals(op.getName())) { for (PolicySet ps : cop.getPolicySets()) { for (Object p : ps.getPolicies()) { if (JaasAuthenticationPolicy.class.isInstance(p)) { diff --git a/branches/sca-java-1.x/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/JaasAuthenticationServicePolicyProvider.java b/branches/sca-java-1.x/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/JaasAuthenticationServicePolicyProvider.java index 56927be6bb..220ebd937f 100644 --- a/branches/sca-java-1.x/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/JaasAuthenticationServicePolicyProvider.java +++ b/branches/sca-java-1.x/modules/policy-security/src/main/java/org/apache/tuscany/sca/policy/security/jaas/JaasAuthenticationServicePolicyProvider.java @@ -57,13 +57,11 @@ public class JaasAuthenticationServicePolicyProvider implements PolicyProvider { List polices = new ArrayList(); // FIXME: How do we get a list of effective policySets for a given operation? for(Operation operation : operations) { - if (operation!= null && operation.getName() != null) { - if (operation.getName().equals(op.getName())) { - for (PolicySet ps : operation.getPolicySets()) { - for (Object p : ps.getPolicies()) { - if (JaasAuthenticationPolicy.class.isInstance(p)) { - polices.add((JaasAuthenticationPolicy)p); - } + if (operation!= null && operation.getName() != null && operation.getName().equals(op.getName())) { + for (PolicySet ps : operation.getPolicySets()) { + for (Object p : ps.getPolicies()) { + if (JaasAuthenticationPolicy.class.isInstance(p)) { + polices.add((JaasAuthenticationPolicy)p); } } } @@ -73,7 +71,7 @@ public class JaasAuthenticationServicePolicyProvider implements PolicyProvider { if (service instanceof OperationsConfigurator) { OperationsConfigurator operationsConfigurator = (OperationsConfigurator)service; for (ConfiguredOperation cop : operationsConfigurator.getConfiguredOperations()) { - if (cop.getName().equals(op.getName())) { + if (cop != null && cop.getName() != null && cop.getName().equals(op.getName())) { for (PolicySet ps : cop.getApplicablePolicySets()) { for (Object p : ps.getPolicies()) { if (JaasAuthenticationPolicy.class.isInstance(p)) { -- cgit v1.2.3