acl: fix an off-by-one error in fa_acl_match_ip6_addr which does masked IPv6 comparison 96/42596/2
authorAndrew Yourtchenko <ayourtch@gmail.com>
Wed, 26 Mar 2025 15:52:25 +0000 (16:52 +0100)
committerAndrew Yourtchenko <ayourtch@gmail.com>
Wed, 26 Mar 2025 16:01:36 +0000 (17:01 +0100)
The comparison code for the bit remainder (non-zero part of prefix length modulo 8)
was incorrectly looking one byte further than it should.

Type: fix
Change-Id: Idd27d218e77eff5f368f2ba0a5cefb86ecf605f5
Signed-off-by: Andrew Yourtchenko <ayourtch@gmail.com>
src/plugins/acl/public_inlines.h

index eb9f0de..80edfd6 100644 (file)
@@ -268,8 +268,8 @@ fa_acl_match_ip6_addr (ip6_address_t * addr1, ip6_address_t * addr2,
        }
       if (prefixlen % 8)
        {
-         u8 b1 = *((u8 *) addr1 + 1 + prefixlen / 8);
-         u8 b2 = *((u8 *) addr2 + 1 + prefixlen / 8);
+         u8 b1 = *((u8 *) addr1 + prefixlen / 8);
+         u8 b2 = *((u8 *) addr2 + prefixlen / 8);
          u8 mask0 = (0xff - ((1 << (8 - (prefixlen % 8))) - 1));
          return (b1 & mask0) == b2;
        }