You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-dev@axis.apache.org by ma...@apache.org on 2007/09/04 06:53:20 UTC

svn commit: r572519 - in /webservices/axis2/trunk/c/axiom/src/om: om_element.c om_stax_builder.c

Author: manjula
Date: Mon Sep  3 21:53:19 2007
New Revision: 572519

URL: http://svn.apache.org/viewvc?rev=572519&view=rev
Log:
Fixing memory leaks in axiom.

Modified:
    webservices/axis2/trunk/c/axiom/src/om/om_element.c
    webservices/axis2/trunk/c/axiom/src/om/om_stax_builder.c

Modified: webservices/axis2/trunk/c/axiom/src/om/om_element.c
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/c/axiom/src/om/om_element.c?rev=572519&r1=572518&r2=572519&view=diff
==============================================================================
--- webservices/axis2/trunk/c/axiom/src/om/om_element.c (original)
+++ webservices/axis2/trunk/c/axiom/src/om/om_element.c Mon Sep  3 21:53:19 2007
@@ -1406,7 +1406,7 @@
     axiom_attribute_t *cloned_attr = NULL;
 
     axiom_namespace_t *om_ns = NULL;
-    axiom_namespace_t *cloned_ns = NULL;
+    /*axiom_namespace_t *cloned_ns = NULL;*/
 
     axis2_char_t *key = NULL;
     axutil_qname_t *qn = NULL;
@@ -1436,8 +1436,9 @@
             om_ns = axiom_attribute_get_namespace(om_attr, env);
             if (om_ns)
             {
-                cloned_ns = axiom_namespace_clone(om_ns, env);
-                axiom_attribute_set_namespace(cloned_attr, env, cloned_ns);
+                /*cloned_ns = axiom_namespace_clone(om_ns, env);*/
+                /*axiom_attribute_set_namespace(cloned_attr, env, cloned_ns);*/
+                axiom_attribute_set_namespace(cloned_attr, env, om_ns);                
             }
             qn = axiom_attribute_get_qname(cloned_attr, env);
             key = axutil_qname_to_string(qn, env);
@@ -1449,7 +1450,7 @@
         om_attr = NULL;
         cloned_attr = NULL;
         om_ns = NULL;
-        cloned_ns = NULL;
+        /*cloned_ns = NULL;*/
     }
     return ht_cloned;
 }

Modified: webservices/axis2/trunk/c/axiom/src/om/om_stax_builder.c
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/c/axiom/src/om/om_stax_builder.c?rev=572519&r1=572518&r2=572519&view=diff
==============================================================================
--- webservices/axis2/trunk/c/axiom/src/om/om_stax_builder.c (original)
+++ webservices/axis2/trunk/c/axiom/src/om/om_stax_builder.c Mon Sep  3 21:53:19 2007
@@ -345,6 +345,13 @@
             /** default namespace case */
             /** !temp_ns_prefix is for guththila */
             axiom_element_t *om_ele = NULL;
+
+            if(temp_ns_prefix_str)
+            {
+                axutil_string_free(temp_ns_prefix_str, env);
+                temp_ns_prefix_str = NULL;
+            }    
+            
             temp_ns_prefix_str = axutil_string_create(env, "");
             om_ele = (axiom_element_t *)axiom_node_get_data_element(node, env);
 
@@ -380,6 +387,7 @@
             axutil_hash_set(om_builder->declared_namespaces,
                 prefix, AXIS2_HASH_KEY_STRING, om_ns);
         }
+        
         axutil_string_free(temp_ns_uri_str, env);
         axutil_string_free(temp_ns_prefix_str, env);
 #ifdef WIN32	



---------------------------------------------------------------------
To unsubscribe, e-mail: axis-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: axis-cvs-help@ws.apache.org