BIND 10 trac1613, updated. eadab44ef359700ba6ca50a2dc93b7bef2feae39 [1613] data mangle wasn't necessary, reverted

[BIND 10 source code commits]

The branch, trac1613 has been updated
       via  eadab44ef359700ba6ca50a2dc93b7bef2feae39 (commit)
      from  466339143a9161f821fa63e5545ee9cc72d8e1a6 (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 eadab44ef359700ba6ca50a2dc93b7bef2feae39
Author: Jelte Jansen <jelte at isc.org>
Date:   Thu Feb 16 14:53:01 2012 +0100

    [1613] data mangle wasn't necessary, reverted

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

Summary of changes:
 src/bin/auth/tests/auth_srv_unittest.cc |   21 ---------------------
 src/lib/testutils/srv_test.cc           |   16 ++++------------
 src/lib/testutils/srv_test.h            |    9 +--------
 3 files changed, 5 insertions(+), 41 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/bin/auth/tests/auth_srv_unittest.cc b/src/bin/auth/tests/auth_srv_unittest.cc
index 77cff86..4938476 100644
--- a/src/bin/auth/tests/auth_srv_unittest.cc
+++ b/src/bin/auth/tests/auth_srv_unittest.cc
@@ -177,27 +177,6 @@ TEST_F(AuthSrvTest, builtInQuery) {
     checkAllRcodeCountersZeroExcept(Rcode::NOERROR(), 1);
 }
 
-// Callback used in createRequestMessage that mangles the
-// wiredata to something that should not be parseable (to test
-// really badly formed queries)
-// This specific one simply increments every octet in the array
-void requestMangler(uint8_t* data, size_t data_len) {
-    for (size_t i = 0; i < data_len; ++i) {
-        data[i]++;
-    }
-}
-
-// Same as buildInQuery, but completely malform the sent query
-TEST_F(AuthSrvTest, builtInMalformedQuery) {
-    UnitTestUtil::createRequestMessage(request_message, Opcode::QUERY(),
-                                       default_qid, Name("version.bind"),
-                                       RRClass::CH(), RRType::TXT());
-    createRequestPacket(request_message, IPPROTO_UDP, NULL, &requestMangler);
-    server.processMessage(*io_message, parse_message, response_obuffer,
-                          &dnsserv);
-    checkAllRcodeCountersZeroExcept(Rcode::FORMERR(), 1);
-}
-
 // Same test emulating the UDPServer class behavior (defined in libasiolink).
 // This is not a good test in that it assumes internal implementation details
 // of UDPServer, but we've encountered a regression due to the introduction
diff --git a/src/lib/testutils/srv_test.cc b/src/lib/testutils/srv_test.cc
index ca632d5..9d38eb7 100644
--- a/src/lib/testutils/srv_test.cc
+++ b/src/lib/testutils/srv_test.cc
@@ -72,8 +72,7 @@ SrvTestBase::createDataFromFile(const char* const datafile,
 
 void
 SrvTestBase::createRequestPacket(Message& message,
-                                    const int protocol, TSIGContext* context,
-                                    void (*callback)(uint8_t*, size_t))
+                                    const int protocol, TSIGContext* context)
 {
     if (context == NULL) {
         message.toWire(request_renderer);
@@ -88,16 +87,9 @@ SrvTestBase::createRequestPacket(Message& message,
     io_sock = (protocol == IPPROTO_UDP) ? &IOSocket::getDummyUDPSocket() :
         &IOSocket::getDummyTCPSocket();
 
-    const void *data = request_renderer.getData();
-    size_t data_len = request_renderer.getLength();
-
-    if (callback) {
-        // convert to non-const uint8_t for easy manipulation by the callback
-        uint8_t *mdata = const_cast<uint8_t*>(static_cast<const uint8_t*>(data));
-        callback(mdata, data_len);
-    }
-    
-    io_message = new IOMessage(data, data_len, *io_sock, *endpoint);
+    io_message = new IOMessage(request_renderer.getData(),
+                               request_renderer.getLength(),
+                               *io_sock, *endpoint);
 }
 
 // Unsupported requests.  Should result in NOTIMP.
diff --git a/src/lib/testutils/srv_test.h b/src/lib/testutils/srv_test.h
index fe97da7..630232c 100644
--- a/src/lib/testutils/srv_test.h
+++ b/src/lib/testutils/srv_test.h
@@ -83,16 +83,9 @@ protected:
     /// It constructs wire-format DNS packet data from \c message in the
     /// form of \c IOMessage in \c io_message.
     /// The existing content of \c io_message, if any, will be deleted.
-    ///
-    /// If callback is given, it is called after rendering the message,
-    /// but before 'sending' it; this allows the given callback to modify
-    /// the data and introduce bad requests that would otherwise be hard
-    /// to reproduce (it passes the wire-format data as non-const uint8_t,
-    /// so tread lightly when using this).
     void createRequestPacket(isc::dns::Message& message,
                              const int protocol = IPPROTO_UDP,
-                             isc::dns::TSIGContext* context = NULL,
-                             void (*callback)(uint8_t*, size_t) = NULL);
+                             isc::dns::TSIGContext* context = NULL);
 
     MockSession notify_session;
     MockServer dnsserv;