You are viewing a plain text version of this content. The canonical link for it is here.
Posted to axis-cvs@ws.apache.org by da...@apache.org on 2006/01/23 12:22:16 UTC

svn commit: r371527 - in /webservices/axis2/trunk/c: ./ include/ modules/core/description/ modules/wsdl/ test/unit/ test/unit/wsdl/

Author: damitha
Date: Mon Jan 23 03:21:15 2006
New Revision: 371527

URL: http://svn.apache.org/viewcvs?rev=371527&view=rev
Log:
Cleaning the code

Modified:
    webservices/axis2/trunk/c/configure.ac
    webservices/axis2/trunk/c/include/axis2_wsdl_component.h
    webservices/axis2/trunk/c/modules/core/description/svc.c
    webservices/axis2/trunk/c/modules/wsdl/wsdl_component.c
    webservices/axis2/trunk/c/test/unit/Makefile.am
    webservices/axis2/trunk/c/test/unit/main.c
    webservices/axis2/trunk/c/test/unit/wsdl/Makefile.am
    webservices/axis2/trunk/c/test/unit/wsdl/main.c
    webservices/axis2/trunk/c/test/unit/wsdl/wsdl_test.h

Modified: webservices/axis2/trunk/c/configure.ac
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/configure.ac?rev=371527&r1=371526&r2=371527&view=diff
==============================================================================
--- webservices/axis2/trunk/c/configure.ac (original)
+++ webservices/axis2/trunk/c/configure.ac Mon Jan 23 03:21:15 2006
@@ -132,6 +132,7 @@
     test/unit/core/phaseresolver/Makefile \
     test/unit/core/deployment/Makefile \
     test/unit/util/Makefile \
+    test/unit/wsdl/Makefile \
     test/unit/xml/Makefile \
     test/unit/xml/om/Makefile\
     test/core/transport/Makefile\

