upstream/bind9
1
0
Fork 0
mirror of https://github.com/isc-projects/bind9.git synced 2026-06-11 04:40:00 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Merge branch '4106-lock-order-inversion-in-resolver.c' into 'bind-9.18'

Browse source 
[9.18] Fix a lock-order-inversion bug in resolver.c

See merge request isc-projects/bind9!8000
This commit is contained in:
Arаm Sаrgsyаn 2023-06-06 11:56:01 +00:00
commit e72c92c497
1 changed files with 2 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
lib/dns/resolver.c
View file

@ -4441,9 +4441,7 @@ fctx_destroy(fetchctx_t *fctx, bool exiting) {
if (bucket_empty && exiting &&
isc_refcount_decrement(&res->activebuckets) == 1)
{
LOCK(&res->lock);
send_shutdown_events(res);
UNLOCK(&res->lock);
}
isc_refcount_destroy(&fctx->references);
@ -10260,10 +10258,7 @@ send_shutdown_events(dns_resolver_t *res) {
isc_event_t *event, *next_event;
isc_task_t *etask;
/*
* Caller must be holding the resolver lock.
*/
LOCK(&res->lock);
for (event = ISC_LIST_HEAD(res->whenshutdown); event != NULL;
event = next_event)
{
@ -10273,6 +10268,7 @@ send_shutdown_events(dns_resolver_t *res) {
event->ev_sender = res;
isc_task_sendanddetach(&etask, &event);
}
UNLOCK(&res->lock);
}
static void
@ -10287,7 +10283,6 @@ spillattimer_countdown(isc_task_t *task, isc_event_t *event) {
UNUSED(task);
LOCK(&res->lock);
INSIST(!atomic_load_acquire(&res->exiting));
if (res->spillat > res->spillatmin) {
res->spillat--;
logit = true;
@ -10644,7 +10639,6 @@ dns_resolver_shutdown(dns_resolver_t *res) {
RTRACE("shutdown");
LOCK(&res->lock);
if (atomic_compare_exchange_strong(&res->exiting, &is_false, true)) {
RTRACE("exiting");
@ -10673,7 +10667,6 @@ dns_resolver_shutdown(dns_resolver_t *res) {
true);
RUNTIME_CHECK(result == ISC_R_SUCCESS);
}
UNLOCK(&res->lock);
}
void