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

2 files changed, 7 insertions, 9 deletions

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<JaasAuthenticationPolicy> polices = new ArrayList<JaasAuthenticationPolicy>();
         // 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)) {