summaryrefslogtreecommitdiff
path: root/pjsip/src/pjsip/sip_transport_tcp.c
diff options
context:
space:
mode:
Diffstat (limited to 'pjsip/src/pjsip/sip_transport_tcp.c')
-rw-r--r--pjsip/src/pjsip/sip_transport_tcp.c12
1 files changed, 10 insertions, 2 deletions
diff --git a/pjsip/src/pjsip/sip_transport_tcp.c b/pjsip/src/pjsip/sip_transport_tcp.c
index d2e0a28b..222cb13f 100644
--- a/pjsip/src/pjsip/sip_transport_tcp.c
+++ b/pjsip/src/pjsip/sip_transport_tcp.c
@@ -1470,8 +1470,16 @@ static pj_bool_t on_connect_complete(pj_activesock_t *asock,
/* Mark that pending connect() operation has completed. */
tcp->has_pending_connect = PJ_FALSE;
- if (tcp->base.is_shutdown || tcp->base.is_destroying)
- return PJ_FALSE;
+ /* If transport is being shutdown/destroyed, proceed as error connect.
+ * Note that it is important to notify application via on_data_sent()
+ * as otherwise the transport reference counter may never reach zero
+ * (see #1898).
+ */
+ if ((tcp->base.is_shutdown || tcp->base.is_destroying) &&
+ status == PJ_SUCCESS)
+ {
+ status = PJ_ECANCELLED;
+ }
/* Check connect() status */
if (status != PJ_SUCCESS) {