[Asterisk-code-review] chan sip.c: Start ICE negotiation when response is sent or r... (asterisk[11])

[Joshua Colp]

Joshua Colp has uploaded a new patch set (#4).

Change subject: chan_sip.c: Start ICE negotiation when response is sent or received.
......................................................................

chan_sip.c: Start ICE negotiation when response is sent or received.

The current logic for ICE negotiation starts it
when receiving an SDP with ICE candidates. This is
incorrect as browsers only start ICE negotiation when
they receive SDP. Starting ICE negotiation early
would result in negotiation failure and ultimately
no audio.

This change makes it so ICE negotiation is only started
when a response with SDP is received or when a response
with SDP is sent.

ASTERISK-24146

Change-Id: Ife7fd7d65026393bd4b09805aa788ea358ec3c35
---
M channels/chan_sip.c
1 file changed, 5 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/79/679/4
-- 
To view, visit https://gerrit.asterisk.org/679
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ife7fd7d65026393bd4b09805aa788ea358ec3c35
Gerrit-PatchSet: 4
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Eugene Voityuk <eugene at thirdlane.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>