[asterisk-dev] [Code Review] Fix DNS resolution for IPv4 when UDP transport is IPv6 only.

Terry Wilson reviewboard at asterisk.org
Thu Mar 8 14:42:58 CST 2012


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

Ship it!


Just a nitpick, otherwise looks good to me.


/branches/1.8/channels/chan_sip.c
<https://reviewboard.asterisk.org/r/1667/#comment10265>

    Should transport be enum sip_transport_type?



/branches/1.8/channels/chan_sip.c
<https://reviewboard.asterisk.org/r/1667/#comment10583>

    enum sip_transport_type


- Terry


On Jan. 11, 2012, 10:38 a.m., Joshua Colp wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1667/
> -----------------------------------------------------------
> 
> (Updated Jan. 11, 2012, 10:38 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> This changes the address family filtering when doing DNS resolution to take the transport type into account. By taking the transport type into account the correct bound address can be used to return the proper address family. Previously the UDP address was always used so if it were set to IPv6 only then IPv4 resolution was not possible on the TCP or TLS transport.
> 
> 
> Diffs
> -----
> 
>   /branches/1.8/channels/chan_sip.c 350362 
> 
> Diff: https://reviewboard.asterisk.org/r/1667/diff
> 
> 
> Testing
> -------
> 
> Confirmed registrations and calls are now possible when UDP transport is IPv6 only and TCP/TLS transport is IPv4 only.
> 
> 
> Thanks,
> 
> Joshua
> 
>

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


More information about the asterisk-dev mailing list