Modified: webservices/axis2/trunk/c/include/axis2_wsdl_component.h
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/include/axis2_wsdl_component.h?rev=371527&r1=371526&r2=371527&view=diff
==============================================================================
--- webservices/axis2/trunk/c/include/axis2_wsdl_component.h (original)
+++ webservices/axis2/trunk/c/include/axis2_wsdl_component.h Mon Jan 23 03:21:15 2006
@@ -76,7 +76,7 @@
     axis2_status_t (AXIS2_CALL *
     set_component_property) (axis2_wsdl_component_t *wsdl_component,
                                 axis2_env_t **env, 
-                                void *key, 
+                                axis2_char_t *key, 
                                 void *value);
     
     /**
@@ -87,7 +87,7 @@
     void *(AXIS2_CALL *
     get_component_property) (axis2_wsdl_component_t *wsdl_component, 
                                 axis2_env_t **env,
-                                void *key);
+                                axis2_char_t *key);
  
 
 

Modified: webservices/axis2/trunk/c/modules/core/description/svc.c
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/modules/core/description/svc.c?rev=371527&r1=371526&r2=371527&view=diff
==============================================================================
--- webservices/axis2/trunk/c/modules/core/description/svc.c (original)
+++ webservices/axis2/trunk/c/modules/core/description/svc.c Mon Jan 23 03:21:15 2006
@@ -390,10 +390,8 @@
     
 	svc_impl->svc.ops->free = axis2_svc_free;
 	svc_impl->svc.ops->add_op = axis2_svc_add_op;
-	svc_impl->svc.ops->get_op_with_qname = 
-		axis2_svc_get_op_with_qname;
-	svc_impl->svc.ops->get_op_with_name = 
-		axis2_svc_get_op_with_name;
+	svc_impl->svc.ops->get_op_with_qname = axis2_svc_get_op_with_qname;
+	svc_impl->svc.ops->get_op_with_name = axis2_svc_get_op_with_name;
 	svc_impl->svc.ops->get_ops = axis2_svc_get_ops;
 	svc_impl->svc.ops->set_parent = axis2_svc_set_parent;
 	svc_impl->svc.ops->get_parent = axis2_svc_get_parent;
@@ -405,71 +403,40 @@
     svc_impl->svc.ops->is_param_locked = axis2_svc_is_param_locked;
     svc_impl->svc.ops->set_svc_interface = axis2_svc_set_svc_interface;
     svc_impl->svc.ops->get_svc_interface = axis2_svc_get_svc_interface;
-    
     svc_impl->svc.ops->engage_module = axis2_svc_engage_module;
-    
     svc_impl->svc.ops->add_module_ops = axis2_svc_add_module_ops;
-    
-    svc_impl->svc.ops->add_to_engaged_module_list = axis2_svc_add_to_engaged_module_list;
-    
+    svc_impl->svc.ops->add_to_engaged_module_list = 
+            axis2_svc_add_to_engaged_module_list;
     svc_impl->svc.ops->get_engaged_modules = axis2_svc_get_engaged_modules;
-    
     svc_impl->svc.ops->get_wsdl_op = axis2_svc_get_wsdl_op;
-    
     svc_impl->svc.ops->set_context_path = axis2_svc_set_context_path;
-    
     svc_impl->svc.ops->get_context_path = axis2_svc_get_context_path;
-    
     svc_impl->svc.ops->set_style = axis2_svc_set_style;
-    
     svc_impl->svc.ops->get_style = axis2_svc_get_style;
-    
     svc_impl->svc.ops->get_inflow = axis2_svc_get_inflow;
-    
     svc_impl->svc.ops->set_inflow = axis2_svc_set_inflow;
-    
     svc_impl->svc.ops->get_outflow = axis2_svc_get_outflow;
-    
     svc_impl->svc.ops->set_outflow = axis2_svc_set_outflow;
-    
     svc_impl->svc.ops->get_fault_inflow = axis2_svc_get_fault_inflow;
-    
     svc_impl->svc.ops->set_fault_inflow = axis2_svc_set_fault_inflow;
-    
     svc_impl->svc.ops->get_fault_outflow = axis2_svc_get_fault_outflow;
-    
     svc_impl->svc.ops->set_fault_outflow = axis2_svc_set_fault_outflow;
-    
     svc_impl->svc.ops->get_op_by_soap_action = axis2_svc_get_op_by_soap_action;
-    
-    svc_impl->svc.ops->get_op_by_soap_action_and_endpoint = axis2_svc_get_op_by_soap_action_and_endpoint;
-    
+    svc_impl->svc.ops->get_op_by_soap_action_and_endpoint = 
+            axis2_svc_get_op_by_soap_action_and_endpoint;
     svc_impl->svc.ops->get_axis2_svc_name = axis2_svc_get_axis2_svc_name;
-    
     svc_impl->svc.ops->set_axis2_svc_name = axis2_svc_set_axis2_svc_name;
-    
     svc_impl->svc.ops->set_last_update = axis2_svc_set_last_update;
-    
     svc_impl->svc.ops->get_last_update = axis2_svc_get_last_update;
-    
     svc_impl->svc.ops->get_filename = axis2_svc_get_filename;
-    
     svc_impl->svc.ops->set_filename = axis2_svc_set_filename;
-    
     svc_impl->svc.ops->get_endpoints = axis2_svc_get_endpoints;
-    
     svc_impl->svc.ops->set_endpoints = axis2_svc_set_endpoints;
-    
     svc_impl->svc.ops->set_endpoint = axis2_svc_set_endpoint;
-    
     svc_impl->svc.ops->get_endpoint = axis2_svc_get_endpoint;
-    
     svc_impl->svc.ops->get_namespace = axis2_svc_get_namespace;
-    
     svc_impl->svc.ops->add_mapping = axis2_svc_add_mapping;
-    
     svc_impl->svc.ops->add_module_ref = axis2_svc_add_module_ref;
-    
     svc_impl->svc.ops->get_modules = axis2_svc_get_modules;
     
 	return &(svc_impl->svc);	
@@ -535,20 +502,31 @@
     axis2_svc_impl_t *svc_impl = NULL;
     
     AXIS2_FUNC_PARAM_CHECK(svc, env, AXIS2_FAILURE);
-    
     svc_impl = AXIS2_INTF_TO_IMPL(svc);
     
 	if(NULL != svc->ops)
+    {
 		AXIS2_FREE((*env)->allocator, svc->ops);
+        svc->ops = NULL;
+    }
     
     if(NULL != svc->param_container)
+    {
 	    AXIS2_PARAM_CONTAINER_FREE(svc->param_container, env);
+        svc->param_container = NULL;
+    }
     
     if(NULL != svc->flow_container)
+    {
 	    AXIS2_FLOW_CONTAINER_FREE(svc->flow_container, env);
+        svc->flow_container = NULL;
+    }
     
     if(NULL != svc->wsdl_svc)
-	    AXIS2_PARAM_CONTAINER_FREE(svc->wsdl_svc, env);
+    {
+	    AXIS2_WSDL_SVC_FREE(svc->wsdl_svc, env);
+        svc->wsdl_svc = NULL;
+    }
     
     svc_impl->parent = NULL;
     
@@ -559,14 +537,15 @@
         for (hi = axis2_hash_first (svc_impl->wasaction_opeartionmap, env); hi;
                  hi = axis2_hash_next ( env, hi))
         {
-            struct axis2_op *opt = NULL;
+            axis2_op_t *opt = NULL;
             axis2_hash_this (hi, NULL, NULL, &val);
-            opt = (struct axis2_op *) val;
+            opt = (axis2_op_t *) val;
             if (opt)
-               AXIS2_OP_FREE (opt, env);
-            
+            {
+                AXIS2_OP_FREE (opt, env);
+                opt = NULL;
+            }
             val = NULL;
-            opt = NULL;
                
         }
 		axis2_hash_free(svc_impl->wasaction_opeartionmap, env);
@@ -575,29 +554,34 @@
     
     if(NULL != svc_impl->module_list)
     {
-        void *val = NULL;
         int i = 0;
-        for (i = 0; i < AXIS2_ARRAY_LIST_SIZE(svc_impl->module_list, env); i++)
+        int size = 0;
+        
+        size = AXIS2_ARRAY_LIST_SIZE(svc_impl->module_list, env);
+        for (i = 0; i < size; i++)
         {
             axis2_qname_t *qname = NULL;
             qname = AXIS2_ARRAY_LIST_GET(svc_impl->module_list, env, i);
-            
-            qname = (axis2_qname_t *) val;
             if (qname)
-               AXIS2_QNAME_FREE (qname, env);
-            
-            val = NULL;
-            qname = NULL;
-               
+            {
+                AXIS2_QNAME_FREE (qname, env);
+                qname = NULL;
+            }  
         }
 	    AXIS2_ARRAY_LIST_FREE(svc_impl->module_list, env);
         svc_impl->module_list = NULL;
     }
 
     if(NULL != svc_impl->axis_svc_name)
-        AXIS2_FREE((*env)->allocator, svc_impl->axis_svc_name);        
-    
-	AXIS2_FREE((*env)->allocator, svc_impl);
+    {
+        AXIS2_FREE((*env)->allocator, svc_impl->axis_svc_name);
+        svc_impl->axis_svc_name = NULL;        
+    }        
+    if(svc_impl)
+    {
+	    AXIS2_FREE((*env)->allocator, svc_impl);
+        svc_impl = NULL;
+    }
     
 	return AXIS2_SUCCESS;
 }
@@ -614,10 +598,13 @@
     AXIS2_PARAM_CHECK((*env)->error, axis2_opt, AXIS2_FALSE);
     
     status = AXIS2_OP_SET_PARENT(axis2_opt, env, svc);
-    if(AXIS2_TRUE ==status)
+    if(AXIS2_SUCCESS == status)
     {
-        status = AXIS2_WSDL_INTERFACE_SET_OP(axis2_svc_get_svc_interface(svc, env),
-            env, axis2_opt, AXIS2_OP_T);
+        axis2_wsdl_interface_t *wsdl_interface = NULL;
+        
+        wsdl_interface = axis2_svc_get_svc_interface(svc, env);
+        status = AXIS2_WSDL_INTERFACE_SET_OP(wsdl_interface, env, axis2_opt, 
+            AXIS2_OP_T);
     }
     return status;
 }
@@ -627,50 +614,57 @@
                                         axis2_env_t **env,
 		                                axis2_qname_t *op_name)
 {
-    struct axis2_op *op_l = NULL;
+    axis2_svc_impl_t *svc_impl = NULL;
+    axis2_op_t *op_l = NULL;
     axis2_char_t *op_str = NULL;
     /*axis2_hash_t *all_ops = NULL; */
     
     AXIS2_FUNC_PARAM_CHECK(svc, env, NULL);
     AXIS2_PARAM_CHECK((*env)->error, op_name, NULL);
