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

Terry Wilson reviewboard at asterisk.org
Tue Aug 2 10:07:55 CDT 2011


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

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/0868b5cf/attachment-0001.htm>


More information about the asterisk-dev mailing list