Changeset 5000 for pjproject/trunk/pjsip/src/pjsip/sip_transport_tcp.c

Timestamp:

Mar 19, 2015 5:07:01 AM (9 years ago)

Author:

riza

Message:

Misc (re #1782): Check if the transport being shutdown in on_accept_complete and on_connect_complete callback

File:

• pjproject/trunk/pjsip/src/pjsip/sip_transport_tcp.c (modified) (5 diffs)

pjproject/trunk/pjsip/src/pjsip/sip_transport_tcp.c

@@ -1096 +1096 @@
     PJ_ASSERT_RETURN(sock != PJ_INVALID_SOCKET, PJ_TRUE);
 
+    if (!listener->is_registered)
+        return PJ_FALSE;
+
     PJ_LOG(4,(listener->factory.obj_name, 
               "TCP listener %.*s:%d: got incoming TCP connection "
@@ -1134 +1137 @@
             tcp_destroy(&tcp->base, status);
         } else {
+            if (tcp->base.is_shutdown || tcp->base.is_destroying) {
+                return PJ_TRUE;
+            }
             /* Start keep-alive timer */
             if (PJSIP_TCP_KEEP_ALIVE_INTERVAL) {
@@ -1143 +1149 @@
                 pj_gettimeofday(&tcp->last_activity);
             }
-
             /* Notify application of transport state accepted */
             state_cb = pjsip_tpmgr_get_state_cb(tcp->base.tpmgr);
             if (state_cb) {
                 pjsip_transport_state_info state_info;
-            
+
                 pj_bzero(&state_info, sizeof(state_info));
                 (*state_cb)(&tcp->base, PJSIP_TP_STATE_CONNECTED, &state_info);
@@ -1423 +1428 @@
     tcp->has_pending_connect = PJ_FALSE;
 
+    if (tcp->base.is_shutdown || tcp->base.is_destroying) 
+        return PJ_FALSE;
+
     /* Check connect() status */
     if (status != PJ_SUCCESS) {
@@ -1483 +1491 @@
     if (state_cb) {
         pjsip_transport_state_info state_info;
-    
+        
         pj_bzero(&state_info, sizeof(state_info));
         (*state_cb)(&tcp->base, PJSIP_TP_STATE_CONNECTED, &state_info);