+    svc_impl = AXIS2_INTF_TO_IMPL(svc);
     
     op_str = AXIS2_QNAME_GET_LOCALPART(op_name, env);
-    /* commented until AXIS2_WSDL_INTERFACE_GET_ALL_OPS is implemented
+    /*TODO commented until AXIS2_WSDL_INTERFACE_GET_ALL_OPS is implemented
     all_ops = AXIS2_WSDL_INTERFACE_GET_ALL_OPS(
         axis2_svc_get_svc_interface(svc, env), env);
-    opeartion_l = (struct axis2_op) (axis2_hash_get (all_ops, 
+    op_l = (struct axis2_op) (axis2_hash_get (all_ops, 
         op_str, AXIS2_HASH_KEY_STRING));
     */
     if(NULL == op_l )
     {
         op_l = (struct axis2_op *) (axis2_hash_get (
-                AXIS2_INTF_TO_IMPL(svc)->wasaction_opeartionmap, op_str,
-                AXIS2_HASH_KEY_STRING));
+            svc_impl->wasaction_opeartionmap, op_str, AXIS2_HASH_KEY_STRING));
     }
     
     return op_l;	
 }	
 
-struct axis2_op * AXIS2_CALL
+axis2_op_t * AXIS2_CALL
 axis2_svc_get_op_with_name (axis2_svc_t *svc, 
                                     axis2_env_t **env,
 		                            const axis2_char_t* nc_name)
 {
+    axis2_wsdl_interface_t *wsdl_interface = NULL;
+    axis2_hash_t *ops = NULL;
+    
     AXIS2_FUNC_PARAM_CHECK(svc, env, NULL);
     AXIS2_PARAM_CHECK((*env)->error, nc_name, NULL);
     
-    return (axis2_op_t *) axis2_hash_get(AXIS2_WSDL_INTERFACE_GET_OPS(
-        axis2_svc_get_svc_interface(svc, env), env), nc_name, AXIS2_HASH_KEY_STRING);
+    wsdl_interface = axis2_svc_get_svc_interface(svc, env);
+    ops = AXIS2_WSDL_INTERFACE_GET_OPS(wsdl_interface, env);
+    return (axis2_op_t *) axis2_hash_get(ops, nc_name, AXIS2_HASH_KEY_STRING);
 }
 
 axis2_hash_t * AXIS2_CALL
 axis2_svc_get_ops (axis2_svc_t *svc, 
                             axis2_env_t **env)
 {
+    axis2_wsdl_interface_t *wsdl_interface = NULL;
+    
     AXIS2_FUNC_PARAM_CHECK(svc, env, NULL);
     
-    return AXIS2_WSDL_INTERFACE_GET_OPS(axis2_svc_get_svc_interface(svc,
-        env), env);
+    wsdl_interface = axis2_svc_get_svc_interface(svc, env);
+    return AXIS2_WSDL_INTERFACE_GET_OPS(wsdl_interface, env);
 }
 	
 axis2_status_t AXIS2_CALL
