diff options
author | Alexei Gradinari <alex2grad@gmail.com> | 2016-07-19 16:22:39 -0400 |
---|---|---|
committer | Alexei Gradinari <alex2grad@gmail.com> | 2016-07-21 11:21:05 -0400 |
commit | 5997ec7c9e70f3b12afb948c421355fb87aed44e (patch) | |
tree | 29ce7c5e422d525d1095180061f1aa05acd70d18 /res/res_pjsip_pubsub.c | |
parent | 962c7ef5d91c546794313e01b4c264efdf3d1ead (diff) |
res_pjsip_pubsub: fixed a bug when pjsip_tx_data_dec_ref is called twice.
This patch removed call of pjsip_tx_data_dec_ref in send_notify
if send_request failed.
The pjsip_dlg_send_request deletes the message on error by itself.
It seems this patch fixes next issues:
ASTERISK-26199
ASTERISK-26166
ASTERISK-26174
Change-Id: I8b05917c93d993f95d604c042ace5f1a5500f59a
Diffstat (limited to 'res/res_pjsip_pubsub.c')
-rw-r--r-- | res/res_pjsip_pubsub.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/res/res_pjsip_pubsub.c b/res/res_pjsip_pubsub.c index f0e921cb8..3ac3f342f 100644 --- a/res/res_pjsip_pubsub.c +++ b/res/res_pjsip_pubsub.c @@ -2271,7 +2271,7 @@ static int send_notify(struct sip_subscription_tree *sub_tree, unsigned int forc } if (sip_subscription_send_request(sub_tree, tdata)) { - pjsip_tx_data_dec_ref(tdata); + /* do not call pjsip_tx_data_dec_ref(tdata). The pjsip_dlg_send_request deletes the message on error */ return -1; } |