BIND 10 trac2165_2, updated. 7f00956d43e793624bddf52454a8eedf91376e69 [2165] Don't strip RRSIGs from DNSSEC rrsets

BIND 10 source code commits bind10-changes at lists.isc.org
Wed Aug 22 05:02:54 UTC 2012


The branch, trac2165_2 has been updated
       via  7f00956d43e793624bddf52454a8eedf91376e69 (commit)
      from  9a13449fd43574629557b2b0aa67592a58729fc5 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 7f00956d43e793624bddf52454a8eedf91376e69
Author: Mukund Sivaraman <muks at isc.org>
Date:   Wed Aug 22 09:38:25 2012 +0530

    [2165] Don't strip RRSIGs from DNSSEC rrsets

-----------------------------------------------------------------------

Summary of changes:
 src/lib/datasrc/database.cc |   12 ++++++------
 src/lib/datasrc/database.h  |    1 -
 2 files changed, 6 insertions(+), 7 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/lib/datasrc/database.cc b/src/lib/datasrc/database.cc
index 09f7e3b..022e2b9 100644
--- a/src/lib/datasrc/database.cc
+++ b/src/lib/datasrc/database.cc
@@ -638,7 +638,7 @@ DatabaseClient::Finder::findWildcardMatch(
 ZoneFinder::ResultContext
 DatabaseClient::Finder::logAndCreateResult(
     const Name& name, const string* wildname, const RRType& type,
-    ZoneFinder::Result code, ConstRRsetPtr rrset, const FindOptions options,
+    ZoneFinder::Result code, ConstRRsetPtr rrset,
     const isc::log::MessageID& log_id, FindResultFlags flags) const
 {
     if (rrset) {
@@ -662,7 +662,7 @@ DatabaseClient::Finder::logAndCreateResult(
                 arg(getClass()).arg(*wildname);
         }
     }
-    return (ResultContext(code, stripRRsigs(rrset, options), flags));
+    return (ResultContext(code, rrset, flags));
 }
 
 DatabaseClient::Finder::FindDNSSECContext::FindDNSSECContext(
@@ -810,7 +810,7 @@ DatabaseClient::Finder::findOnNameResult(const Name& name,
         // - when we are looking for glue records (FIND_GLUE_OK), or
         // - when the query type is DS (which cancels the delegation)
         return (logAndCreateResult(name, wildname, type, DELEGATION,
-                                   nsi->second, options,
+                                   stripRRsigs(nsi->second, options),
                                    wild ? DATASRC_DATABASE_WILDCARD_NS :
                                    DATASRC_DATABASE_FOUND_DELEGATION_EXACT,
                                    flags));
@@ -827,7 +827,7 @@ DatabaseClient::Finder::findOnNameResult(const Name& name,
                       ", expected 1");
         }
         return (logAndCreateResult(name, wildname, type, CNAME,
-                                   cni->second, options,
+                                   stripRRsigs(cni->second, options),
                                    wild ? DATASRC_DATABASE_WILDCARD_CNAME :
                                    DATASRC_DATABASE_FOUND_CNAME,
                                    flags));
@@ -885,12 +885,12 @@ DatabaseClient::Finder::findOnNameResult(const Name& name,
         // This log message covers both normal and wildcard cases, so we pass
         // NULL for 'wildname'.
         return (logAndCreateResult(name, NULL, type, NXRRSET,
-                                   dnssec_rrset, options,
+                                   dnssec_rrset,
                                    DATASRC_DATABASE_FOUND_NXRRSET_NSEC,
                                    flags | RESULT_NSEC_SIGNED));
     }
     return (logAndCreateResult(name, wildname, type, NXRRSET,
-                               dnssec_rrset, options,
+                               dnssec_rrset,
                                wild ? DATASRC_DATABASE_WILDCARD_NXRRSET :
                                DATASRC_DATABASE_FOUND_NXRRSET,
                                flags | dnssec_ctx.getResultFlags()));
diff --git a/src/lib/datasrc/database.h b/src/lib/datasrc/database.h
index 2a2b484..65ddfcc 100644
--- a/src/lib/datasrc/database.h
+++ b/src/lib/datasrc/database.h
@@ -1336,7 +1336,6 @@ public:
                                          const isc::dns::RRType& type,
                                          ZoneFinder::Result code,
                                          isc::dns::ConstRRsetPtr rrset,
-                                         const FindOptions options,
                                          const isc::log::MessageID& log_id,
                                          FindResultFlags flags) const;
 



More information about the bind10-changes mailing list