[asterisk-dev] [Code Review] 3275: res_rtp_asterisk: Fix the one way audio problems when resuming held calls with ICE

Jonathan Rose reviewboard at asterisk.org
Thu Feb 27 15:53:15 CST 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3275/
-----------------------------------------------------------

(Updated Feb. 27, 2014, 3:53 p.m.)


Review request for Asterisk Developers, Joshua Colp, Kevin Harwell, and Matt Jordan.


Changes
-------

Hit all the findings!

Also spotted another ref leak in the only other ao2_iterator in this file.


Bugs: ASTERISK-22911
    https://issues.asterisk.org/jira/browse/ASTERISK-22911


Repository: Asterisk


Description
-------

This patch provides a fix for the hold problem by doing the following:

Once an ICE session is marked as started, we start adding any new remote candidates into a separate list until we get another attempt to start the ICE session.
Once a call to start the ice session is made, instead of immediately quitting if the session is already started, we check for a difference in the two candidates lists.  If the lists are identical, we wipe out the new list and keep the old one and just quit then going on with the current ICE session. If the lists are changed, we toss the old list and adopt the new one and restart the ICE session.


Diffs (updated)
-----

  /branches/11/res/res_rtp_asterisk.c 409132 

Diff: https://reviewboard.asterisk.org/r/3275/diff/


Testing
-------

SIPML client to Asterisk to Desk Phone
SIPML calls desk phone
audio test, got two way audio
SIPML holds call
SIPML resumes call
audio test, got two way audio (previously this would cause one way audio from the SIPML client to the desk phone)


Thanks,

Jonathan Rose

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140227/710d6da0/attachment.html>


More information about the asterisk-dev mailing list