]> git.ipfire.org Git - thirdparty/kea.git/commitdiff
[#2780] Fixed a crash in the FLQ tests
authorMarcin Siodelski <marcin@isc.org>
Fri, 14 Apr 2023 08:29:59 +0000 (10:29 +0200)
committerMarcin Siodelski <marcin@isc.org>
Fri, 14 Apr 2023 08:29:59 +0000 (10:29 +0200)
src/lib/dhcpsrv/tests/flq_allocator_unittest.cc

index 15c021b59ae0012e1b0335e54fa4c7ffad653ee9..5e60cf81b23c52bb5f18c238f3c598d33fe9fd6c 100644 (file)
@@ -26,13 +26,13 @@ public:
     /// @brief Creates a DHCPv4 lease for an address and MAC address.
     ///
     /// @param address Lease address.
-    /// @param hw_address_seed a seed from which the MAC address is generated.
+    /// @param hw_address_seed a seed from which the hardware address is generated.
     /// @return Created lease pointer.
     Lease4Ptr
     createLease4(const IOAddress& address, uint64_t hw_address_seed) const {
-        vector<uint8_t> hw_address_vec(6);
+        vector<uint8_t> hw_address_vec(sizeof(hw_address_seed));
         for (auto i = 0; i < sizeof(hw_address_seed); ++i) {
-            hw_address_vec[i] = hw_address_seed >> sizeof(hw_address_seed-1-i) & 0xFF;
+            hw_address_vec[i] = (hw_address_seed >> i) & 0xFF;
         }
         auto hw_address = boost::make_shared<HWAddr>(hw_address_vec, HTYPE_ETHER);
         auto lease = boost::make_shared<Lease4>(address, hw_address, ClientIdPtr(),
@@ -343,7 +343,7 @@ public:
     createLease6(Lease::Type type, const IOAddress& address, uint64_t duid_seed) const {
         vector<uint8_t> duid_vec(sizeof(duid_seed));
         for (auto i = 0; i < sizeof(duid_seed); ++i) {
-            duid_vec[i] = duid_seed >> sizeof(duid_seed-1-i) & 0xFF;
+            duid_vec[i] = (duid_seed >> i) & 0xFF;
         }
         auto duid = boost::make_shared<DUID>(duid_vec);
         auto lease = boost::make_shared<Lease6>(type, address, duid, 1, 1800,