Author: rfeng
Date: Mon Mar 10 12:31:49 2008
New Revision: 635666

URL: http://svn.apache.org/viewvc?rev=635666&view=rev
Log:
Check the null for implementation policy provider to avoid NPE

Modified:
    
incubator/tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/assembly/CompositeActivatorImpl.java

Modified: 
incubator/tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/assembly/CompositeActivatorImpl.java
URL: 
http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/assembly/CompositeActivatorImpl.java?rev=635666&r1=635665&r2=635666&view=diff
==============================================================================
--- 
incubator/tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/assembly/CompositeActivatorImpl.java
 (original)
+++ 
incubator/tuscany/java/sca/modules/core/src/main/java/org/apache/tuscany/sca/core/assembly/CompositeActivatorImpl.java
 Mon Mar 10 12:31:49 2008
@@ -397,8 +397,11 @@
             throw new IllegalStateException("Provider factory not found for 
class: " + implementation.getClass()
                 .getName());
         }
-        for(PolicyProviderFactory f: 
providerFactories.getPolicyProviderFactories()){
-            
component.addPolicyProvider(f.createImplementationPolicyProvider(component, 
implementation));
+        for (PolicyProviderFactory f : 
providerFactories.getPolicyProviderFactories()) {
+            PolicyProvider policyProvider = 
f.createImplementationPolicyProvider(component, implementation);
+            if (policyProvider != null) {
+                component.addPolicyProvider(policyProvider);
+            }
         }
         
     }



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to