From ec3c959c690fa2ccd66b314a75f26c22689d3a0c Mon Sep 17 00:00:00 2001 From: Razvan Becheriu Date: Mon, 17 Aug 2020 16:50:54 +0300 Subject: [PATCH] [#1373] also check fqdn ending byte --- src/hooks/dhcp/flex_option/tests/flex_option_unittests.cc | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/hooks/dhcp/flex_option/tests/flex_option_unittests.cc b/src/hooks/dhcp/flex_option/tests/flex_option_unittests.cc index 8c15b04d78..5f2277435a 100644 --- a/src/hooks/dhcp/flex_option/tests/flex_option_unittests.cc +++ b/src/hooks/dhcp/flex_option/tests/flex_option_unittests.cc @@ -796,6 +796,7 @@ TEST_F(FlexOptionTest, processAdd) { EXPECT_EQ(0, memcmp(&buffer_fqdn[1], "example", 7)); EXPECT_EQ(3, buffer_fqdn[8]); EXPECT_EQ(0, memcmp(&buffer_fqdn[9], "com", 3)); + EXPECT_EQ(0, buffer_fqdn[12]); } // Verify that ADD action does not add an already existing option. @@ -891,6 +892,7 @@ TEST_F(FlexOptionTest, processSupersede) { EXPECT_EQ(0, memcmp(&buffer_fqdn[1], "example", 7)); EXPECT_EQ(3, buffer_fqdn[8]); EXPECT_EQ(0, memcmp(&buffer_fqdn[9], "com", 3)); + EXPECT_EQ(0, buffer_fqdn[12]); } // Verify that SUPERSEDE action supersedes an already existing option. @@ -943,6 +945,7 @@ TEST_F(FlexOptionTest, processSupersedeExisting) { EXPECT_EQ(0, memcmp(&buffer_fqdn[1], "example", 7)); EXPECT_EQ(3, buffer_fqdn[8]); EXPECT_EQ(0, memcmp(&buffer_fqdn[9], "com", 3)); + EXPECT_EQ(0, buffer_fqdn[12]); } // Verify that SUPERSEDE action does not supersede an empty value.