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 sa...@apache.org on 2006/11/24 08:32:44 UTC

svn commit: r478790 - in /webservices/axis2/trunk/c: modules/core/clientapi/ modules/core/transport/http/ modules/core/transport/http/sender/ samples/client/math/

Author: samisa
Date: Thu Nov 23 23:32:42 2006
New Revision: 478790

URL: http://svn.apache.org/viewvc?view=rev&rev=478790
Log:
Fixed more memory leaks, specially those related to cased where the invocation fails

Modified:
    webservices/axis2/trunk/c/modules/core/clientapi/mep_client.c
    webservices/axis2/trunk/c/modules/core/clientapi/svc_client.c
    webservices/axis2/trunk/c/modules/core/transport/http/http_transport_utils.c
    webservices/axis2/trunk/c/modules/core/transport/http/sender/http_client.c
    webservices/axis2/trunk/c/samples/client/math/math_client.c

Modified: webservices/axis2/trunk/c/modules/core/clientapi/mep_client.c
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/c/modules/core/clientapi/mep_client.c?view=diff&rev=478790&r1=478789&r2=478790
==============================================================================
--- webservices/axis2/trunk/c/modules/core/clientapi/mep_client.c (original)
+++ webservices/axis2/trunk/c/modules/core/clientapi/mep_client.c Thu Nov 23 23:32:42 2006
@@ -703,6 +703,12 @@
         if (AXIS2_ERROR_GET_STATUS_CODE(env->error) != AXIS2_SUCCESS)
         {
             AXIS2_ERROR_SET(env->error, AXIS2_ERROR_BLOCKING_INVOCATION_EXPECTS_RESPONSE, AXIS2_FAILURE);
+            if (engine)
+            {
+                AXIS2_ENGINE_FREE(engine, env);
+                engine = NULL;
+            }
+            AXIS2_MSG_CTX_FREE(response, env);
             return NULL;
         }
     }

Modified: webservices/axis2/trunk/c/modules/core/clientapi/svc_client.c
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/c/modules/core/clientapi/svc_client.c?view=diff&rev=478790&r1=478789&r2=478790
==============================================================================
--- webservices/axis2/trunk/c/modules/core/clientapi/svc_client.c (original)
+++ webservices/axis2/trunk/c/modules/core/clientapi/svc_client.c Thu Nov 23 23:32:42 2006
@@ -898,8 +898,6 @@
 
         if (res_msg_ctx)
           soap_envelope = AXIS2_MSG_CTX_GET_SOAP_ENVELOPE(res_msg_ctx, env);
-        else
-          return NULL;
     }
 
     if (qname_free_flag)

Modified: webservices/axis2/trunk/c/modules/core/transport/http/http_transport_utils.c
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/c/modules/core/transport/http/http_transport_utils.c?view=diff&rev=478790&r1=478789&r2=478790
==============================================================================
--- webservices/axis2/trunk/c/modules/core/transport/http/http_transport_utils.c (original)
+++ webservices/axis2/trunk/c/modules/core/transport/http/http_transport_utils.c Thu Nov 23 23:32:42 2006
@@ -1042,6 +1042,7 @@
     {
         AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NULL_IN_STREAM_IN_MSG_CTX,
                 AXIS2_FAILURE);
+        AXIS2_FREE(env->allocator, callback_ctx);
         return NULL;
     }
     property = AXIS2_MSG_CTX_GET_PROPERTY(msg_ctx, env,

Modified: webservices/axis2/trunk/c/modules/core/transport/http/sender/http_client.c
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/c/modules/core/transport/http/sender/http_client.c?view=diff&rev=478790&r1=478789&r2=478790
==============================================================================
--- webservices/axis2/trunk/c/modules/core/transport/http/sender/http_client.c (original)
+++ webservices/axis2/trunk/c/modules/core/transport/http/sender/http_client.c Thu Nov 23 23:32:42 2006
@@ -290,6 +290,10 @@
     if (client_impl->sockfd < 0)
     {
         AXIS2_ERROR_SET(env->error, AXIS2_ERROR_SOCKET_ERROR, AXIS2_FAILURE);
+        if (str_body)
+        {
+            AXIS2_FREE(env->allocator, str_body);
+        }
         return AXIS2_FAILURE;
     }
     /* ONLY FOR TESTING

Modified: webservices/axis2/trunk/c/samples/client/math/math_client.c
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/c/samples/client/math/math_client.c?view=diff&rev=478790&r1=478789&r2=478790
==============================================================================
--- webservices/axis2/trunk/c/samples/client/math/math_client.c (original)
+++ webservices/axis2/trunk/c/samples/client/math/math_client.c Thu Nov 23 23:32:42 2006
@@ -117,7 +117,6 @@
                 " %d :: %s", env->error->error_number,
                 AXIS2_ERROR_GET_MESSAGE(env->error));
         printf("math stub invoke FAILED!\n");
-		  return -1;
     }
 
     if (stub)



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