[Asterisk-code-review] res_pjsip_sdp_rtp: Don't wait for ICE if not negotiated (asterisk[16])

Torrey Searle asteriskteam at digium.com
Mon Mar 9 07:31:43 CDT 2020


Hello Joshua Colp, Friendly Automation, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/c/asterisk/+/13857

to look at the new patch set (#3).

Change subject: res_pjsip_sdp_rtp: Don't wait for ICE if not negotiated
......................................................................

res_pjsip_sdp_rtp: Don't wait for ICE if not negotiated

If ICE support is enabled but not negotiated, the rtp->ice structure is
not being destroyed. This leads to Asterisk waiting for ICE to complete
instead of immediately starting the DTLS handshake, resulting in the
call leg having no RTP.

ASTERISK-28769 #close

Change-Id: I17c137546dc9ecfb9583c24dcf4c2ced8bbd7a27
---
M res/res_pjsip_sdp_rtp.c
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/57/13857/3
-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/13857
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: I17c137546dc9ecfb9583c24dcf4c2ced8bbd7a27
Gerrit-Change-Number: 13857
Gerrit-PatchSet: 3
Gerrit-Owner: Torrey Searle <tsearle at gmail.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20200309/307204b3/attachment-0001.html>


More information about the asterisk-code-review mailing list