[#722,!3-p] Addressed review comment

src/bin/dhcp6/dhcp6_srv.cc
    Dhcp6Srv::sanityCheckDuid() - reworked sanity check logic

diff --git a/src/bin/dhcp6/dhcp6_srv.cc b/src/bin/dhcp6/dhcp6_srv.cc
index 48ab4d8986af7c1c1e445ab8c636b8061effd8de..6a58d6aa13ffab6449537a76410a13bc67c42ce9 100644 (file)
--- a/src/bin/dhcp6/dhcp6_srv.cc
+++ b/src/bin/dhcp6/dhcp6_srv.cc
@@ -1353,9 +1353,10 @@ void Dhcpv6Srv::sanityCheckDUID(const OptionPtr& opt, const std::string& opt_nam
     // The client-id or server-id has to have at least 3 bytes of useful data:
     // two for duid type and one more for actual duid value.
     uint16_t len = opt->len() - opt->getHeaderLen();
-    if ( (len < 3) || (len > DUID::MAX_DUID_LEN) ) {
-        isc_throw(RFCViolation, "Received " << opt_name << " option with invalid length: "
-                  << len << ", expected 3..128.");
+    if (len < 3 || len > DUID::MAX_DUID_LEN || opt->getData().empty()) {
+        isc_throw(RFCViolation, "Received invalid DUID for " << opt_name << ", received "
+                  << len << " byte(s). It must be at least 3 and no more than "
+                  << DUID::MAX_DUID_LEN);
     }
 }