Pause the dbiterator before calling dns_db_find

zone.c:integrity_checks() acquires a read lock while iterating the
zone database, and calls zone_check_mx() which acquires another
read lock. If another thread tries to acquire a write lock in the
meantime, it can deadlock. Calling dns_dbiterator_pause() to release
the first read lock prevents this.

diff --git a/lib/dns/zone.c b/lib/dns/zone.c
index 5235e1b610d74edf109aea165e35060cc5c7ed49..7bd8c05c9b6a7c405ffd6c9a4915c21a37a37b3a 100644 (file)
--- a/lib/dns/zone.c
+++ b/lib/dns/zone.c
@@ -3326,6 +3326,8 @@ integrity_checks(dns_zone_t *zone, dns_db_t *db) {
                        goto next;
                }
 
+               dns_dbiterator_pause(dbiterator);
+
                /*
                 * Don't check the NS records at the origin.
                 */