[asterisk-dev] [Code Review] Fix several bugs with SDP parsing and well-formedness of responses

[opticron]

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

(Updated Oct. 12, 2011, 9:17 a.m.)


Review request for Asterisk Developers.


Changes
-------

Fix red blob closely related to the code being modified.


Summary
-------

Fix bug ASTERISK-16558 which dealt with the order of responses to incoming streams defined by SDP.

Fix unreported bug where offering multiple same-type streams would cause Asterisk to reply with an incorrect SDP response missing one or more streams without a proper declination.

Fix bugs related to a single non-audio stream being offered with responses requesting codecs that were not offered in the initial invite along with an additional audio stream that was not in the initial invite.


This addresses bug ASTERISK-16558.
    https://issues.asterisk.org/jira/browse/ASTERISK-16558


Diffs (updated)
-----

  trunk/channels/chan_sip.c 340161 
  trunk/channels/sip/include/sip.h 340161 

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


Testing
-------

Ran this through a large portion of the external testsuite, but it needs quite a bit more testing before commit.


Thanks,

opticron

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20111012/3c9b8511/attachment.htm>