Changeset 3077 for pjproject/trunk/pjsip/src/pjsip/sip_transport.c

Timestamp:

Jan 27, 2010 9:09:27 AM (14 years ago)

Author:

bennylp

Message:

Fixed ticket #1036: Stray timer may be registered by SIP TCP and TLS transport in destroy routine, which could crash the program on Symbian (thanks Tamàs Solymosi for the report)

File:

• pjproject/trunk/pjsip/src/pjsip/sip_transport.c (modified) (2 diffs)

pjproject/trunk/pjsip/src/pjsip/sip_transport.c

@@ -817 +817 @@
     if (pj_atomic_dec_and_get(tp->ref_cnt) == 0) {
         pj_lock_acquire(tp->tpmgr->lock);
-        /* Verify again. */
-        if (pj_atomic_get(tp->ref_cnt) == 0) {
+        /* Verify again. Do not register timer if the transport is
+         * being destroyed.
+         */
+        if (pj_atomic_get(tp->ref_cnt) == 0 && !tp->is_destroying) {
             pj_time_val delay;
             
@@ -898 +900 @@
     pj_lock_acquire(tp->lock);
     pj_lock_acquire(mgr->lock);
+
+    tp->is_destroying = PJ_TRUE;
 
     /*