From 7ec42e4be45c0486ce80461293f377fb4b904dc0 Mon Sep 17 00:00:00 2001
From: Michael Graff <mgraff@isc.org>
Date: Tue, 26 Oct 1999 23:57:38 +0000
Subject: [PATCH] Handle the case where kill_name() is called on an already
 dead name correctly.

---
 lib/dns/adb.c | 19 ++++++++++++++++++-
 1 file changed, 18 insertions(+), 1 deletion(-)

diff --git a/lib/dns/adb.c b/lib/dns/adb.c
index c471652d95..f184bdc66c 100644
--- a/lib/dns/adb.c
+++ b/lib/dns/adb.c
@@ -258,11 +258,28 @@ kill_name(dns_adb_t *adb, dns_adbname_t **n, isc_eventtype_t ev)
 	name = *n;
 	*n = NULL;
 	INSIST(DNS_ADBNAME_VALID(name));
-	INSIST(name->dead == ISC_FALSE);
 
+	/*
+	 * If we're dead already, just check to see if we should go
+	 * away now or not.
+	 */
+	if (name->dead && ISC_LIST_EMPTY(name->fetches)) {
+		unlink_name(adb, name);
+		free_adbname(adb, &name);
+		return;
+	}
+
+	/*
+	 * Clean up the name's various lists.  These two are destructive
+	 * in that they will always empty the list.
+	 */
 	clean_handles_at_name(name, ev);
 	clean_namehooks_at_name(adb, name);
 
+	/*
+	 * If fetches are running, cancel them.  If none are running, we can
+	 * just kill the name here.
+	 */
 	if (ISC_LIST_EMPTY(name->fetches)) {
 		unlink_name(adb, name);
 		free_adbname(adb, &name);