On Fri, Nov 10, 2017 at 05:03:54AM +0000, Mark Andrews via RT wrote:
> insert_and_delete calls dns_rbt_addname which updates node->data on
> ISC_R_EXISTS and then returns ISC_R_SUCCESS.  This results in the
> expected node count not matching the reported node count.

The branch looks good and is correct. The previous code was incorrect.

Was this case reached btw? It looks very unlikely that two 32-byte long
random alphabetic names will have a collision when 1024 names are
inserted into the tree.

		Mukund