[asterisk-dev] [Code Review]: If DNS is unavailable at start-up, update peer address so registration works when DNS comes back

[Terry Wilson]

> On Aug. 2, 2011, 10:26 a.m., kobaz wrote:
> > I can't reproduce the bug.  If I start up asterisk with my dns server blocking requests, it cannot look up the hostname and cannot register.  As soon as I unblock requests, asterisk will find the hostname and successfully register.
> > 
> > This is without the patch.  I'm trying to see what situation this fixes.

You have to be registering to a peer, not just a hostname.


- Terry


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


On Aug. 2, 2011, 10:19 a.m., Terry Wilson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1335/
> -----------------------------------------------------------
> 
> (Updated Aug. 2, 2011, 10:19 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> If Asterisk starts and no DNS is available, outbound registrations will fail indefinitely. This patch copies the address from the sip_registry struct, which will be updated, to the peer->addr when necessary.
> 
> If dnsmgr is enabled, the registration fails without the patch because even though the address on the registry is updated via dnsmgr, the address is just copied on the first try. Since we use ast_sockaddr_copy, dnsmgr can't update the address that is copied to the sip_pvt or peers.
> 
> 
> This addresses bug ASTERISK-18000.
>     https://issues.asterisk.org/jira/browse/ASTERISK-18000
> 
> 
> Diffs
> -----
> 
>   /branches/1.8/channels/chan_sip.c 330109 
> 
> Diff: https://reviewboard.asterisk.org/r/1335/diff
> 
> 
> Testing
> -------
> 
> Disable DNS server, load asterisk, watch registration fail. Start DNS server, watch registration succeed. Both with and without dnsmgr tested.
> 
> 
> Thanks,
> 
> Terry
> 
>

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