You are viewing a plain text version of this content. The canonical link for it is here.
Posted to savan-dev@ws.apache.org by da...@apache.org on 2010/02/10 15:43:21 UTC
svn commit: r908520 -
/webservices/savan/trunk/c/src/msgreceivers/savan_msg_recv.c
Author: damitha
Date: Wed Feb 10 14:43:21 2010
New Revision: 908520
URL: http://svn.apache.org/viewvc?rev=908520&view=rev
Log:
Fixing memory leak
Modified:
webservices/savan/trunk/c/src/msgreceivers/savan_msg_recv.c
Modified: webservices/savan/trunk/c/src/msgreceivers/savan_msg_recv.c
URL: http://svn.apache.org/viewvc/webservices/savan/trunk/c/src/msgreceivers/savan_msg_recv.c?rev=908520&r1=908519&r2=908520&view=diff
==============================================================================
--- webservices/savan/trunk/c/src/msgreceivers/savan_msg_recv.c (original)
+++ webservices/savan/trunk/c/src/msgreceivers/savan_msg_recv.c Wed Feb 10 14:43:21 2010
@@ -500,8 +500,14 @@
{
AXIS2_LOG_ERROR(env->log, AXIS2_LOG_SI, "[savan] Could not create the data resource. Check \
whether resource path is correct and accessible. Exit loading the Savan module");
- AXIS2_HANDLE_ERROR(env, SAVAN_ERROR_DATABASE_CREATION_ERROR, AXIS2_FAILURE);
+ return AXIS2_FAILURE;
+ }
+ subscriber = savan_subs_mgr_get_subscriber_from_msg(env, msg_ctx, subs_mgr, NULL);
+ if(!subscriber)
+ {
+ AXIS2_LOG_ERROR(env->log, AXIS2_LOG_SI,
+ "[savan] Failed get subscriber using msg from subs manager");
return AXIS2_FAILURE;
}
@@ -510,7 +516,6 @@
{
AXIS2_LOG_ERROR(env->log, AXIS2_LOG_SI,
"[savan] Failed to build soap envelope for response message");
- AXIS2_HANDLE_ERROR(env, SAVAN_ERROR_FAILED_TO_BUILD_SOAP_ENV, AXIS2_FAILURE);
return AXIS2_FAILURE;
}
@@ -524,15 +529,6 @@
/* Expires element */
- subscriber = savan_subs_mgr_get_subscriber_from_msg(env, msg_ctx, subs_mgr, NULL);
- if(!subscriber)
- {
- AXIS2_LOG_ERROR(env->log, AXIS2_LOG_SI,
- "[savan] Failed get subscriber using msg from subs manager");
- AXIS2_HANDLE_ERROR(env, SAVAN_ERROR_SUBSCRIBER_NOT_FOUND, AXIS2_FAILURE);
-
- return AXIS2_FAILURE;
- }
expires = savan_subscriber_get_expires(subscriber, env);
if(expires)
{