Changeset 1903

Timestamp:

Apr 1, 2008 6:21:46 PM (17 years ago)

Author:

bennylp

Message:

Ticket #519: STUN request is not freed when transaction times out (thanks Amit Sharma)

File:

• pjproject/trunk/pjnath/src/pjnath/stun_session.c (modified) (2 diffs)

pjproject/trunk/pjnath/src/pjnath/stun_session.c

@@ -382 +382 @@
                                         src_addr, src_addr_len);
     }
+
+    /* Destroy the transmit data. This will remove the transaction
+     * from the pending list too. 
+     */
+    pj_stun_msg_destroy_tdata(sess, tdata);
+    tdata = NULL;
 }
 
@@ -1033 +1039 @@
     }
 
-    /* If transaction has completed, destroy the transmit data.
-     * This will remove the transaction from the pending list too.
-     */
-    if (pj_stun_client_tsx_is_complete(tdata->client_tsx)) {
-        pj_stun_msg_destroy_tdata(sess, tdata);
-        tdata = NULL;
-    }
-
     return PJ_SUCCESS;
 }