Browse Source

[1539] fixed a small regression: unsupportedRequest now excludes update.

JINMEI Tatuya 13 years ago
parent
commit
7433c00e24
2 changed files with 3 additions and 3 deletions
  1. 1 1
      src/bin/auth/auth_srv.cc
  2. 2 2
      src/bin/auth/tests/auth_srv_unittest.cc

+ 1 - 1
src/bin/auth/auth_srv.cc

@@ -758,7 +758,7 @@ AuthSrvImpl::processNotify(const IOMessage& io_message, Message& message,
 
 
 bool
 bool
 AuthSrvImpl::processUpdate(const IOMessage& /*io_message*/,
 AuthSrvImpl::processUpdate(const IOMessage& /*io_message*/,
-                           Message& /*message*/, 
+                           Message& /*message*/,
                            OutputBuffer& /*buffer*/,
                            OutputBuffer& /*buffer*/,
                            std::auto_ptr<TSIGContext> /*tsig_context*/)
                            std::auto_ptr<TSIGContext> /*tsig_context*/)
 {
 {

+ 2 - 2
src/bin/auth/tests/auth_srv_unittest.cc

@@ -247,8 +247,8 @@ TEST_F(AuthSrvTest, iqueryViaDNSServer) {
 // Unsupported requests.  Should result in NOTIMP.
 // Unsupported requests.  Should result in NOTIMP.
 TEST_F(AuthSrvTest, unsupportedRequest) {
 TEST_F(AuthSrvTest, unsupportedRequest) {
     unsupportedRequest();
     unsupportedRequest();
-    // unsupportedRequest tries 14 different opcodes
-    checkAllRcodeCountersZeroExcept(Rcode::NOTIMP(), 14);
+    // unsupportedRequest tries 13 different opcodes
+    checkAllRcodeCountersZeroExcept(Rcode::NOTIMP(), 13);
 }
 }
 
 
 // Multiple questions.  Should result in FORMERR.
 // Multiple questions.  Should result in FORMERR.