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 sa...@apache.org on 2007/03/28 09:50:37 UTC

svn commit: r523215 - in /webservices/axis2/trunk/c/modules/core/transport/http/sender: http_client.c http_sender.c

Author: samisa
Date: Wed Mar 28 00:50:31 2007
New Revision: 523215

URL: http://svn.apache.org/viewvc?view=rev&rev=523215
Log:
Fixed memory leak due to not freeing client - AXIS2C-536

Modified:
    webservices/axis2/trunk/c/modules/core/transport/http/sender/http_client.c
    webservices/axis2/trunk/c/modules/core/transport/http/sender/http_sender.c

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=523215&r1=523214&r2=523215
==============================================================================
--- 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 Wed Mar 28 00:50:31 2007
@@ -115,7 +115,7 @@
 {
     axis2_http_client_t *client_l = NULL;
     AXIS2_ENV_CHECK(env, void);
-
+    
     client_l = (axis2_http_client_t *)client;
     axis2_http_client_free(client_l, env);
     return;

Modified: webservices/axis2/trunk/c/modules/core/transport/http/sender/http_sender.c
URL: http://svn.apache.org/viewvc/webservices/axis2/trunk/c/modules/core/transport/http/sender/http_sender.c?view=diff&rev=523215&r1=523214&r2=523215
==============================================================================
--- webservices/axis2/trunk/c/modules/core/transport/http/sender/http_sender.c (original)
+++ webservices/axis2/trunk/c/modules/core/transport/http/sender/http_sender.c Wed Mar 28 00:50:31 2007
@@ -240,17 +240,16 @@
 
 	if (!send_via_get)
 	{
+        axis2_property_t *property = NULL;
 
 		/* We put the client into msg_ctx so that we can free it once the processing
 		 * is done at client side
 		 */
-		/*property = axis2_property_create(env);
-		  axis2_property_set_scope(property, env, AXIS2_SCOPE_REQUEST);
-		  axis2_property_set_free_func(property, env,
-		  axis2_http_client_free_void_arg);
-		  axis2_property_set_value(property, env, sender->client);
-		   axis2_msg_ctx_set_property(msg_ctx, env, AXIS2_HTTP_CLIENT,
-		  property);*/
+        property = axis2_property_create(env);
+        axis2_property_set_scope(property, env, AXIS2_SCOPE_REQUEST);
+        axis2_property_set_free_func(property, env, axis2_http_client_free_void_arg);
+        axis2_property_set_value(property, env, sender->client);
+        axis2_msg_ctx_set_property(msg_ctx, env, AXIS2_HTTP_CLIENT, property);
 
 		doing_mtom =  axis2_msg_ctx_get_doing_mtom(msg_ctx, env);
 



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