]> git.ipfire.org Git - thirdparty/asterisk.git/commit
dnsmgr.c: dnsmgr_refresh() incorrectly flags change with DNS round-robin
authorAllan Nathanson <42244061+Allan-N@users.noreply.github.com>
Sun, 29 Sep 2024 22:45:51 +0000 (18:45 -0400)
committerAllan Nathanson <42244061+Allan-N@users.noreply.github.com>
Thu, 10 Oct 2024 15:34:22 +0000 (15:34 +0000)
commit99221c81edc572d0a1a35c29e98267fa71ced720
tree84f3fc34381d5191a60e0de20c578bc49fc91e64
parentf9d5d659c6fe57e6ab329060a843f04ec9715394
dnsmgr.c: dnsmgr_refresh() incorrectly flags change with DNS round-robin

The dnsmgr_refresh() function checks to see if the IP address associated
with a name/service has changed. The gotcha is that the ast_get_ip_or_srv()
function only returns the first IP address returned by the DNS query. If
there are multiple IPs associated with the name and the returned order is
not consistent (e.g. with DNS round-robin) then the other IP addresses are
not included in the comparison and the entry is flagged as changed even
though the IP is still valid.

Updated the code to check all IP addresses and flag a change only if the
original IP is no longer valid.

Resolves: #924
include/asterisk/acl.h
main/acl.c
main/dnsmgr.c