|
 |
900526 |
commit 3a2ea636103eaf40404fb82f228605d384c36434
|
|
|
900526 |
Author: Mark Andrews <marka@isc.org>
|
|
|
900526 |
Date: Tue Dec 17 09:08:59 2013 +1100
|
|
|
900526 |
|
|
|
900526 |
3692. [bug] Two calls to dns_db_getoriginnode were fatal if there
|
|
|
900526 |
was no data at the node. [RT #35080]
|
|
|
900526 |
|
|
|
900526 |
(cherry picked from commit 161e803a5608956271d8120be37a1b383d14b647)
|
|
|
900526 |
|
|
|
900526 |
diff --git a/lib/dns/rbtdb.c b/lib/dns/rbtdb.c
|
|
|
900526 |
index 2dd4aa0..941b77e 100644
|
|
|
900526 |
--- a/lib/dns/rbtdb.c
|
|
|
900526 |
+++ b/lib/dns/rbtdb.c
|
|
|
900526 |
@@ -1638,8 +1638,11 @@ decrement_reference(dns_rbtdb_t *rbtdb, dns_rbtnode_t *node,
|
|
|
900526 |
|
|
|
900526 |
nodelock = &rbtdb->node_locks[bucket];
|
|
|
900526 |
|
|
|
900526 |
+#define KEEP_NODE(n, r) \
|
|
|
900526 |
+ ((n)->data != NULL || (n)->down != NULL || (n) == (r)->origin_node)
|
|
|
900526 |
+
|
|
|
900526 |
/* Handle easy and typical case first. */
|
|
|
900526 |
- if (!node->dirty && (node->data != NULL || node->down != NULL)) {
|
|
|
900526 |
+ if (!node->dirty && KEEP_NODE(node, rbtdb)) {
|
|
|
900526 |
dns_rbtnode_refdecrement(node, &nrefs);
|
|
|
900526 |
INSIST((int)nrefs >= 0);
|
|
|
900526 |
if (nrefs == 0) {
|
|
|
900526 |
@@ -1708,12 +1711,11 @@ decrement_reference(dns_rbtdb_t *rbtdb, dns_rbtnode_t *node,
|
|
|
900526 |
isc_refcount_decrement(&nodelock->references, &refs);
|
|
|
900526 |
INSIST((int)refs >= 0);
|
|
|
900526 |
|
|
|
900526 |
- /*
|
|
|
900526 |
- * XXXDCL should this only be done for cache zones?
|
|
|
900526 |
- */
|
|
|
900526 |
- if (node->data != NULL || node->down != NULL)
|
|
|
900526 |
+ if (KEEP_NODE(node, rbtdb))
|
|
|
900526 |
goto restore_locks;
|
|
|
900526 |
|
|
|
900526 |
+#undef KEEP_NODE
|
|
|
900526 |
+
|
|
|
900526 |
if (write_locked) {
|
|
|
900526 |
/*
|
|
|
900526 |
* We can now delete the node.
|