[5553] Addressed review comments

    Adjusted for addition DHCP types, changed log
    message to INFO level.

diff --git a/src/bin/dhcp4/dhcp4_srv.cc b/src/bin/dhcp4/dhcp4_srv.cc
index ead5560ccb129ec838eabb797f0425d7d5f042f4..87ca9d0cef3558354450f8949a8b04d4d1978d53 100644 (file)
--- a/src/bin/dhcp4/dhcp4_srv.cc
+++ b/src/bin/dhcp4/dhcp4_srv.cc
@@ -2847,12 +2847,13 @@ Dhcpv4Srv::acceptMessageType(const Pkt4Ptr& query) const {
             break;
 
         case DHCP_NOTYPE:
-            LOG_DEBUG(bad_packet4_logger, DBG_DHCP4_DETAIL, DHCP4_PACKET_DROP_0009)
-                      .arg(query->getLabel());
+            LOG_INFO(bad_packet4_logger, DHCP4_PACKET_DROP_0009)
+                     .arg(query->getLabel());
             break;
+
         default:
             // If we receive a message with a non-existing type, we are logging it.
-            if (type > DHCPLEASEQUERYDONE) {
+            if (type >= DHCP_TYPES_EOF) {
                 LOG_DEBUG(bad_packet4_logger, DBG_DHCP4_DETAIL, DHCP4_PACKET_DROP_0005)
                           .arg(query->getLabel())
                           .arg(type);

diff --git a/src/bin/dhcp4/tests/dhcp4_srv_unittest.cc b/src/bin/dhcp4/tests/dhcp4_srv_unittest.cc
index a2bdd6fd7fa4816fa210dfe1dda227e35c13d815..e1d14878f5a84c85eac59c7f8041fe890c7d206d 100644 (file)
--- a/src/bin/dhcp4/tests/dhcp4_srv_unittest.cc
+++ b/src/bin/dhcp4/tests/dhcp4_srv_unittest.cc
@@ -3999,7 +3999,7 @@ TEST_F(Dhcpv4SrvTest, acceptMessageType) {
     ASSERT_EQ(DHCP_NOTYPE, pkt->getType());
     EXPECT_FALSE(srv.acceptMessageType(Pkt4Ptr(new Pkt4(&bin[0], bin.size()))));
 
-    // Verify that we drop packets with types > DHCPLEASEQUERYDONE
+    // Verify that we drop packets with types >= DHCP_TYPES_EOF
     // Make Discover with type changed to 0xff
     std::vector<uint8_t> bin2;
     const char* invalid_msg_type =
@@ -4022,6 +4022,8 @@ TEST_F(Dhcpv4SrvTest, acceptMessageType) {
 
     bin.clear();
     isc::util::encode::decodeHex(invalid_msg_type, bin);
+
+    std::cout << "bin[32]: " << (int)(bin[32]) << std::endl;
     pkt.reset(new Pkt4(&bin[0], bin.size()));
     pkt->unpack();
     ASSERT_EQ(0xff, pkt->getType());

diff --git a/src/lib/dhcp/dhcp4.h b/src/lib/dhcp/dhcp4.h
index 8fd8297ea9178477238671651dfe887b9fed1eed..56d2586f689f8477af7cab95625e6da4c49cfe38 100644 (file)
--- a/src/lib/dhcp/dhcp4.h
+++ b/src/lib/dhcp/dhcp4.h
@@ -239,10 +239,11 @@ enum DHCPMessageType {
     DHCPLEASEUNKNOWN    =  12,
     DHCPLEASEACTIVE     =  13,
     DHCPBULKLEASEQUERY  =  14,
-    DHCPLEASEQUERYDONE  =  15
+    DHCPLEASEQUERYDONE  =  15,
 // DHCPACTIVELEASEQUERY =  16,
-// DHCPLEASEQUERYSTATUS =  17,
-// DHCPTLS              =  18
+    DHCPLEASEQUERYSTATUS =  17,
+    DHCPTLS              =  18,
+    DHCP_TYPES_EOF
 };
 
 static const uint16_t DHCP4_CLIENT_PORT = 68;