svn commit: r713062 - /webservices/savan/trunk/c/src/subs_mgr/subs_mgr.c

[damitha]

Author: damitha
Date: Tue Nov 11 07:37:04 2008
New Revision: 713062

URL: http://svn.apache.org/viewvc?rev=713062&view=rev
Log:
Adding trace messages and handling null subscriber scenario

Modified:
    webservices/savan/trunk/c/src/subs_mgr/subs_mgr.c

Modified: webservices/savan/trunk/c/src/subs_mgr/subs_mgr.c
URL: 
http://svn.apache.org/viewvc/webservices/savan/trunk/c/src/subs_mgr/subs_mgr.c?rev=713062&r1=713061&r2=713062&view=diff
==============================================================================
--- webservices/savan/trunk/c/src/subs_mgr/subs_mgr.c (original)
+++ webservices/savan/trunk/c/src/subs_mgr/subs_mgr.c Tue Nov 11 07:37:04 2008
@@ -52,11 +52,18 @@
     
     axis2_char_t *id = NULL;
 
-    AXIS2_LOG_DEBUG(env->log, AXIS2_LOG_SI, "[savan] 
Start:savan_subs_mgr_add_subscriber");
+    AXIS2_LOG_TRACE(env->log, AXIS2_LOG_SI, "[savan] 
Entry:savan_subs_mgr_add_subscriber");
 
     conf_ctx = axis2_msg_ctx_get_conf_ctx(msg_ctx, env);
     conf = axis2_conf_ctx_get_conf(conf_ctx, env);
     subscriber = savan_subscriber_create(env);
+    if(!subscriber)
+    {
+        AXIS2_ERROR_SET(env->error, AXIS2_ERROR_NO_MEMORY, AXIS2_FAILURE);
+        AXIS2_LOG_ERROR(env->log, AXIS2_LOG_SI, "[savan] Cound not create 
subscriber");
+        return NULL;
+    }
+
     if(add_sub_node)
     {
         add_sub_elem = 
(axiom_element_t*)axiom_node_get_data_element(add_sub_node, env);
@@ -125,7 +132,7 @@
                 "[savan] Subscriber %s could not be added to the topic:%s", 
id, topic_url);
     }
 
-    AXIS2_LOG_DEBUG(env->log, AXIS2_LOG_SI, "[savan] 
End:savan_subs_mgr_add_subscriber");
+    AXIS2_LOG_TRACE(env->log, AXIS2_LOG_SI, "[savan] 
Exit:savan_subs_mgr_add_subscriber");
     return NULL;   
 }
 
@@ -151,7 +158,7 @@
     axis2_conf_ctx_t *conf_ctx = NULL;
     axis2_conf_t *conf = NULL;
 
-    AXIS2_LOG_DEBUG(env->log, AXIS2_LOG_SI, "[savan] 
Start:savan_subs_mgr_remove_subscriber");
+    AXIS2_LOG_TRACE(env->log, AXIS2_LOG_SI, "[savan] 
Entry:savan_subs_mgr_remove_subscriber");
 
     conf_ctx = axis2_msg_ctx_get_conf_ctx(msg_ctx, env);
     conf = axis2_conf_ctx_get_conf(conf_ctx, env);
@@ -187,8 +194,8 @@
     AXIS2_LOG_DEBUG(env->log, AXIS2_LOG_SI, 
         "[savan] Subscriber %s removed from the topic:%s", id, topic_url);
 
-    AXIS2_LOG_DEBUG(env->log, AXIS2_LOG_SI, 
-        "[savan] End:savan_subs_mgr_remove_subscriber");
+    AXIS2_LOG_TRACE(env->log, AXIS2_LOG_SI, 
+        "[savan] Exit:savan_subs_mgr_remove_subscriber");
 
     return NULL;   
 }
@@ -280,7 +287,7 @@
         }
     }
 
-    AXIS2_LOG_DEBUG(env->log, AXIS2_LOG_SI, "[savan] 
Exit:savan_subs_mgr_get_subscriber");
+    AXIS2_LOG_TRACE(env->log, AXIS2_LOG_SI, "[savan] 
Exit:savan_subs_mgr_get_subscriber");
     return subs_node;
 }