[Asterisk-code-review] chan_sip: voice frames are no longer transmitted after emitting a COLP (asterisk[16])

George Joseph asteriskteam at digium.com
Tue Dec 31 08:38:40 CST 2019


George Joseph has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/13525 )

Change subject: chan_sip: voice frames are no longer transmitted after emitting a COLP
......................................................................

chan_sip: voice frames are no longer transmitted after emitting a COLP

The SIP transaction state was reset when emitting an UPDATE or a re-INVITE
related to a COLP, preventing RTP packets to be emitted.

ASTERISK-28647

Change-Id: Ie7a30fa7a97f711e7ba6cc17f221a0993d48bd8b
---
M channels/chan_sip.c
1 file changed, 0 insertions(+), 1 deletion(-)

Approvals:
  George Joseph: Looks good to me, approved; Approved for Submit



diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index 04b02c9..580954a 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -15756,7 +15756,6 @@
 			initialize_initreq(p, &req);
 			p->lastinvite = p->ocseq;
 			ast_set_flag(&p->flags[0], SIP_OUTGOING);
-			p->invitestate = INV_CALLING;
 			send_request(p, &req, XMIT_CRITICAL, p->ocseq);
 		} else if ((is_method_allowed(&p->allowed_methods, SIP_UPDATE)) && (!ast_strlen_zero(p->okcontacturi))) {
 			reqprep(&req, p, SIP_UPDATE, 0, 1);

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/13525
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: Ie7a30fa7a97f711e7ba6cc17f221a0993d48bd8b
Gerrit-Change-Number: 13525
Gerrit-PatchSet: 1
Gerrit-Owner: Jean Aunis - Prescom <jean.aunis at prescom.fr>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20191231/511d8b77/attachment-0001.html>


More information about the asterisk-code-review mailing list