Author: nandana
Date: Sun May 18 10:32:06 2008
New Revision: 657596

URL: http://svn.apache.org/viewvc?rev=657596&view=rev
Log:
RAMPART-165 doing the proper null check for axis service

Modified:
    
webservices/rampart/branches/java/1_4/modules/rampart-core/src/main/java/org/apache/rampart/RampartMessageData.java

Modified: 
webservices/rampart/branches/java/1_4/modules/rampart-core/src/main/java/org/apache/rampart/RampartMessageData.java
URL: 
http://svn.apache.org/viewvc/webservices/rampart/branches/java/1_4/modules/rampart-core/src/main/java/org/apache/rampart/RampartMessageData.java?rev=657596&r1=657595&r2=657596&view=diff
==============================================================================
--- 
webservices/rampart/branches/java/1_4/modules/rampart-core/src/main/java/org/apache/rampart/RampartMessageData.java
 (original)
+++ 
webservices/rampart/branches/java/1_4/modules/rampart-core/src/main/java/org/apache/rampart/RampartMessageData.java
 Sun May 18 10:32:06 2008
@@ -21,6 +21,7 @@
 import org.apache.axis2.AxisFault;
 import org.apache.axis2.context.MessageContext;
 import org.apache.axis2.context.OperationContext;
+import org.apache.axis2.description.AxisService;
 import org.apache.axis2.description.Parameter;
 import org.apache.axis2.wsdl.WSDLConstants;
 import org.apache.neethi.Policy;
@@ -169,13 +170,17 @@
                 this.secConvVersion = 
TrustUtil.getWSTVersion((String)msgCtx.getProperty(KEY_WSSC_VERSION));
             }
             
-            Parameter clientSideParam = 
msgCtx.getAxisService().getParameter(PARAM_CLIENT_SIDE);
-            if(clientSideParam != null) {
+            // First obtain the axis service as we have to do a null check, 
there can be situations 
+            // where Axis Service is null
+            AxisService axisService = msgCtx.getAxisService();            
+                    
+            if(axisService != null && 
axisService.getParameter(PARAM_CLIENT_SIDE) != null) {
                 this.isInitiator = true;
             } else {
                 this.isInitiator = !msgCtx.isServerSide();
-                if(this.isInitiator) {
-                    clientSideParam = new Parameter();
+                //TODO if Axis Service is null at this point, do we have to 
create a dummy one ??    
+                if(this.isInitiator && axisService != null ) {
+                    Parameter clientSideParam = new Parameter();
                     clientSideParam.setName(PARAM_CLIENT_SIDE);
                     clientSideParam.setLocked(true);
                     msgCtx.getAxisService().addParameter(clientSideParam);


Reply via email to