Changeset 175 for pjproject/trunk/pjsip/src/pjsip/sip_dialog.c

Timestamp:

Feb 10, 2006 2:04:05 PM (18 years ago)

Author:

bennylp

Message:

Added more logging, and fixed bug when ACK is responded!

File:

• pjproject/trunk/pjsip/src/pjsip/sip_dialog.c (modified) (12 diffs)

pjproject/trunk/pjsip/src/pjsip/sip_dialog.c

@@ -201 +201 @@
     *p_dlg = dlg;
 
+
+    PJ_LOG(5,(dlg->obj_name, "UAC dialog created"));
+
     return PJ_SUCCESS;
 
@@ -381 +384 @@
     /* Done. */
     *p_dlg = dlg;
+    PJ_LOG(5,(dlg->obj_name, "UAS dialog created"));
     return PJ_SUCCESS;
 
@@ -496 +500 @@
     *new_dlg = dlg;
 
+    PJ_LOG(5,(dlg->obj_name, "Forked dialog created"));
     return PJ_SUCCESS;
 
@@ -577 +582 @@
  * Increment session counter.
  */
-PJ_DEF(pj_status_t) pjsip_dlg_inc_session( pjsip_dialog *dlg )
-{
-    PJ_ASSERT_RETURN(dlg, PJ_EINVAL);
+PJ_DEF(pj_status_t) pjsip_dlg_inc_session( pjsip_dialog *dlg,
+                                           pjsip_module *mod )
+{
+    PJ_ASSERT_RETURN(dlg && mod, PJ_EINVAL);
 
     pj_mutex_lock(dlg->mutex);
@@ -585 +591 @@
     pj_mutex_unlock(dlg->mutex);
 
+    PJ_LOG(5,(dlg->obj_name, "Session count inc to %d by %.*s",
+              dlg->sess_count, (int)mod->name.slen, mod->name.ptr));
+
     return PJ_SUCCESS;
 }
@@ -591 +600 @@
  * Decrement session counter.
  */
-PJ_DEF(pj_status_t) pjsip_dlg_dec_session( pjsip_dialog *dlg )
+PJ_DEF(pj_status_t) pjsip_dlg_dec_session( pjsip_dialog *dlg,
+                                           pjsip_module *mod)
 {
     pj_status_t status;
 
     PJ_ASSERT_RETURN(dlg, PJ_EINVAL);
+
+    PJ_LOG(5,(dlg->obj_name, "Session count dec to %d by %.*s",
+              dlg->sess_count-1, (int)mod->name.slen, mod->name.ptr));
 
     pj_mutex_lock(dlg->mutex);
@@ -626 +639 @@
                      PJ_EINVAL);
     PJ_ASSERT_RETURN(dlg->usage_cnt < PJSIP_MAX_MODULE, PJ_EBUG);
+
+    PJ_LOG(5,(dlg->obj_name, "Module %.*s added as dialog usage",
+              (int)mod->name.slen, mod->name.ptr));
 
     pj_mutex_lock(dlg->mutex);
@@ -795 +811 @@
                      PJSIP_ENOTREQUESTMSG);
 
+    PJ_LOG(5,(dlg->obj_name, "Sending %s",
+              pjsip_tx_data_get_info(tdata)));
+
     /* Update CSeq */
     pj_mutex_lock(dlg->mutex);
@@ -1024 +1043 @@
     PJ_ASSERT_RETURN(tsx->mod_data[dlg->ua->id] == dlg, PJ_EINVALIDOP);
 
+    PJ_LOG(5,(dlg->obj_name, "Sending %s",
+              pjsip_tx_data_get_info(tdata)));
+
     /* Check that transaction method and cseq match the response. 
      * This operation is sloooww (search CSeq header twice), that's why
@@ -1078 +1100 @@
     pjsip_transaction *tsx = NULL;
     unsigned i;
+
+    PJ_LOG(5,(dlg->obj_name, "Received %s",
+              pjsip_rx_data_get_info(rdata)));
 
     /* Lock the dialog. */
@@ -1144 +1169 @@
     int res_code;
 
+    PJ_LOG(5,(dlg->obj_name, "Received %s",
+              pjsip_rx_data_get_info(rdata)));
+
     /* Lock the dialog. */
     pj_mutex_lock(dlg->mutex);
@@ -1250 +1278 @@
     unsigned i;
 
+    PJ_LOG(5,(dlg->obj_name, "Transaction %s state changed to %s",
+              tsx->obj_name, pjsip_tsx_state_str(tsx->state)));
+
     /* Lock the dialog. */
     pj_mutex_lock(dlg->mutex);