]> git.ipfire.org Git - thirdparty/bind9.git/commit
Do not recheck DNS_ZONEFLG_LOADPENDING in zone_asyncload()
authorMichał Kępień <michal@isc.org>
Thu, 15 Feb 2018 19:31:54 +0000 (20:31 +0100)
committerMichał Kępień <michal@isc.org>
Thu, 15 Feb 2018 19:31:54 +0000 (20:31 +0100)
commitf5079bb87767e433b0d2a73291c846238781f793
treeb76f229aad64d12425eaf18fb5aa37a93fc37b1a
parentb9e9361c7b67c9bee700aed46f91713ba09a9432
Do not recheck DNS_ZONEFLG_LOADPENDING in zone_asyncload()

Remove a block of code which dates back to commit 8a2ab2b9203, when
dns_zone_asyncload() did not yet check DNS_ZONEFLG_LOADPENDING.
Currently, no race in accessing DNS_ZONEFLG_LOADPENDING is possible any
more, because:

  - dns_zone_asyncload() is still the only function which may queue
    zone_asyncload(),

  - dns_zone_asyncload() accesses DNS_ZONEFLG_LOADPENDING under a lock
    (and potentially queues an event under the same lock),

  - DNS_ZONEFLG_LOADPENDING is not cleared until the load actually
    completes.

Thus, the rechecking code can be safely removed from zone_asyncload().

Note that this also brings zone_asyncload() to a state in which the
completion callback is always invoked.  This is required to prevent
leaking memory in case something goes wrong in zone_asyncload() and a
zone table the zone belongs to is indefinitely left with a positive
reference count.
lib/dns/zone.c