mir/bind
2
0
Fork 0
mirror of https://gitlab.isc.org/isc-projects/bind9 synced 2025-08-30 22:15:20 +00:00
Code Issues Releases Wiki Activity

3314. [bug] The masters list could be updated while refesh_callback

Browse Source 
and stub_callback were using it. [RT #26732]
This commit is contained in:
Mark Andrews
2012-04-26 12:58:39 +10:00
parent 393fd55d91
commit 1a04af5311
2 changed files with 47 additions and 55 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
CHANGES
View File

@@ -1,3 +1,6 @@
3314. [bug] The masters list could be updated while refesh_callback
and stub_callback were using it. [RT #26732]
3313. [protocol] Add TLSA record type. [RT #28989] 3313. [protocol] Add TLSA record type. [RT #28989]
3312. [bug] named-checkconf didn't detect a bad dns64 clients acl. 3312. [bug] named-checkconf didn't detect a bad dns64 clients acl.

99
lib/dns/zone.c
View File

@@ -458,6 +458,7 @@ struct dns_zonemgr {
isc_ratelimiter_t * rl; isc_ratelimiter_t * rl;
isc_rwlock_t rwlock; isc_rwlock_t rwlock;
isc_mutex_t iolock; isc_mutex_t iolock;
isc_rwlock_t urlock;
/* Locked by rwlock. */ /* Locked by rwlock. */
dns_zonelist_t zones; dns_zonelist_t zones;
@@ -475,7 +476,7 @@ struct dns_zonemgr {
dns_iolist_t high; dns_iolist_t high;
dns_iolist_t low; dns_iolist_t low;
/* Locked by rwlock. */ /* Locked by urlock. */
/* LRU cache */ /* LRU cache */
struct dns_unreachable unreachable[UNREACH_CHACHE_SIZE]; struct dns_unreachable unreachable[UNREACH_CHACHE_SIZE];
}; };
@@ -3014,7 +3015,8 @@ check_nsec3param(dns_zone_t *zone, dns_db_t *db) {
*/ */
static void static void
set_refreshkeytimer(dns_zone_t *zone, dns_rdata_keydata_t *key, set_refreshkeytimer(dns_zone_t *zone, dns_rdata_keydata_t *key,
isc_stdtime_t now) { isc_stdtime_t now)
{
const char me[] = "set_refreshkeytimer"; const char me[] = "set_refreshkeytimer";
isc_stdtime_t then; isc_stdtime_t then;
isc_time_t timenow, timethen; isc_time_t timenow, timethen;
@@ -9889,6 +9891,8 @@ stub_callback(isc_task_t *task, isc_event_t *event) {
TIME_NOW(&now); TIME_NOW(&now);
LOCK_ZONE(zone);
if (DNS_ZONE_FLAG(zone, DNS_ZONEFLG_EXITING)) { if (DNS_ZONE_FLAG(zone, DNS_ZONEFLG_EXITING)) {
zone_debuglog(zone, me, 1, "exiting"); zone_debuglog(zone, me, 1, "exiting");
exiting = ISC_TRUE; exiting = ISC_TRUE;
@@ -9901,9 +9905,7 @@ stub_callback(isc_task_t *task, isc_event_t *event) {
if (revent->result != ISC_R_SUCCESS) { if (revent->result != ISC_R_SUCCESS) {
if (revent->result == ISC_R_TIMEDOUT && if (revent->result == ISC_R_TIMEDOUT &&
!DNS_ZONE_FLAG(zone, DNS_ZONEFLG_NOEDNS)) { !DNS_ZONE_FLAG(zone, DNS_ZONEFLG_NOEDNS)) {
LOCK_ZONE(zone);
DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_NOEDNS); DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_NOEDNS);
UNLOCK_ZONE(zone);
dns_zone_log(zone, ISC_LOG_DEBUG(1), dns_zone_log(zone, ISC_LOG_DEBUG(1),
"refreshing stub: timeout retrying " "refreshing stub: timeout retrying "
" without EDNS master %s (source %s)", " without EDNS master %s (source %s)",
@@ -9945,9 +9947,7 @@ stub_callback(isc_task_t *task, isc_event_t *event) {
"refreshing stub: rcode (%.*s) retrying " "refreshing stub: rcode (%.*s) retrying "
"without EDNS master %s (source %s)", "without EDNS master %s (source %s)",
(int)rb.used, rcode, master, source); (int)rb.used, rcode, master, source);
LOCK_ZONE(zone);
DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_NOEDNS); DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_NOEDNS);
UNLOCK_ZONE(zone);
goto same_master; goto same_master;
} }
@@ -9969,9 +9969,7 @@ stub_callback(isc_task_t *task, isc_event_t *event) {
master, source); master, source);
goto next_master; goto next_master;
} }
LOCK_ZONE(zone);
DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_USEVC); DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_USEVC);
UNLOCK_ZONE(zone);
goto same_master; goto same_master;
} }
@@ -10026,21 +10024,17 @@ stub_callback(isc_task_t *task, isc_event_t *event) {
ZONEDB_UNLOCK(&zone->dblock, isc_rwlocktype_write); ZONEDB_UNLOCK(&zone->dblock, isc_rwlocktype_write);
dns_db_detach(&stub->db); dns_db_detach(&stub->db);
if (zone->masterfile != NULL) { if (zone->masterfile != NULL)
dns_zone_dump(zone); zone_needdump(zone, 0);
TIME_NOW(&zone->loadtime);
}
dns_message_destroy(&msg); dns_message_destroy(&msg);
isc_event_free(&event); isc_event_free(&event);
LOCK_ZONE(zone);
dns_request_destroy(&zone->request); dns_request_destroy(&zone->request);
DNS_ZONE_CLRFLAG(zone, DNS_ZONEFLG_REFRESH); DNS_ZONE_CLRFLAG(zone, DNS_ZONEFLG_REFRESH);
DNS_ZONE_JITTER_ADD(&now, zone->refresh, &zone->refreshtime); DNS_ZONE_JITTER_ADD(&now, zone->refresh, &zone->refreshtime);
isc_interval_set(&i, zone->expire, 0); isc_interval_set(&i, zone->expire, 0);
DNS_ZONE_TIME_ADD(&now, zone->expire, &zone->expiretime); DNS_ZONE_TIME_ADD(&now, zone->expire, &zone->expiretime);
zone_settimer(zone, &now); zone_settimer(zone, &now);
UNLOCK_ZONE(zone);
goto free_stub; goto free_stub;
next_master: next_master:
@@ -10051,7 +10045,6 @@ stub_callback(isc_task_t *task, isc_event_t *event) {
if (msg != NULL) if (msg != NULL)
dns_message_destroy(&msg); dns_message_destroy(&msg);
isc_event_free(&event); isc_event_free(&event);
LOCK_ZONE(zone);
dns_request_destroy(&zone->request); dns_request_destroy(&zone->request);
/* /*
* Skip to next failed / untried master. * Skip to next failed / untried master.
@@ -10089,25 +10082,23 @@ stub_callback(isc_task_t *task, isc_event_t *event) {
DNS_ZONE_CLRFLAG(zone, DNS_ZONEFLG_REFRESH); DNS_ZONE_CLRFLAG(zone, DNS_ZONEFLG_REFRESH);
zone_settimer(zone, &now); zone_settimer(zone, &now);
UNLOCK_ZONE(zone);
goto free_stub; goto free_stub;
} }
} }
queue_soa_query(zone); queue_soa_query(zone);
UNLOCK_ZONE(zone);
goto free_stub; goto free_stub;
same_master: same_master:
if (msg != NULL) if (msg != NULL)
dns_message_destroy(&msg); dns_message_destroy(&msg);
isc_event_free(&event); isc_event_free(&event);
LOCK_ZONE(zone);
dns_request_destroy(&zone->request); dns_request_destroy(&zone->request);
UNLOCK_ZONE(zone);
ns_query(zone, NULL, stub); ns_query(zone, NULL, stub);
UNLOCK_ZONE(zone);
goto done; goto done;
free_stub: free_stub:
UNLOCK_ZONE(zone);
stub->magic = 0; stub->magic = 0;
dns_zone_idetach(&stub->zone); dns_zone_idetach(&stub->zone);
INSIST(stub->db == NULL); INSIST(stub->db == NULL);
@@ -10138,6 +10129,7 @@ refresh_callback(isc_task_t *task, isc_event_t *event) {
isc_result_t result; isc_result_t result;
isc_uint32_t serial, oldserial = 0; isc_uint32_t serial, oldserial = 0;
unsigned int j; unsigned int j;
isc_boolean_t do_queue_xfrin = ISC_FALSE;
zone = revent->ev_arg; zone = revent->ev_arg;
INSIST(DNS_ZONE_VALID(zone)); INSIST(DNS_ZONE_VALID(zone));
@@ -10146,6 +10138,10 @@ refresh_callback(isc_task_t *task, isc_event_t *event) {
ENTER; ENTER;
TIME_NOW(&now);
LOCK_ZONE(zone);
/* /*
* if timeout log and next master; * if timeout log and next master;
*/ */
@@ -10153,14 +10149,10 @@ refresh_callback(isc_task_t *task, isc_event_t *event) {
isc_sockaddr_format(&zone->masteraddr, master, sizeof(master)); isc_sockaddr_format(&zone->masteraddr, master, sizeof(master));
isc_sockaddr_format(&zone->sourceaddr, source, sizeof(source)); isc_sockaddr_format(&zone->sourceaddr, source, sizeof(source));
TIME_NOW(&now);
if (revent->result != ISC_R_SUCCESS) { if (revent->result != ISC_R_SUCCESS) {
if (revent->result == ISC_R_TIMEDOUT && if (revent->result == ISC_R_TIMEDOUT &&
!DNS_ZONE_FLAG(zone, DNS_ZONEFLG_NOEDNS)) { !DNS_ZONE_FLAG(zone, DNS_ZONEFLG_NOEDNS)) {
LOCK_ZONE(zone);
DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_NOEDNS); DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_NOEDNS);
UNLOCK_ZONE(zone);
dns_zone_log(zone, ISC_LOG_DEBUG(1), dns_zone_log(zone, ISC_LOG_DEBUG(1),
"refresh: timeout retrying without EDNS " "refresh: timeout retrying without EDNS "
"master %s (source %s)", master, source); "master %s (source %s)", master, source);
@@ -10181,10 +10173,8 @@ refresh_callback(isc_task_t *task, isc_event_t *event) {
&zone->sourceaddr, &zone->sourceaddr,
&now)) &now))
{ {
LOCK_ZONE(zone);
DNS_ZONE_SETFLAG(zone, DNS_ZONE_SETFLAG(zone,
DNS_ZONEFLG_SOABEFOREAXFR); DNS_ZONEFLG_SOABEFOREAXFR);
UNLOCK_ZONE(zone);
goto tcp_transfer; goto tcp_transfer;
} }
dns_zone_log(zone, ISC_LOG_DEBUG(1), dns_zone_log(zone, ISC_LOG_DEBUG(1),
@@ -10231,9 +10221,7 @@ refresh_callback(isc_task_t *task, isc_event_t *event) {
"refresh: rcode (%.*s) retrying without " "refresh: rcode (%.*s) retrying without "
"EDNS master %s (source %s)", "EDNS master %s (source %s)",
(int)rb.used, rcode, master, source); (int)rb.used, rcode, master, source);
LOCK_ZONE(zone);
DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_NOEDNS); DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_NOEDNS);
UNLOCK_ZONE(zone);
goto same_master; goto same_master;
} }
dns_zone_log(zone, ISC_LOG_INFO, dns_zone_log(zone, ISC_LOG_INFO,
@@ -10261,9 +10249,7 @@ refresh_callback(isc_task_t *task, isc_event_t *event) {
"initiating TCP zone xfer " "initiating TCP zone xfer "
"for master %s (source %s)", "for master %s (source %s)",
master, source); master, source);
LOCK_ZONE(zone);
DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_SOABEFOREAXFR); DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_SOABEFOREAXFR);
UNLOCK_ZONE(zone);
goto tcp_transfer; goto tcp_transfer;
} else { } else {
INSIST(zone->type == dns_zone_stub); INSIST(zone->type == dns_zone_stub);
@@ -10274,9 +10260,7 @@ refresh_callback(isc_task_t *task, isc_event_t *event) {
master, source); master, source);
goto next_master; goto next_master;
} }
LOCK_ZONE(zone);
DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_USEVC); DNS_ZONE_SETFLAG(zone, DNS_ZONEFLG_USEVC);
UNLOCK_ZONE(zone);
goto same_master; goto same_master;
} }
} }
@@ -10337,6 +10321,7 @@ refresh_callback(isc_task_t *task, isc_event_t *event) {
soacnt, master, source); soacnt, master, source);
goto next_master; goto next_master;
} }
/* /*
* Extract serial * Extract serial
*/ */
@@ -10364,7 +10349,9 @@ refresh_callback(isc_task_t *task, isc_event_t *event) {
serial = soa.serial; serial = soa.serial;
if (DNS_ZONE_FLAG(zone, DNS_ZONEFLG_LOADED)) { if (DNS_ZONE_FLAG(zone, DNS_ZONEFLG_LOADED)) {
result = dns_zone_getserial2(zone, &oldserial); result = zone_get_from_db(zone, zone->db, NULL, NULL,
&oldserial, NULL, NULL, NULL, NULL,
NULL);
RUNTIME_CHECK(result == ISC_R_SUCCESS); RUNTIME_CHECK(result == ISC_R_SUCCESS);
zone_debuglog(zone, me, 1, "serial: new %u, old %u", zone_debuglog(zone, me, 1, "serial: new %u, old %u",
serial, oldserial); serial, oldserial);
@@ -10389,12 +10376,10 @@ refresh_callback(isc_task_t *task, isc_event_t *event) {
} }
tcp_transfer: tcp_transfer:
isc_event_free(&event); isc_event_free(&event);
LOCK_ZONE(zone);
dns_request_destroy(&zone->request); dns_request_destroy(&zone->request);
UNLOCK_ZONE(zone);
if (zone->type == dns_zone_slave || if (zone->type == dns_zone_slave ||
zone->type == dns_zone_redirect) { zone->type == dns_zone_redirect) {
queue_xfrin(zone); do_queue_xfrin = TRUE;
} else { } else {
INSIST(zone->type == dns_zone_stub); INSIST(zone->type == dns_zone_stub);
ns_query(zone, rdataset, NULL); ns_query(zone, rdataset, NULL);
@@ -10416,9 +10401,7 @@ refresh_callback(isc_task_t *task, isc_event_t *event) {
&now); &now);
/* Someone removed the file from underneath us! */ /* Someone removed the file from underneath us! */
if (result == ISC_R_FILENOTFOUND) { if (result == ISC_R_FILENOTFOUND) {
LOCK_ZONE(zone);
zone_needdump(zone, DNS_DUMP_DELAY); zone_needdump(zone, DNS_DUMP_DELAY);
UNLOCK_ZONE(zone);
} else if (result != ISC_R_SUCCESS) } else if (result != ISC_R_SUCCESS)
dns_zone_log(zone, ISC_LOG_ERROR, dns_zone_log(zone, ISC_LOG_ERROR,
"refresh: could not set file " "refresh: could not set file "
@@ -10448,7 +10431,6 @@ refresh_callback(isc_task_t *task, isc_event_t *event) {
if (msg != NULL) if (msg != NULL)
dns_message_destroy(&msg); dns_message_destroy(&msg);
isc_event_free(&event); isc_event_free(&event);
LOCK_ZONE(zone);
dns_request_destroy(&zone->request); dns_request_destroy(&zone->request);
/* /*
* Skip to next failed / untried master. * Skip to next failed / untried master.
@@ -10490,25 +10472,24 @@ refresh_callback(isc_task_t *task, isc_event_t *event) {
} }
DNS_ZONE_CLRFLAG(zone, DNS_ZONEFLG_USEALTXFRSRC); DNS_ZONE_CLRFLAG(zone, DNS_ZONEFLG_USEALTXFRSRC);
zone_settimer(zone, &now); zone_settimer(zone, &now);
UNLOCK_ZONE(zone);
goto detach; goto detach;
} }
requeue: requeue:
queue_soa_query(zone); queue_soa_query(zone);
UNLOCK_ZONE(zone);
goto detach; goto detach;
same_master: same_master:
if (msg != NULL) if (msg != NULL)
dns_message_destroy(&msg); dns_message_destroy(&msg);
isc_event_free(&event); isc_event_free(&event);
LOCK_ZONE(zone);
dns_request_destroy(&zone->request); dns_request_destroy(&zone->request);
queue_soa_query(zone); queue_soa_query(zone);
UNLOCK_ZONE(zone);
detach: detach:
UNLOCK_ZONE(zone);
if (do_queue_xfrin)
queue_xfrin(zone);
dns_zone_idetach(&zone); dns_zone_idetach(&zone);
return; return;
} }
@@ -10866,13 +10847,13 @@ ns_query(dns_zone_t *zone, dns_rdataset_t *soardataset, dns_stub_t *stub) {
isc_uint16_t udpsize = SEND_BUFFER_SIZE; isc_uint16_t udpsize = SEND_BUFFER_SIZE;
REQUIRE(DNS_ZONE_VALID(zone)); REQUIRE(DNS_ZONE_VALID(zone));
REQUIRE(LOCKED_ZONE(zone));
REQUIRE((soardataset != NULL && stub == NULL) || REQUIRE((soardataset != NULL && stub == NULL) ||
(soardataset == NULL && stub != NULL)); (soardataset == NULL && stub != NULL));
REQUIRE(stub == NULL || DNS_STUB_VALID(stub)); REQUIRE(stub == NULL || DNS_STUB_VALID(stub));
ENTER; ENTER;
LOCK_ZONE(zone);
if (stub == NULL) { if (stub == NULL) {
stub = isc_mem_get(zone->mctx, sizeof(*stub)); stub = isc_mem_get(zone->mctx, sizeof(*stub));
if (stub == NULL) if (stub == NULL)
@@ -11063,10 +11044,9 @@ ns_query(dns_zone_t *zone, dns_rdataset_t *soardataset, dns_stub_t *stub) {
} }
if (message != NULL) if (message != NULL)
dns_message_destroy(&message); dns_message_destroy(&message);
unlock: unlock:
if (key != NULL) if (key != NULL)
dns_tsigkey_detach(&key); dns_tsigkey_detach(&key);
UNLOCK_ZONE(zone);
return; return;
} }
@@ -13807,15 +13787,22 @@ dns_zonemgr_create(isc_mem_t *mctx, isc_taskmgr_t *taskmgr,
zmgr->transfersin = 10; zmgr->transfersin = 10;
zmgr->transfersperns = 2; zmgr->transfersperns = 2;
/* Unreachable lock. */
result = isc_rwlock_init(&zmgr->urlock, 0, 0);
if (result != ISC_R_SUCCESS)
goto free_rwlock;
/* Create a single task for queueing of SOA queries. */ /* Create a single task for queueing of SOA queries. */
result = isc_task_create(taskmgr, 1, &zmgr->task); result = isc_task_create(taskmgr, 1, &zmgr->task);
if (result != ISC_R_SUCCESS) if (result != ISC_R_SUCCESS)
goto free_rwlock; goto free_urlock;
isc_task_setname(zmgr->task, "zmgr", zmgr); isc_task_setname(zmgr->task, "zmgr", zmgr);
result = isc_ratelimiter_create(mctx, timermgr, zmgr->task, result = isc_ratelimiter_create(mctx, timermgr, zmgr->task,
&zmgr->rl); &zmgr->rl);
if (result != ISC_R_SUCCESS) if (result != ISC_R_SUCCESS)
goto free_task; goto free_task;
/* default to 20 refresh queries / notifies per second. */ /* default to 20 refresh queries / notifies per second. */
isc_interval_set(&interval, 0, 1000000000/2); isc_interval_set(&interval, 0, 1000000000/2);
result = isc_ratelimiter_setinterval(zmgr->rl, &interval); result = isc_ratelimiter_setinterval(zmgr->rl, &interval);
@@ -13844,6 +13831,8 @@ dns_zonemgr_create(isc_mem_t *mctx, isc_taskmgr_t *taskmgr,
isc_ratelimiter_detach(&zmgr->rl); isc_ratelimiter_detach(&zmgr->rl);
free_task: free_task:
isc_task_detach(&zmgr->task); isc_task_detach(&zmgr->task);
free_urlock:
isc_rwlock_destroy(&zmgr->urlock);
free_rwlock: free_rwlock:
isc_rwlock_destroy(&zmgr->rwlock); isc_rwlock_destroy(&zmgr->rwlock);
free_mem: free_mem:
@@ -14029,7 +14018,6 @@ dns_zonemgr_shutdown(dns_zonemgr_t *zmgr) {
UNLOCK_ZONE(zone); UNLOCK_ZONE(zone);
} }
RWUNLOCK(&zmgr->rwlock, isc_rwlocktype_read); RWUNLOCK(&zmgr->rwlock, isc_rwlocktype_read);
} }
isc_result_t isc_result_t
@@ -14100,6 +14088,7 @@ zonemgr_free(dns_zonemgr_t *zmgr) {
DESTROYLOCK(&zmgr->iolock); DESTROYLOCK(&zmgr->iolock);
isc_ratelimiter_detach(&zmgr->rl); isc_ratelimiter_detach(&zmgr->rl);
isc_rwlock_destroy(&zmgr->urlock);
isc_rwlock_destroy(&zmgr->rwlock); isc_rwlock_destroy(&zmgr->rwlock);
mctx = zmgr->mctx; mctx = zmgr->mctx;
isc_mem_put(zmgr->mctx, zmgr, sizeof(*zmgr)); isc_mem_put(zmgr->mctx, zmgr, sizeof(*zmgr));
@@ -14513,12 +14502,12 @@ dns_zonemgr_unreachable(dns_zonemgr_t *zmgr, isc_sockaddr_t *remote,
REQUIRE(DNS_ZONEMGR_VALID(zmgr)); REQUIRE(DNS_ZONEMGR_VALID(zmgr));
locktype = isc_rwlocktype_read; locktype = isc_rwlocktype_read;
RWLOCK(&zmgr->rwlock, locktype); RWLOCK(&zmgr->urlock, locktype);
for (i = 0; i < UNREACH_CHACHE_SIZE; i++) { for (i = 0; i < UNREACH_CHACHE_SIZE; i++) {
if (zmgr->unreachable[i].expire >= seconds && if (zmgr->unreachable[i].expire >= seconds &&
isc_sockaddr_equal(&zmgr->unreachable[i].remote, remote) && isc_sockaddr_equal(&zmgr->unreachable[i].remote, remote) &&
isc_sockaddr_equal(&zmgr->unreachable[i].local, local)) { isc_sockaddr_equal(&zmgr->unreachable[i].local, local)) {
result = isc_rwlock_tryupgrade(&zmgr->rwlock); result = isc_rwlock_tryupgrade(&zmgr->urlock);
if (result == ISC_R_SUCCESS) { if (result == ISC_R_SUCCESS) {
locktype = isc_rwlocktype_write; locktype = isc_rwlocktype_write;
zmgr->unreachable[i].last = seconds; zmgr->unreachable[i].last = seconds;
@@ -14526,7 +14515,7 @@ dns_zonemgr_unreachable(dns_zonemgr_t *zmgr, isc_sockaddr_t *remote,
break; break;
} }
} }
RWUNLOCK(&zmgr->rwlock, locktype); RWUNLOCK(&zmgr->urlock, locktype);
return (ISC_TF(i < UNREACH_CHACHE_SIZE)); return (ISC_TF(i < UNREACH_CHACHE_SIZE));
} }
@@ -14547,11 +14536,11 @@ dns_zonemgr_unreachabledel(dns_zonemgr_t *zmgr, isc_sockaddr_t *remote,
REQUIRE(DNS_ZONEMGR_VALID(zmgr)); REQUIRE(DNS_ZONEMGR_VALID(zmgr));
locktype = isc_rwlocktype_read; locktype = isc_rwlocktype_read;
RWLOCK(&zmgr->rwlock, locktype); RWLOCK(&zmgr->urlock, locktype);
for (i = 0; i < UNREACH_CHACHE_SIZE; i++) { for (i = 0; i < UNREACH_CHACHE_SIZE; i++) {
if (isc_sockaddr_equal(&zmgr->unreachable[i].remote, remote) && if (isc_sockaddr_equal(&zmgr->unreachable[i].remote, remote) &&
isc_sockaddr_equal(&zmgr->unreachable[i].local, local)) { isc_sockaddr_equal(&zmgr->unreachable[i].local, local)) {
result = isc_rwlock_tryupgrade(&zmgr->rwlock); result = isc_rwlock_tryupgrade(&zmgr->urlock);
if (result == ISC_R_SUCCESS) { if (result == ISC_R_SUCCESS) {
locktype = isc_rwlocktype_write; locktype = isc_rwlocktype_write;
zmgr->unreachable[i].expire = 0; zmgr->unreachable[i].expire = 0;
@@ -14564,7 +14553,7 @@ dns_zonemgr_unreachabledel(dns_zonemgr_t *zmgr, isc_sockaddr_t *remote,
break; break;
} }
} }
RWUNLOCK(&zmgr->rwlock, locktype); RWUNLOCK(&zmgr->urlock, locktype);
} }
void void
@@ -14577,7 +14566,7 @@ dns_zonemgr_unreachableadd(dns_zonemgr_t *zmgr, isc_sockaddr_t *remote,
REQUIRE(DNS_ZONEMGR_VALID(zmgr)); REQUIRE(DNS_ZONEMGR_VALID(zmgr));
RWLOCK(&zmgr->rwlock, isc_rwlocktype_write); RWLOCK(&zmgr->urlock, isc_rwlocktype_write);
for (i = 0; i < UNREACH_CHACHE_SIZE; i++) { for (i = 0; i < UNREACH_CHACHE_SIZE; i++) {
/* Existing entry? */ /* Existing entry? */
if (isc_sockaddr_equal(&zmgr->unreachable[i].remote, remote) && if (isc_sockaddr_equal(&zmgr->unreachable[i].remote, remote) &&
@@ -14616,7 +14605,7 @@ dns_zonemgr_unreachableadd(dns_zonemgr_t *zmgr, isc_sockaddr_t *remote,
zmgr->unreachable[oldest].remote = *remote; zmgr->unreachable[oldest].remote = *remote;
zmgr->unreachable[oldest].local = *local; zmgr->unreachable[oldest].local = *local;
} }
RWUNLOCK(&zmgr->rwlock, isc_rwlocktype_write); RWUNLOCK(&zmgr->urlock, isc_rwlocktype_write);
} }
void void