]> git.ipfire.org Git - thirdparty/bind9.git/commit
Fix a race between "rndc reconfig" and waiting for a ./DNSKEY fetch to complete
authorMichał Kępień <michal@isc.org>
Tue, 6 Mar 2018 10:33:16 +0000 (11:33 +0100)
committerMichał Kępień <michal@isc.org>
Thu, 8 Mar 2018 12:19:54 +0000 (13:19 +0100)
commit012ca0a27dc2058090d97027076cf09b1bb2474a
tree43c126701b2831719e49c815d87933ca8039ecf9
parent8be9ce2f06f0c98dc760ec28b788d75057341ca5
Fix a race between "rndc reconfig" and waiting for a ./DNSKEY fetch to complete

Calling nextpart() after reconfiguring ns1 is not safe, because the
expected log message may appear in ns5/named.run before nextpart() is
run.  With the TTL for ./DNSKEY set to 20 seconds, ns5 will refresh it
after 10 seconds, by which time wait_for_log() will already have failed.
This results in a false negative.

However, just calling nextpart() before reconfiguring ns1 would
introduce a different problem: if ns5 refreshed ./DNSKEY between these
two steps, the subsequent wait_for_log() call would return immediately
as it would come across the log message about a failure while refreshing
./DNSKEY instead of the expected success.  This in turn would result in
a different false negative as the root key would still be uninitialized
by the time "rndc secroots" is called.

Prevent both kinds of false negatives by:

  - calling nextpart() before reconfiguring ns1, in order to prevent the
    first case described above,

  - looking for a more specific log message, in order to prevent the
    second case described above.

Also look for a more specific log message in the first part of the
relevant check, not to fix any problem, but just to emphasize that a
different fetch result is expected in that case.

With these tweaks in place, if a (failed) ./DNSKEY refresh is scheduled
between nextpart() and reconfiguring ns1, wait_for_log() will just wait
for two more seconds (one "hour"), at which point another refresh
attempt will be made that will succeed.
bin/tests/system/mkeys/tests.sh