[1607] use the result of findZone() to check if qname is zone origin.

This should be more efficient than expensive name comparison.
This is a small (unrelated) optimization, but hopefully it's okay.

diff --git a/src/bin/auth/query.cc b/src/bin/auth/query.cc
index 59e3b7f9997368906e0eef24ac3e44701e83e10c..7d50393202c4c419348a75c4434d1c62e836a10c 100644 (file)
--- a/src/bin/auth/query.cc
+++ b/src/bin/auth/query.cc
@@ -503,7 +503,9 @@ Query::process() {
             // section, insert apex NS records into the authority section
             // and AAAA/A RRS of each of the NS RDATA into the additional
             // section.
-            if (qname_ != result.zone_finder->getOrigin() ||
+            // Checking the findZone() is a lightweight check to see if
+            // qname is the zone origin.
+            if (result.code != result::SUCCESS ||
                 db_context->code != ZoneFinder::SUCCESS ||
                 (qtype_ != RRType::NS() && !qtype_is_any))
             {