<span style="font-family: Arial, Helvetica, sans-serif; font-size: 10pt">Could this also be causing some timing issues with in-call DTMF? The issues did not appear to be in 1.6.2.11 but have showed up in 1.6.2.12-rc1?<br />
<br />
Bryant<br />
<div id="divSignature"></div>
<br />
<span style="font-family: tahoma,arial,sans-serif; font-size: 10pt;"><hr width="100%" size="2" align="center" />
<b>From</b>: "Russell Bryant" <russell@digium.com><br />
<b>Sent</b>: Wednesday, September 01, 2010 2:41 PM<br />
<b>To</b>: "Asterisk Developers" <asterisk-dev@lists.digium.com>, "Russell Bryant" <russell@digium.com>, "Terry Wilson" <twilson@digium.com><br />
<b>Subject</b>: Re: [asterisk-dev] [Code Review] Fix SRTP for changing SSRC and multiple a=crypto SDP lines</span><br />
<br />
<br />
-----------------------------------------------------------<br />
This is an automatically generated e-mail. To reply, visit:<br />
https://reviewboard.asterisk.org/r/878/#review2651<br />
-----------------------------------------------------------<br />
<br />
Ship it!<br />
<br />
<br />
- Russell<br />
<br />
<br />
On 2010-08-26 01:29:16, Terry Wilson wrote:<br />
> <br />
> -----------------------------------------------------------<br />
> This is an automatically generated e-mail. To reply, visit:<br />
> https://reviewboard.asterisk.org/r/878/<br />
> -----------------------------------------------------------<br />
> <br />
> (Updated 2010-08-26 01:29:16)<br />
> <br />
> <br />
> Review request for Asterisk Developers.<br />
> <br />
> <br />
> Summary<br />
> -------<br />
> <br />
> Adding code to Asterisk that changed the SSRC during bridges and masquerades broke SRTP functionality. Also broken was handling the situation where an incoming INVITE had more than one crypto offer. This patch caches the SRTP policies the we use so that we can change the ssrc and inform libsrtp of the new streams. It also uses the first acceptable a=crypto line from the incoming INVITE.<br />
> <br />
> <br />
> This addresses bug 17563.<br />
> https://issues.asterisk.org/view.php?id=17563<br />
> <br />
> <br />
> Diffs<br />
> -----<br />
> <br />
> /branches/1.8/channels/chan_sip.c 283320 <br />
> /branches/1.8/include/asterisk/res_srtp.h 283320 <br />
> /branches/1.8/main/rtp_engine.c 283320 <br />
> /branches/1.8/res/res_rtp_asterisk.c 283320 <br />
> /branches/1.8/res/res_srtp.c 283320 <br />
> <br />
> Diff: https://reviewboard.asterisk.org/r/878/diff<br />
> <br />
> <br />
> Testing<br />
> -------<br />
> <br />
> I tested by 1) Setting up Polycom phones to send two a=crypto lines 2) Changing SIP hold/unhold to call the rtp change_source callback to verify that changing source worked 3) Doing transfers that would cause a masquerade and therefore a source change 4) astobj2 show stats to verify that there were no object leaks with the above tests.<br />
> <br />
> <br />
> Thanks,<br />
> <br />
> Terry<br />
> <br />
><br />
<br />
<br />
-- <br />
_____________________________________________________________________<br />
-- Bandwidth and Colocation Provided by http://www.api-digital.com --<br />
<br />
asterisk-dev mailing list<br />
To UNSUBSCRIBE or update options visit:<br />
http://lists.digium.com/mailman/listinfo/asterisk-dev<br />
<br /></span>