[asterisk-dev] [Code Review] 3168: PJSIP: Correct address to which ACK is sent in NAT situations

Joshua Colp reviewboard at asterisk.org
Thu Jan 30 14:11:18 CST 2014


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



branches/12/res/res_pjsip_nat.c
<https://reviewboard.asterisk.org/r/3168/#comment20217>

    You can also use pjsip_rdata_get_dlg here to retrieve a dialog, if one is present, and have this work for non-INVITE stuff as well (just in case).


- Joshua Colp


On Jan. 30, 2014, 7:59 p.m., opticron wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/3168/
> -----------------------------------------------------------
> 
> (Updated Jan. 30, 2014, 7:59 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-23106
>     https://issues.asterisk.org/jira/browse/ASTERISK-23106
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> In NAT scenarios where a call is placed to a Grandstream phone, res_pjsip will sometimes send the ACK to a 200 OK to the private address of the device behind the NAT instead of the address of the NAT device. This corrects that behavior by rewriting the address in the Contact header in the incoming 200 OK and the dialog's target address if necessary (since it has already been rewritten to the incorrect private address).
> 
> 
> Diffs
> -----
> 
>   branches/12/res/res_pjsip_nat.c 406916 
> 
> Diff: https://reviewboard.asterisk.org/r/3168/diff/
> 
> 
> Testing
> -------
> 
> Manual testing and a new testsuite test at https://reviewboard.asterisk.org/r/3169/
> 
> 
> Thanks,
> 
> opticron
> 
>

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


More information about the asterisk-dev mailing list