[asterisk-dev] [Code Review] Properly route responses according to the Via headers in the request

[Matthew Nicholson]

> On 2010-12-17 09:51:48, Mark Michelson wrote:
> > /branches/1.4/channels/chan_sip.c, lines 4998-5000
> > <https://reviewboard.asterisk.org/r/1059/diff/2/?file=14712#file14712line4998>
> >
> >     This shouldn't be necessary since strsep() will return the position past the end of a sequence of the tokens you pass to it.
> >     
> >     To give an example, if you have a string str set to "Mark,,',,''',is cool" and call strsep(&str, ",'") once then you'll be returned a pointer to the 'i' in "is cool"

Ok, I'll make these adjustments.  These changes probably need to be made in the other branches that use this function too.


- Matthew


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


On 2010-12-17 09:27:27, Matthew Nicholson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1059/
> -----------------------------------------------------------
> 
> (Updated 2010-12-17 09:27:27)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> This patch makes asterisk respect the Via headers in a request when responding to it. This is necessary in the even that a stateless proxy is in between asterisk and the requester. Without this patch, the response is simply routed back to the address we received the initial request from (unless nat=yes is set).
> 
> 
> Diffs
> -----
> 
>   /branches/1.4/channels/chan_sip.c 294163 
> 
> Diff: https://reviewboard.asterisk.org/r/1059/diff
> 
> 
> Testing
> -------
> 
> Briefly tested using openser as a stateless proxy and another asterisk machine as the requester.  I sent and invite, then some INFO DTMF messages.  Without the patch, our asterisk machine sends all responses to the INFO requests to the proxy, with the patch they are properly routed to the requesting asterisk machine.  I also briefly tested with openser configured to use the Record-Route header as a stateful proxy.
> 
> 
> Thanks,
> 
> Matthew
> 
>

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