more polish

diff --git a/pdns/dnsparser.cc b/pdns/dnsparser.cc
index 1a1fbde99e3a956f3be125bbaa4dc37d4d3a86ee..ccdcddfd0757a7aa08d6d2f76931b7bb6b055b37 100644 (file)
--- a/pdns/dnsparser.cc
+++ b/pdns/dnsparser.cc
@@ -386,7 +386,7 @@ void PacketReader::xfrNodeOrLocatorID(NodeOrLocatorID& ret)
   if (d_pos + sizeof(ret) > d_content.size()) {
     throw std::out_of_range("Attempt to read 64 bit value outside of packet");
   }
-  memcpy(&ret, &d_content.at(d_pos), sizeof(ret));
+  memcpy(&ret.content, &d_content.at(d_pos), sizeof(ret.content));
   d_pos += sizeof(ret);
 }
 

diff --git a/pdns/dnswriter.cc b/pdns/dnswriter.cc
index bcb2884779cffe15e7e3050475e8fa766907d00a..1255dbea73a7e8c78285acb4d2fecb68116a19b6 100644 (file)
--- a/pdns/dnswriter.cc
+++ b/pdns/dnswriter.cc
@@ -140,7 +140,7 @@ template <typename Container> void GenericDNSPacketWriter<Container>::xfr48BitIn
   d_content.insert(d_content.end(), bytes, bytes + sizeof(bytes));
 }
 
-template <typename Container> void GenericDNSPacketWriter<Container>::xfrNodeOrLocatorID(NodeOrLocatorID val)
+template <typename Container> void GenericDNSPacketWriter<Container>::xfrNodeOrLocatorID(const NodeOrLocatorID& val)
 {
   d_content.insert(d_content.end(), val.content, val.content + sizeof(val.content));
 }

diff --git a/pdns/dnswriter.hh b/pdns/dnswriter.hh
index 96e8d4f9a227d7a3ea24eb2883394e5f113f4a86..55b5f6f0625d0cdf76b4c75a2206d99dfc1e3d71 100644 (file)
--- a/pdns/dnswriter.hh
+++ b/pdns/dnswriter.hh
@@ -86,7 +86,7 @@ public:
   void truncate();
 
   void xfr48BitInt(uint64_t val);
-  void xfrNodeOrLocatorID(NodeOrLocatorID val);
+  void xfrNodeOrLocatorID(const NodeOrLocatorID& val);
   void xfr32BitInt(uint32_t val);
   void xfr16BitInt(uint16_t val);
   void xfrType(uint16_t val)

diff --git a/pdns/rcpgenerator.cc b/pdns/rcpgenerator.cc
index 20f707cb0c88745cd7a796e07cc19154a6bfce3e..3b5ce6f1ade2e08debd26313d313511091c0dec9 100644 (file)
--- a/pdns/rcpgenerator.cc
+++ b/pdns/rcpgenerator.cc
@@ -66,7 +66,7 @@ void RecordTextReader::xfrNodeOrLocatorID(NodeOrLocatorID& val) {
     throw RecordTextException("while parsing colon-delimited 64-bit field: '" + d_string.substr(d_pos, len) + "' is invalid");
   }
 
-  std::memcpy(&val, tmpbuf.s6_addr, sizeof(val));
+  std::memcpy(&val.content, tmpbuf.s6_addr, sizeof(val.content));
   d_pos += len;
 }