[asterisk-dev] [Code Review] better SDP parsing algorithm for trunk

Matthew Nicholson mnicholson at digium.com
Wed Oct 28 15:22:57 CDT 2009


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



trunk/channels/chan_sip.c
<https://reviewboard.asterisk.org/r/414/#comment2830>

    This function can probably benefit from your new parsing algorithms too.


- Matthew


On 2009-10-26 04:47:48, frawd wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/414/
> -----------------------------------------------------------
> 
> (Updated 2009-10-26 04:47:48)
> 
> 
> Review request for Asterisk Developers, Olle E Johansson and Matthew Nicholson.
> 
> 
> Summary
> -------
> 
> See https://reviewboard.asterisk.org/r/385/ for the initial 1.4 patch.
> 
> Changes from 1.4 patch:
> - Port to trunk API and its new things (SIREN, T.38, ...)
> - Added function "process_sdp_o" for version checking
> - Added function "process_sdp_a_text" for T.140 and RED stuff
> 
> This patch cleans up asterisk's SDP parsing algorithm, resolving a few bugs including #14994. It does the parsing line by line, making a distinction between session-level and media-specific parameters. It also optimizes the parsing adding functions for audio/video/image specific scanning.
> I added debug information for a better understanding on how the parsing is actually done (shows each SDP line parsed with OK or UNSUPPORTED).
> 
> 
> This addresses bug 0014994.
>     https://issues.asterisk.org/view.php?id=0014994
> 
> 
> Diffs
> -----
> 
>   trunk/channels/chan_sip.c 225766 
> 
> Diff: https://reviewboard.asterisk.org/r/414/diff
> 
> 
> Testing
> -------
> 
> It compiles OK.
> 
> When I can figure out how to load modules in trunk, I can do some basic testing. But I cannot test that in production (trunk...).
> For now I got dozens of:
> "Module '*****' could not be loaded."
> 
> 
> Thanks,
> 
> frawd
> 
>




More information about the asterisk-dev mailing list