From: Thomas Markwalder <tmark@isc.org>
Date: Fri, 7 Nov 2025 17:57:54 +0000 (-0500)
Subject: [#4161] Addressed review comments
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=HEAD;p=thirdparty%2Fkea.git

[#4161] Addressed review comments

Update stats after #3140 merge

modified:   src/bin/dhcp6/dhcp6_srv.cc
modified:   src/bin/dhcp6/tests/addr_reg_unittest.cc
---

diff --git a/src/bin/dhcp6/dhcp6_srv.cc b/src/bin/dhcp6/dhcp6_srv.cc
index 6a1b1a9c41..04a50acfed 100644
--- a/src/bin/dhcp6/dhcp6_srv.cc
+++ b/src/bin/dhcp6/dhcp6_srv.cc
@@ -4565,10 +4565,9 @@ Dhcpv6Srv::processAddrRegInform(AllocEngine::ClientContext6& ctx) {
 
     if (allow_address_registration && !allow_address_registration->boolValue()) {
         LOG_DEBUG(packet6_logger, DBG_DHCP6_BASIC, DHCP6_ADDR6_REGISTER_DISABLED_DROP)
-                .arg(addr_reg_inf->getLabel());
+                .arg(ctx.query_->getLabel());
         StatsMgr::instance().addValue("pkt6-admin-filtered",
                                       static_cast<int64_t>(1));
-
         StatsMgr::instance().addValue("pkt6-receive-drop",
                                       static_cast<int64_t>(1));
         return(Pkt6Ptr());
diff --git a/src/bin/dhcp6/tests/addr_reg_unittest.cc b/src/bin/dhcp6/tests/addr_reg_unittest.cc
index ffee9521ae..be4cf2623b 100644
--- a/src/bin/dhcp6/tests/addr_reg_unittest.cc
+++ b/src/bin/dhcp6/tests/addr_reg_unittest.cc
@@ -1915,11 +1915,15 @@ TEST_F(AddrRegTest, allowAddressRegistrationFalse) {
     // Verify no response.
     ASSERT_FALSE(srv_->processAddrRegInform(ctx));
 
-    // Drop stat should have been bumped by one and the log emitted.
+    // Admin-filtered and drop stat should have been bumped by one and the log emitted.
     ObservationPtr stat;
+    stat = StatsMgr::instance().getObservation("pkt6-admin-filtered");
+    ASSERT_TRUE(stat);
+    EXPECT_EQ(1, stat->getInteger().first);
     stat = StatsMgr::instance().getObservation("pkt6-receive-drop");
     ASSERT_TRUE(stat);
     EXPECT_EQ(1, stat->getInteger().first);
+
     EXPECT_EQ(1, countFile("DHCP6_ADDR6_REGISTER_DISABLED_DROP ADDR-REG-INFORM"));
 }