[asterisk-dev] [svn-commits] jrose: branch 11 r399457 - in /branches/11: ./ channels/chan_sip.c

[Jonathan Rose]

On Fri, Sep 20, 2013 at 12:26 AM, Olle E. Johansson <oej at edvina.net> wrote:

>
> 19 sep 2013 kl. 18:45 skrev SVN commits to the Digium repositories <
> svn-commits at lists.digium.com>:
>
> Prior to this patch, Asterisk would incorrectly use the previous endpoint
> addresses in SDP in spite of providing its own port. T38 is never meant to
> be done through directmedia and Asterisk should always be in the media path
> for these streams.
>
>
> Just being curious: Why was T.38 never meant to use directmedia? Any
> technichal reasons in the protocol?
> Or just something missing in our code?
>
> /O
>
> --
> _____________________________________________________________________
> -- Bandwidth and Colocation Provided by http://www.api-digital.com --
>
> asterisk-dev mailing list
> To UNSUBSCRIBE or update options visit:
>    http://lists.digium.com/mailman/listinfo/asterisk-dev
>

In all honesty I'm not entirely sure why this is the case, I was simply
informed that directmedia is intended to be suspended for T.38 (by Josh
Colp and Mark Michelson) and noticed that Asterisk was issuing other
contact information like ports used and such in the image SDP without
writing a new c= section due to what appeared to be a case of backwards
string comparison logic.

-- 
Jonathan R. Rose
Digium, Inc. | Software Engineer
445 Jan Davis Drive NW - Huntsville, AL 35806 - US
direct +1 256 428 6139

Check us out at: http://digium.com & http://asterisk.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130920/774ae8ed/attachment-0001.htm>