[asterisk-dev] [Code Review] 3658: SIP/SDP fmtp line may include whitespace between attributes

[opticron]

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

Ship it!



/branches/11/channels/chan_sip.c
<https://reviewboard.asterisk.org/r/3658/#comment22463>

    It appears that this will need to be merged into 12/trunk when this patch goes in.


- opticron


On June 20, 2014, 10:02 a.m., Alexander Traud wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3658/
> -----------------------------------------------------------
> 
> (Updated June 20, 2014, 10:02 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-23916
>     https://issues.asterisk.org/jira/browse/ASTERISK-23916
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> In SIP/SDP, the fmtp line is allowed to include whitespace between attributes. RFC 3267 chapter 8.3 (from 2001) includes an example for this. This was not removed in the updated RFC 4867 in 2007. Here, I propose to backport this one-line change into Asterisk 11 LTS.
> 
> This diff is based on (an excerpt of) SVN revision 397526, review 2723, and ASTERISK-21981. The change was tested with Nokia mobile phones, which allow more than one media attribute with the format AMR.
> 
> 
> Diffs
> -----
> 
>   /branches/11/channels/chan_sip.c 416067 
> 
> Diff: https://reviewboard.asterisk.org/r/3658/diff/
> 
> 
> Testing
> -------
> 
> 11.10
> 
> 
> Thanks,
> 
> Alexander Traud
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140624/f18c882c/attachment-0001.html>