From 9719eff91e5fc7382d3ae42dd50580c638f63b93 Mon Sep 17 00:00:00 2001
From: shankar
Date: Mon, 5 Apr 2010 14:02:51 +0000
Subject: Fixing invalid error loggings

git-svn-id: http://svn.apache.org/repos/asf/axis/axis2/c/core/trunk@930860 13f79535-47bb-0310-9956-ffa450edef68
---
 axiom/src/om/om_children_qname_iterator.c |  2 --
 axiom/src/soap/soap_body.c                | 29 +++++++++++++++++------------
 axiom/src/soap/soap_builder.c             |  1 +
 3 files changed, 18 insertions(+), 14 deletions(-)

(limited to 'axiom')

diff --git a/axiom/src/om/om_children_qname_iterator.c b/axiom/src/om/om_children_qname_iterator.c
index d508a9e..b1b8152 100644
--- a/axiom/src/om/om_children_qname_iterator.c
+++ b/axiom/src/om/om_children_qname_iterator.c
@@ -38,8 +38,6 @@ axiom_children_qname_iterator_create(
 {
     axiom_children_qname_iterator_t *iterator = NULL;
 
-    AXIS2_ENV_CHECK(env, NULL);
-    AXIS2_PARAM_CHECK(env->error, current_child, NULL);
 
     iterator = (axiom_children_qname_iterator_t *)AXIS2_MALLOC(env->allocator,
         sizeof(axiom_children_qname_iterator_t));
diff --git a/axiom/src/soap/soap_body.c b/axiom/src/soap/soap_body.c
index 8fb7d8f..69ee7ae 100644
--- a/axiom/src/soap/soap_body.c
+++ b/axiom/src/soap/soap_body.c
@@ -144,21 +144,26 @@ axiom_soap_body_has_fault(
     }
     else
     {
-        while(!axiom_node_is_complete(soap_body->om_ele_node, env))
+        /* This soap body could have been built programatically. Do the following only if soap
+         * body is created from soap_builder */
+        if(soap_body->soap_builder)
         {
-            if(axiom_soap_builder_next(soap_body->soap_builder, env) != AXIS2_SUCCESS)
+            while(!axiom_node_is_complete(soap_body->om_ele_node, env))
             {
-                /* problem in building the SOAP body. Note that has_fault is about soap fault,
-                 * not about problem in building the node. So, even though there is a problem
-                 * building the body, has_fault should be AXIS2_FALSE
-                 */
-                break;
-            }
+                if(axiom_soap_builder_next(soap_body->soap_builder, env) != AXIS2_SUCCESS)
+                {
+                    /* problem in building the SOAP body. Note that has_fault is about soap fault,
+                     * not about problem in building the node. So, even though there is a problem
+                     * building the body, has_fault should be AXIS2_FALSE
+                     */
+                    break;
+                }
 
-            if(soap_body->soap_fault)
-            {
-                soap_body->has_fault = AXIS2_TRUE;
-                break;
+                if(soap_body->soap_fault)
+                {
+                    soap_body->has_fault = AXIS2_TRUE;
+                    break;
+                }
             }
         }
     }
diff --git a/axiom/src/soap/soap_builder.c b/axiom/src/soap/soap_builder.c
index 0b5829e..f8236ad 100644
--- a/axiom/src/soap/soap_builder.c
+++ b/axiom/src/soap/soap_builder.c
@@ -151,6 +151,7 @@ axiom_soap_builder_create(
     soap_builder->soap_version = AXIOM_SOAP12;
     soap_builder->last_node_status = -1;
     soap_builder->om_builder = stax_builder;
+    soap_builder->done = AXIS2_FALSE;
 
     status = axiom_soap_builder_identify_soap_version(soap_builder, env, soap_version);
     if(status != AXIS2_SUCCESS)
-- 
cgit v1.1-32-gdbae