[2098] make sure unexpected method call doesn't happen by exception/assert

diff --git a/src/lib/datasrc/memory/rdata_serialization.cc b/src/lib/datasrc/memory/rdata_serialization.cc
index 6c3e06c35b245eae79146dbe8bbf61cf9890ba00..6ac18d0e6a802ca72dca24ad9c5b31659c04a02e 100644 (file)
--- a/src/lib/datasrc/memory/rdata_serialization.cc
+++ b/src/lib/datasrc/memory/rdata_serialization.cc
@@ -250,6 +250,7 @@ public:
     virtual void setCompressMode(CompressMode) {}
     virtual void writeName(const LabelSequence&, bool) {
         // We don't need this version of writeName
+        isc_throw(Unexpected, "unexpected version of writeName is called");
     }
 
     // Called for each domain name in the RDATA, from the RDATA's toWire()

diff --git a/src/lib/dns/benchmarks/message_renderer_bench.cc b/src/lib/dns/benchmarks/message_renderer_bench.cc
index 752b1f664c72e7258c2fe3035ee9399a349e6ee1..6376498087a16e3d74035758d86bae1965228b2a 100644 (file)
--- a/src/lib/dns/benchmarks/message_renderer_bench.cc
+++ b/src/lib/dns/benchmarks/message_renderer_bench.cc
@@ -116,7 +116,11 @@ public:
     virtual void writeName(const Name& name, const bool = false) {
         name.toWire(getBuffer());
     }
-    virtual void writeName(const LabelSequence&, const bool) {} // unused
+    virtual void writeName(const LabelSequence&, const bool) {
+        // We shouldn't use this version of writeName (and we internally
+        // control it, so we simply assert it here)
+        assert(false);
+    }
 };
 
 void

diff --git a/src/lib/dns/benchmarks/oldmessagerenderer.cc b/src/lib/dns/benchmarks/oldmessagerenderer.cc
index a6c7990115a98d9247c9765c448bace21798041a..ed575a083fc6f42bc3414244551bac4cd1cbc5ec 100644 (file)
--- a/src/lib/dns/benchmarks/oldmessagerenderer.cc
+++ b/src/lib/dns/benchmarks/oldmessagerenderer.cc
@@ -277,7 +277,9 @@ OldMessageRenderer::writeName(const Name& name, const bool compress) {
 
 void
 OldMessageRenderer::writeName(const LabelSequence&, const bool) {
-    // we don't use this mode for the benchmark
+    // We shouldn't use this version of writeName (and we internally
+    // control it, so we simply assert it here)
+    assert(false);
 }
 
 }