]> git.ipfire.org Git - thirdparty/bind9.git/commit
Fix MX checks for dynamic updates
authorMichał Kępień <michal@isc.org>
Mon, 26 Feb 2018 11:30:48 +0000 (12:30 +0100)
committerEvan Hunt <each@isc.org>
Mon, 26 Feb 2018 23:19:03 +0000 (15:19 -0800)
commitb33ce857302d939e1aa2e6ca480b65a3410f5ff1
tree9d625e91b560db494a0bcd655e74616176909173
parent6069b3d06c0e4330042c0b861c55ea43f2266b51
Fix MX checks for dynamic updates

The check_mx() function in lib/ns/update.c incorrectly tests whether the
DNS_RDATA_CHECKMX/DNS_RDATA_CHECKMXFAIL flags are set for each applied
MX record update as these flags are never set in code paths related to
dynamic updates; they can only be set when loading a zone from a master
file (DNS_ZONEOPT_CHECKMX -> DNS_MASTER_CHECKMX -> DNS_RDATA_CHECKMX).
This flaw allows MX records containing IP addresses to be added to a
zone even when "check-mx fail;" is used.

Ensure correct behavior by modifying the relevant tests in check_mx() so
that they use DNS_ZONEOPT_CHECKMX/DNS_ZONEOPT_CHECKMXFAIL instead.

(cherry picked from commit 857a40c87bb80669ecc5cf1c51301d967e7aba53)
bin/named/update.c
bin/tests/system/nsupdate/ns1/named.conf.in
bin/tests/system/nsupdate/tests.sh