Browse Source

[1177] Don't assume findPreviousName contains NSEC

So don't say it's necessary bug, it might be badly signed zone as well.
Michal 'vorner' Vaner 13 years ago
parent
commit
06a24c6882
1 changed files with 6 additions and 4 deletions
  1. 6 4
      src/lib/datasrc/database.cc

+ 6 - 4
src/lib/datasrc/database.cc

@@ -575,11 +575,12 @@ DatabaseClient::Finder::find(const isc::dns::Name& name,
                                 result_status = WILDCARD_NXRRSET;
                                 result_rrset = nci->second;
                             } else {
-                                // The previous doesn't contain NSEC, bug?
+                                // The previous doesn't contain NSEC.
+                                // Badly signed zone or a bug?
                                 isc_throw(DataSourceError, "No NSEC in " +
                                           coverName.toText() + ", but it was "
                                           "returned as previous - "
-                                          "accessor error?");
+                                          "accessor error? Badly signed zone?");
                             }
                         }
                         break;
@@ -618,11 +619,12 @@ DatabaseClient::Finder::find(const isc::dns::Name& name,
                         result_status = NXDOMAIN;
                         result_rrset = nci->second;
                     } else {
-                        // The previous doesn't contain NSEC, bug?
+                        // The previous doesn't contain NSEC.
+                        // Badly signed zone or a bug?
                         isc_throw(DataSourceError, "No NSEC in " +
                                   coverName.toText() + ", but it was "
                                   "returned as previous - "
-                                  "accessor error?");
+                                  "accessor error? Badly signed zone?");
                     }
                 }
                 catch (const isc::NotImplemented&) {