@@ -723,7 +717,7 @@
                         axis2_env_t **env,
 		                axis2_param_t *param)
 {
-    struct axis2_param_container *param_container_l = NULL;
+    axis2_param_container_t *param_container_l = NULL;
         
     AXIS2_FUNC_PARAM_CHECK(svc, env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK((*env)->error, param, AXIS2_FAILURE);
@@ -731,10 +725,11 @@
    
     if(axis2_svc_is_param_locked(svc, env, AXIS2_PARAM_GET_NAME(param, env)))
     {
-        AXIS2_ERROR_SET((*env)->error, AXIS2_ERROR_PARAMETER_LOCKED_CANNOT_OVERRIDE,
-            AXIS2_FAILURE);
+        AXIS2_ERROR_SET((*env)->error, 
+            AXIS2_ERROR_PARAMETER_LOCKED_CANNOT_OVERRIDE, AXIS2_FAILURE);
         return AXIS2_FAILURE;
-    } else
+    } 
+    else
     {
         param_container_l = (struct axis2_param_container *)
             AXIS2_WSDL_COMPONENT_GET_COMPONENT_PROPERTY(svc->wsdl_svc->
@@ -749,10 +744,10 @@
                         axis2_env_t **env,
 		                const axis2_char_t *name)
 {
-    struct axis2_param_container *param_container_l = NULL;
+    axis2_param_container_t *param_container_l = NULL;
     AXIS2_FUNC_PARAM_CHECK(svc, env, AXIS2_FAILURE);
     
-    param_container_l = (struct axis2_param_container *)
+    param_container_l = (axis2_param_container_t *)
         AXIS2_WSDL_COMPONENT_GET_COMPONENT_PROPERTY(svc->wsdl_svc->
             wsdl_component, env, AXIS2_PARAMETER_KEY);
     return AXIS2_PARAM_CONTAINER_GET_PARAM(param_container_l, env, name);
@@ -765,7 +760,7 @@
     struct axis2_param_container *param_container_l = NULL;
 	AXIS2_FUNC_PARAM_CHECK(svc, env, NULL);
 	
-    param_container_l = (struct axis2_param_container *)
+    param_container_l = (axis2_param_container_t *)
     AXIS2_WSDL_COMPONENT_GET_COMPONENT_PROPERTY(svc->wsdl_svc->
         wsdl_component, env, AXIS2_PARAMETER_KEY);
     
@@ -804,7 +799,8 @@
     if(AXIS2_TRUE == locked)
     {
         ret = AXIS2_TRUE;
-    } else 
+    } 
+    else 
     {
         param_l = axis2_svc_get_param(svc, env, param_name);
         if(NULL == param_l)
@@ -844,7 +840,7 @@
 {
     struct axis2_module_desc * modu = NULL;
     axis2_array_list_t *collection_module = NULL;
-    struct axis2_phase_resolver *phase_resolver = NULL;
+    axis2_phase_resolver_t *phase_resolver = NULL;
     int i = 0;
     axis2_status_t status = AXIS2_FAILURE;
     int size = 0;
@@ -859,20 +855,22 @@
             env, AXIS2_MODULEREF_KEY);
     
     size = AXIS2_ARRAY_LIST_SIZE(collection_module, env);
-    if(AXIS2_TRUE != size)
+    if(AXIS2_SUCCESS != AXIS2_ERROR_GET_STATUS_CODE((*env)->error))
     {
         return AXIS2_FAILURE;
     }
     for(i = 0; i < size; i++)
     {
-        modu = (struct axis2_module_desc *) AXIS2_ARRAY_LIST_GET(collection_module,
+        modu = (axis2_module_desc_t *) AXIS2_ARRAY_LIST_GET(collection_module,
             env, i);
         if(AXIS2_QNAME_EQUALS(AXIS2_MODULE_DESC_GET_NAME(modu, env), env,  
             AXIS2_MODULE_DESC_GET_NAME(moduleref, env)))
         {
-            /* module has alredy been engaged on the service. Operation terminated !!! */
-            AXIS2_ERROR_SET((*env)->error, AXIS2_ERROR_MODULE_ALREADY_ENGAGED_TO_SVC,
-                AXIS2_FAILURE);
+            /* module has alredy been engaged on the service. Operation 
+             *terminated !!! 
+             */
+            AXIS2_ERROR_SET((*env)->error, 
+                AXIS2_ERROR_MODULE_ALREADY_ENGAGED_TO_SVC, AXIS2_FAILURE);
             return AXIS2_FAILURE;            
         }
         
@@ -883,11 +881,17 @@
     {
         return AXIS2_FAILURE;
     }
-    status = AXIS2_PHASE_RESOLVER_ENGAGE_MODULE_TO_SVC(phase_resolver, env, svc, moduleref);
+    status = AXIS2_PHASE_RESOLVER_ENGAGE_MODULE_TO_SVC(phase_resolver, env, svc, 
+        moduleref);
     if(status)
     {
         status = AXIS2_ARRAY_LIST_ADD(collection_module, env, moduleref);
     }
+    if(phase_resolver)
+    {
+        AXIS2_PHASE_RESOLVER_FREE(phase_resolver, env);
+        phase_resolver = NULL;
+    }
     
     return status;
 }
@@ -895,15 +899,15 @@
 axis2_status_t AXIS2_CALL
 axis2_svc_add_module_ops(axis2_svc_t *svc,
                             axis2_env_t **env,
-                            struct axis2_module_desc *module_desc,
-                            struct axis2_conf *axis2_config) 
+                            axis2_module_desc_t *module_desc,
+                            axis2_conf_t *axis2_config) 
 {
     axis2_hash_t *map = NULL;
     axis2_hash_index_t *index = NULL;
-    struct axis2_phase_resolver *pr = NULL;
-    struct axis2_op *axis2_opt = NULL;
+    axis2_phase_resolver_t *pr = NULL;
+    axis2_op_t *axis2_opt = NULL;
     axis2_array_list_t *params = NULL;
-    struct axis2_param *param = NULL;
+    axis2_param_t *param = NULL;
     axis2_status_t status = AXIS2_FAILURE;
    
     AXIS2_FUNC_PARAM_CHECK(svc, env, AXIS2_FAILURE);
@@ -925,29 +929,35 @@
         int j = 0;
         void *v = NULL;
         axis2_hash_this (index, NULL, NULL, &v);
-        axis2_opt = (struct axis2_op *) v;
+        axis2_opt = (axis2_op_t *) v;
         params = AXIS2_OP_GET_PARAMS(axis2_opt, env);
         /* Adding wsa-maping into service */
         size = AXIS2_ARRAY_LIST_SIZE(params, env);
         
-        if(AXIS2_TRUE != AXIS2_ERROR_GET_STATUS_CODE((*env)->error))
+        if(AXIS2_SUCCESS != AXIS2_ERROR_GET_STATUS_CODE((*env)->error))
         {
             if(pr)
+            {
                 AXIS2_PHASE_RESOLVER_FREE(pr, env);
+                pr = NULL;
+            }
             return AXIS2_FAILURE;
         }
         for(j = 0; j < size; j++)
         {
-            param = (struct axis2_param *) AXIS2_ARRAY_LIST_GET(params, env, j);
+            param = (axis2_param_t *) AXIS2_ARRAY_LIST_GET(params, env, j);
             if(0 == AXIS2_STRCMP( AXIS2_PARAM_GET_NAME(param, env), 
                     AXIS2_WSA_ACTION))
             {
                 status = axis2_svc_add_mapping(svc, env,
                     (axis2_char_t *) AXIS2_PARAM_GET_VALUE(param, env), axis2_opt);
-                if(AXIS2_FAILURE == status)
+                if(AXIS2_SUCCESS != status)
                 {
                     if(pr)
-                        AXIS2_PHASE_RESOLVER_FREE(pr, env); 
+                    {
+                        AXIS2_PHASE_RESOLVER_FREE(pr, env);
+                        pr = NULL;                        
+                    }
                     return status;
                 }
             }
@@ -956,10 +966,13 @@
         
         status = AXIS2_PHASE_RESOLVER_BUILD_MODULE_OP(pr, env, axis2_opt);
         
-        if(AXIS2_FAILURE == status)
+        if(AXIS2_SUCCESS != status)
         {
             if(pr)
-                AXIS2_PHASE_RESOLVER_FREE(pr, env); 
+            {
+                AXIS2_PHASE_RESOLVER_FREE(pr, env);
+                pr = NULL;                
+            }
             return status;
         }
         
@@ -968,7 +981,10 @@
     }
     
     if(pr)
+    {
         AXIS2_PHASE_RESOLVER_FREE(pr, env); 
+        pr = NULL;
+    }
     return status;
 }
 
@@ -988,6 +1004,10 @@
     collection_module = (axis2_array_list_t *) 
         AXIS2_WSDL_COMPONENT_GET_COMPONENT_PROPERTY(svc->wsdl_svc->wsdl_component, 
             env, AXIS2_MODULEREF_KEY);
+    if(!collection_module)
+    {
+        return AXIS2_FAILURE;
+    }
     
     size = AXIS2_ARRAY_LIST_SIZE(collection_module, env);
     if(AXIS2_TRUE != size)
@@ -1022,9 +1042,9 @@
                             axis2_env_t **env,
                             axis2_qname_t *op_name)
 {
-    struct axis2_wsdl_interface *svc_interface = NULL;
-        
+    axis2_wsdl_interface_t *svc_interface = NULL;
     axis2_char_t * op_str = NULL;
+    
     AXIS2_FUNC_PARAM_CHECK(svc, env, NULL);
     AXIS2_PARAM_CHECK((*env)->error, op_name, NULL);
     
@@ -1051,11 +1071,11 @@
     context_path_l = AXIS2_STRDUP(context_path, env);
     if(context_path_l)
     {
-        return AXIS2_WSDL_COMPONENT_SET_COMPONENT_PROPERTY(svc->wsdl_svc->
-            wsdl_component, env, AXIS2_CONTEXTPATH_KEY, context_path);
+        AXIS2_ERROR_SET((*env)->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
+        return AXIS2_FAILURE;        
     }
-    else
-        return AXIS2_FAILURE;
+    return AXIS2_WSDL_COMPONENT_SET_COMPONENT_PROPERTY(svc->wsdl_svc->
+            wsdl_component, env, AXIS2_CONTEXTPATH_KEY, context_path);
 }
 
 axis2_char_t * AXIS2_CALL
@@ -1073,16 +1093,17 @@
                         axis2_char_t * style) 
 {
     axis2_char_t *style_l = NULL;
-    
     AXIS2_FUNC_PARAM_CHECK(svc, env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK((*env)->error, style, AXIS2_FAILURE);
     
     style_l = AXIS2_STRDUP(style, env);
-    if(style_l)
-        return AXIS2_WSDL_COMPONENT_SET_COMPONENT_PROPERTY(svc->wsdl_svc->
-            wsdl_component, env, AXIS2_STYLE_KEY, style);
-    else
+    if(!style_l)
+    {
+        AXIS2_ERROR_SET((*env)->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return AXIS2_FAILURE;
+    }
+    return AXIS2_WSDL_COMPONENT_SET_COMPONENT_PROPERTY(svc->wsdl_svc->
+        wsdl_component, env, AXIS2_STYLE_KEY, style_l);
 }
 
 axis2_char_t * AXIS2_CALL
@@ -1188,15 +1209,15 @@
     axis2_hash_t *endpoints = NULL;
     axis2_hash_index_t *index = NULL;
     void *value = NULL;
-    struct axis2_wsdl_endpoint * endpoint = NULL;
+    axis2_wsdl_endpoint_t * endpoint = NULL;
     
     AXIS2_FUNC_PARAM_CHECK(svc, env, NULL);
     AXIS2_PARAM_CHECK((*env)->error, soap_action, NULL);
+    svc_impl = AXIS2_INTF_TO_IMPL(svc);
+    
     if(0 == AXIS2_STRCMP(soap_action, ""))
         return NULL;
     
-    svc_impl = AXIS2_INTF_TO_IMPL(svc);
-    
     endpoints = axis2_svc_get_endpoints(svc, env);
     if(!endpoints)
     {
@@ -1205,10 +1226,13 @@
     index = axis2_hash_first (endpoints, env);
     if(NULL != index)
     {
+        axis2_qname_t *endpoint_qname = NULL;
+        
         axis2_hash_this(index, NULL, NULL, &value);
-        endpoint = (struct axis2_wsdl_endpoint *) value;
+        endpoint = (axis2_wsdl_endpoint_t *) value;
+        endpoint_qname = AXIS2_WSDL_ENDPOINT_GET_NAME(endpoint, env);
         return axis2_svc_get_op_by_soap_action_and_endpoint(svc, env, 
-            soap_action, AXIS2_WSDL_ENDPOINT_GET_NAME(endpoint, env));
+            soap_action, endpoint_qname);
     }
 
     return NULL;
@@ -1224,20 +1248,20 @@
     axis2_hash_t *binding_ops = NULL;
     int count = 0;
     axis2_hash_index_t *index_i = NULL;
-    const void *k = NULL;
     struct axis2_wsdl_binding_op *binding_op = NULL;
     struct axis2_wsdl_soap_op *element = NULL;
     struct axis2_op *op = NULL;
     axis2_linked_list_t *extensiblity_elements = NULL;
     axis2_qname_t *type_1 = NULL;
-    axis2_qname_t *type_2 = NULL;
+    axis2_wsdl_binding_t *wsdl_binding = NULL;
+    axis2_wsdl_endpoint_t *wsdl_endpoint = NULL;
         
     svc_impl = AXIS2_INTF_TO_IMPL(svc);
     
     type_1 = axis2_qname_create(env, "op", AXIS2_SOAP_11_OP, NULL);
-    binding_ops = AXIS2_WSDL_BINDING_GET_BINDING_OPS(
-        AXIS2_WSDL_ENDPOINT_GET_BINDING(axis2_svc_get_endpoint(svc, env, endpoint),
-            env), env);
+    wsdl_endpoint = axis2_svc_get_endpoint(svc, env, endpoint);
+    wsdl_binding = AXIS2_WSDL_ENDPOINT_GET_BINDING(wsdl_endpoint, env);
+    binding_ops = AXIS2_WSDL_BINDING_GET_BINDING_OPS(wsdl_binding, env);
     if(NULL == binding_ops)
     {
         return NULL;
@@ -1247,22 +1271,29 @@
     {
         int j = 0;
         int size = 0;
+        const void *k = NULL;
+        axis2_status_t status = AXIS2_FAILURE;
+        
         axis2_hash_this (index_i, &k, NULL, NULL);
         binding_op = (struct axis2_wsdl_binding_op *) k;
         extensiblity_elements = AXIS2_WSDL_COMPONENT_GET_EXTENSIBILITY_ELEMENTS(
             binding_op->extensible_component->wsdl_component, env);
         
         size = AXIS2_LINKED_LIST_SIZE(extensiblity_elements, env);
-        if(AXIS2_TRUE != size)
+        status = AXIS2_ERROR_GET_STATUS_CODE((*env)->error);
+        if(AXIS2_SUCCESS != status)
         {
             return NULL;
         }
         while(j < size)
         {
+            axis2_qname_t *type_2 = NULL;
+            
             element = (struct axis2_wsdl_soap_op *) AXIS2_LINKED_LIST_GET(
                 extensiblity_elements, env, j); 
                         
-            type_2 = AXIS2_WSDL_EXTENSIBLE_ELEMENT_GET_TYPE(element->extensible_element, env);
+            type_2 = AXIS2_WSDL_EXTENSIBLE_ELEMENT_GET_TYPE(element->
+                extensible_element, env);
             if(AXIS2_QNAME_EQUALS(type_2, env, type_1))
             {
                 if(0 == AXIS2_STRCMP(AXIS2_WSDL_SOAP_OP_GET_SOAP_ACTION(
@@ -1275,6 +1306,8 @@
                 }
                     
             }
+            AXIS2_QNAME_FREE(type_2, env);
+            type_2 = NULL;
             j++;
         }
         index_i = axis2_hash_next (env, index_i);
@@ -1283,7 +1316,8 @@
     {
         return op;
     }
-    
+    AXIS2_QNAME_FREE(type_1, env);
+    type_1 = NULL;
     return NULL;
 }
        
@@ -1299,14 +1333,21 @@
                             axis2_env_t **env,
                             axis2_char_t *axis_svc_name) 
 {
-    axis2_svc_impl_t *svc_impl = AXIS2_INTF_TO_IMPL(svc);
+    axis2_svc_impl_t *svc_impl = NULL;
+    
+    AXIS2_FUNC_PARAM_CHECK(svc, env, AXIS2_FAILURE);
+    AXIS2_PARAM_CHECK((*env)->error, axis_svc_name, AXIS2_FAILURE);
+    svc_impl = AXIS2_INTF_TO_IMPL(svc);
+    
     if(svc_impl->axis_svc_name)
     {
         AXIS2_FREE((*env)->allocator, svc_impl->axis_svc_name);
+        svc_impl->axis_svc_name = NULL;
     }
     svc_impl->axis_svc_name = AXIS2_STRDUP(axis_svc_name, env);
     if(!svc_impl->axis_svc_name)
     {
+        AXIS2_ERROR_SET((*env)->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return AXIS2_FAILURE;
     }
     return AXIS2_SUCCESS;
@@ -1349,10 +1390,12 @@
     if(svc_impl->filename)
     {
         AXIS2_FREE((*env)->allocator, svc_impl->filename);
+        svc_impl->filename = NULL;
     }
     svc_impl->filename = (axis2_char_t *) AXIS2_STRDUP(filename, env);
     if(!svc_impl->filename)
     {
+        AXIS2_ERROR_SET((*env)->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
         return AXIS2_FAILURE;
     }
     return AXIS2_SUCCESS;
@@ -1381,7 +1424,7 @@
     return AXIS2_WSDL_SVC_SET_ENDPOINT(svc->wsdl_svc, env, endpoint);
 }
 
-struct axis2_wsdl_endpoint * AXIS2_CALL
+axis2_wsdl_endpoint_t * AXIS2_CALL
 axis2_svc_get_endpoint(axis2_svc_t *svc,
                             axis2_env_t **env,
                             axis2_qname_t * qname) 
@@ -1400,9 +1443,14 @@
 axis2_svc_add_mapping(axis2_svc_t *svc,
                             axis2_env_t **env,
                             axis2_char_t * mapping_key , 
-                            struct axis2_op * axis2_opt)
+                            axis2_op_t * axis2_opt)
 {
-    axis2_svc_impl_t *svc_impl = AXIS2_INTF_TO_IMPL(svc);
+    axis2_svc_impl_t *svc_impl = NULL;
+    
+    AXIS2_FUNC_PARAM_CHECK(svc, env, AXIS2_FAILURE);
+    AXIS2_PARAM_CHECK((*env)->error, mapping_key, AXIS2_FAILURE);
+    AXIS2_PARAM_CHECK((*env)->error, axis2_opt, AXIS2_FAILURE);
+    svc_impl = AXIS2_INTF_TO_IMPL(svc);
     
     axis2_hash_set(svc_impl->wasaction_opeartionmap, mapping_key, 
         AXIS2_HASH_KEY_STRING, axis2_opt);
@@ -1417,7 +1465,8 @@
     AXIS2_FUNC_PARAM_CHECK(svc, env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK((*env)->error, moduleref, AXIS2_FAILURE);
     
-    return AXIS2_ARRAY_LIST_ADD(AXIS2_INTF_TO_IMPL(svc)->module_list, env, moduleref);
+    return AXIS2_ARRAY_LIST_ADD(AXIS2_INTF_TO_IMPL(svc)->module_list, env, 
+        moduleref);
 }
 
 axis2_array_list_t *AXIS2_CALL

Modified: webservices/axis2/trunk/c/modules/wsdl/wsdl_component.c
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/modules/wsdl/wsdl_component.c?rev=371527&r1=371526&r2=371527&view=diff
==============================================================================
--- webservices/axis2/trunk/c/modules/wsdl/wsdl_component.c (original)
+++ webservices/axis2/trunk/c/modules/wsdl/wsdl_component.c Mon Jan 23 03:21:15 2006
@@ -46,14 +46,14 @@
 axis2_wsdl_component_set_component_property(
 	                                    axis2_wsdl_component_t *wsdl_component, 
                                         axis2_env_t **env,
-                                        void *key, 
+                                        axis2_char_t *key, 
                                         void *value);
 	
 void * AXIS2_CALL
 axis2_wsdl_component_get_component_property(
 		                                axis2_wsdl_component_t *wsdl_component,
                                         axis2_env_t **env,
-                                        void *key);
+                                        axis2_char_t *key);
 
 
                                         
@@ -231,7 +231,6 @@
     
     AXIS2_FUNC_PARAM_CHECK(wsdl_component, env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK((*env)->error, properties, AXIS2_FAILURE);
-    
     component_impl = AXIS2_INTF_TO_IMPL(wsdl_component);
     
     if(component_impl->component_properties)
@@ -252,11 +251,12 @@
                 param_container = (axis2_param_container_t *) val;
                 AXIS2_PARAM_CONTAINER_FREE(param_container, env);
                 param_container = NULL;
+                val = NULL;
+                key = NULL;
             }
             
             if(0 == AXIS2_STRCMP(AXIS2_MODULEREF_KEY, (axis2_char_t *)key))
             {
-                void *val = NULL;
                 int i = 0;
                 int size = 0;
                 module_desc_list = (axis2_array_list_t *) val;
@@ -265,23 +265,19 @@
                     size = AXIS2_ARRAY_LIST_SIZE(module_desc_list, env);
                     for (i = 0; i < size; i++)
                     {
-                        struct axis2_module_desc *module_desc = NULL;
-                        module_desc = AXIS2_ARRAY_LIST_GET(module_desc_list, env, i);
-                        
-                        module_desc = (struct axis2_module_desc *) val;
+                        axis2_module_desc_t *module_desc = NULL;
+                        module_desc = AXIS2_ARRAY_LIST_GET(module_desc_list, 
+                            env, i);
                         if (module_desc)
+                        {
                            AXIS2_MODULE_DESC_FREE (module_desc, env);
-                        
-                        val = NULL;
-                        module_desc = NULL;
-                           
+                            module_desc = NULL;
+                        } 
                     }
                     AXIS2_ARRAY_LIST_FREE(module_desc_list, env);
                 }
-                
                 module_desc_list = NULL;
             }
-            
             val = NULL;
             key = NULL;
                
@@ -306,7 +302,7 @@
 axis2_wsdl_component_set_component_property(
                                         axis2_wsdl_component_t *wsdl_component, 
                                         axis2_env_t **env,
-                                        void *key, 
+                                        axis2_char_t *key, 
                                         void *value) 
 {
     axis2_wsdl_component_impl_t *component_impl = NULL;
@@ -314,10 +310,10 @@
     AXIS2_FUNC_PARAM_CHECK(wsdl_component, env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK((*env)->error, key, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK((*env)->error, value, AXIS2_FAILURE);
-    
     component_impl = AXIS2_INTF_TO_IMPL(wsdl_component);
     
-	axis2_hash_set (component_impl->component_properties, key, sizeof(key), value);
+	axis2_hash_set (component_impl->component_properties, key, 
+        AXIS2_HASH_KEY_STRING, value);
     
 	return AXIS2_SUCCESS;
 }
@@ -326,13 +322,13 @@
 axis2_wsdl_component_get_component_property(
 		                                axis2_wsdl_component_t *wsdl_component,
                                         axis2_env_t **env, 
-                                        void *key) 
+                                        axis2_char_t *key) 
 {
     AXIS2_FUNC_PARAM_CHECK(wsdl_component, env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK((*env)->error, key, AXIS2_FAILURE);
 	
 	return (void *)(axis2_hash_get (AXIS2_INTF_TO_IMPL(wsdl_component)->
-        component_properties, key, sizeof(key)));
+        component_properties, key, AXIS2_HASH_KEY_STRING));
 }
 
 axis2_status_t AXIS2_CALL
@@ -344,12 +340,16 @@
     
     AXIS2_FUNC_PARAM_CHECK(wsdl_component, env, AXIS2_FAILURE);
     AXIS2_PARAM_CHECK((*env)->error, element, AXIS2_FAILURE);
-    
     component_impl = AXIS2_INTF_TO_IMPL(wsdl_component);
     
     if (NULL == component_impl->elements) 
     {
         component_impl->elements = axis2_linked_list_create(env);
+        if(!component_impl->elements)
+        {
+            AXIS2_ERROR_SET((*env)->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
+            return AXIS2_FAILURE;
+        }
     }
     
     return AXIS2_LINKED_LIST_ADD(component_impl->elements, env, element);

Modified: webservices/axis2/trunk/c/test/unit/Makefile.am
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/test/unit/Makefile.am?rev=371527&r1=371526&r2=371527&view=diff
==============================================================================
--- webservices/axis2/trunk/c/test/unit/Makefile.am (original)
+++ webservices/axis2/trunk/c/test/unit/Makefile.am Mon Jan 23 03:21:15 2006
@@ -1,6 +1,6 @@
 prgbindir=$(prefix)/bin/unit_test
 prgbin_PROGRAMS = main_unit_test_suite
-SUBDIRS = util core xml
+SUBDIRS = util core xml wsdl
 AM_CPPFLAGS = $(CPPFLAGS)
 main_unit_test_suite_SOURCES = main.c
 
@@ -12,6 +12,7 @@
                     -L$(top_builddir)/test/unit/core/context/.libs -ltest_context \
                     -L$(top_builddir)/test/unit/core/phaseresolver/.libs -ltest_phaseresolver \
                     -L$(top_builddir)/test/unit/util/.libs -ltest_util \
+                    -L$(top_builddir)/test/unit/wsdl/.libs -ltest_wsdl \
                     -L$(top_builddir)/test/unit/xml/om/.libs -ltest_om \
                     -L$(AXIS2C_HOME)/lib \
                     -laxis2_util \

Modified: webservices/axis2/trunk/c/test/unit/main.c
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/test/unit/main.c?rev=371527&r1=371526&r2=371527&view=diff
==============================================================================
--- webservices/axis2/trunk/c/test/unit/main.c (original)
+++ webservices/axis2/trunk/c/test/unit/main.c Mon Jan 23 03:21:15 2006
@@ -3,13 +3,17 @@
 #include "xml/om/om_test.h"
 #include "core/phaseresolver/phaseresolver_test.h"
 #include "core/description/description_test.h"
+#include "core/engine/engine_test.h"
+#include "wsdl/wsdl_test.h"
 
 void RunAllTests(void) {
     CuString *output = CuStringNew();
     CuSuite* suite = CuSuiteNew();
 
     CuSuiteAddSuite(suite, axis2_utilGetSuite());
+    CuSuiteAddSuite(suite, axis2_wsdlGetSuite());
     CuSuiteAddSuite(suite, axis2_omGetSuite());
+    CuSuiteAddSuite(suite, axis2_engineGetSuite());
     
     CuSuiteAddSuite(suite, axis2_phaseresolverGetSuite());
     CuSuiteAddSuite(suite, axis2_descriptionGetSuite());

Modified: webservices/axis2/trunk/c/test/unit/wsdl/Makefile.am
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/test/unit/wsdl/Makefile.am?rev=371527&r1=371526&r2=371527&view=diff
==============================================================================
--- webservices/axis2/trunk/c/test/unit/wsdl/Makefile.am (original)
+++ webservices/axis2/trunk/c/test/unit/wsdl/Makefile.am Mon Jan 23 03:21:15 2006
@@ -2,18 +2,29 @@
 prglibdir=$(prefix)/lib/unit_test
 prglib_LTLIBRARIES = libtest_wsdl.la
 AM_CPPFLAGS = $(CPPFLAGS)
-libtest_wsdl_la_SOURCES =
-include_HEADERS=$(top_builddir)/modules/wsdl/test/*.h
 
-libtest_wsdl_la_LIBADD =  -L$(top_builddir)/modules/wsdl/src/.libs \
-                            -laxis2_wsdl
+libtest_wsdl_la_SOURCES = wsdl_test.c \
+                            test_wsdl_component.c
+
+include_HEADERS=$(top_builddir)/test/unit/wsdl/*.h
 
 INCLUDES = -I$(top_builddir)/include \
-            -I${CUTEST_HOME}/include
+            -I${CUTEST_HOME}/include \
+            -I$(top_builddir)/modules/util \
+            -I$(top_builddir)/modules/wsdl \
+            -I$(top_builddir)/modules/core/description \
+            -I$(top_builddir)/modules/core/transport \
+            -I$(top_builddir)/modules/platforms
+
 
 prgbin_PROGRAMS = wsdl_test
 wsdl_test_SOURCES = main.c
-wsdl_test_LDADD   =   libtest_wsdl.la  \
-                    -L$(top_builddir)/modules/wsdl/src/.libs -laxis2_wsdl \
-                    -L$(top_builddir)/modules/util/src/.libs -laxis2_util
+
+wsdl_test_LDADD   =   \
+                     -L$(CUTEST_HOME)/lib \
+                     -lcutest \
+                     $(top_builddir)/test/unit/wsdl/libtest_wsdl.la \
+                    -L$(AXIS2C_HOME)/lib \
+                    -laxis2_util \
+                    -laxis2_wsdl
 

Modified: webservices/axis2/trunk/c/test/unit/wsdl/main.c
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/test/unit/wsdl/main.c?rev=371527&r1=371526&r2=371527&view=diff
==============================================================================
--- webservices/axis2/trunk/c/test/unit/wsdl/main.c (original)
+++ webservices/axis2/trunk/c/test/unit/wsdl/main.c Mon Jan 23 03:21:15 2006
@@ -1,18 +1,20 @@
 #include "wsdl_test.h"
 
-void RunAllTests(void) {
+void RunAllTests(void) 
+{
     CuString *output = CuStringNew();
     CuSuite* suite = CuSuiteNew();
 
-    /*CuSuiteAddSuite(suite, axis2_wsdlGetSuite());
+    CuSuiteAddSuite(suite, axis2_wsdlGetSuite());
 
     CuSuiteRun(suite);
     CuSuiteSummary(suite, output);
     CuSuiteDetails(suite, output);
-    printf("%s\n", output->buffer);*/
+    printf("%s\n", output->buffer);
 }
 
-int main(void) {
+int main(void) 
+{
     RunAllTests();
     return 0;
 }

Modified: webservices/axis2/trunk/c/test/unit/wsdl/wsdl_test.h
URL: http://svn.apache.org/viewcvs/webservices/axis2/trunk/c/test/unit/wsdl/wsdl_test.h?rev=371527&r1=371526&r2=371527&view=diff
==============================================================================
--- webservices/axis2/trunk/c/test/unit/wsdl/wsdl_test.h (original)
+++ webservices/axis2/trunk/c/test/unit/wsdl/wsdl_test.h Mon Jan 23 03:21:15 2006
@@ -2,6 +2,7 @@
 #define WSDL_TEST_H
 
 #include <CuTest.h>
+#include "test_wsdl_component.h"
 
 CuSuite* axis2_wsdlGetSuite();