|
 |
087bff |
From 5462c9908a3b2ba94fc4cf5c6cd0d5ed296093c5 Mon Sep 17 00:00:00 2001
|
|
|
087bff |
From: Phil Sutter <phil@nwl.cc>
|
|
|
087bff |
Date: Fri, 19 Feb 2021 16:54:57 +0100
|
|
|
087bff |
Subject: [PATCH] nft: Fix bitwise expression avoidance detection
|
|
|
087bff |
|
|
|
087bff |
Byte-boundary prefix detection was too sloppy: Any data following the
|
|
|
087bff |
first zero-byte was ignored. Add a follow-up loop making sure there are
|
|
|
087bff |
no stray bits in the designated host part.
|
|
|
087bff |
|
|
|
087bff |
Fixes: 323259001d617 ("nft: Optimize class-based IP prefix matches")
|
|
|
087bff |
Signed-off-by: Phil Sutter <phil@nwl.cc>
|
|
|
087bff |
(cherry picked from commit 330f5df03ad589b46865ceedf2a54cf10a4225ba)
|
|
|
087bff |
---
|
|
|
087bff |
iptables/nft-shared.c | 4 +++-
|
|
|
087bff |
.../testcases/ip6tables/0004-address-masks_0 | 24 +++++++++++++++++++
|
|
|
087bff |
2 files changed, 27 insertions(+), 1 deletion(-)
|
|
|
087bff |
create mode 100755 iptables/tests/shell/testcases/ip6tables/0004-address-masks_0
|
|
|
087bff |
|
|
|
087bff |
diff --git a/iptables/nft-shared.c b/iptables/nft-shared.c
|
|
|
087bff |
index 10553ab26823b..c1664b50f9383 100644
|
|
|
087bff |
--- a/iptables/nft-shared.c
|
|
|
087bff |
+++ b/iptables/nft-shared.c
|
|
|
087bff |
@@ -166,7 +166,7 @@ void add_addr(struct nftnl_rule *r, enum nft_payload_bases base, int offset,
|
|
|
087bff |
{
|
|
|
087bff |
const unsigned char *m = mask;
|
|
|
087bff |
bool bitwise = false;
|
|
|
087bff |
- int i;
|
|
|
087bff |
+ int i, j;
|
|
|
087bff |
|
|
|
087bff |
for (i = 0; i < len; i++) {
|
|
|
087bff |
if (m[i] != 0xff) {
|
|
|
087bff |
@@ -174,6 +174,8 @@ void add_addr(struct nftnl_rule *r, enum nft_payload_bases base, int offset,
|
|
|
087bff |
break;
|
|
|
087bff |
}
|
|
|
087bff |
}
|
|
|
087bff |
+ for (j = i + 1; !bitwise && j < len; j++)
|
|
|
087bff |
+ bitwise = !!m[j];
|
|
|
087bff |
|
|
|
087bff |
if (!bitwise)
|
|
|
087bff |
len = i;
|
|
|
087bff |
diff --git a/iptables/tests/shell/testcases/ip6tables/0004-address-masks_0 b/iptables/tests/shell/testcases/ip6tables/0004-address-masks_0
|
|
|
087bff |
new file mode 100755
|
|
|
087bff |
index 0000000000000..7eb42f08da975
|
|
|
087bff |
--- /dev/null
|
|
|
087bff |
+++ b/iptables/tests/shell/testcases/ip6tables/0004-address-masks_0
|
|
|
087bff |
@@ -0,0 +1,24 @@
|
|
|
087bff |
+#!/bin/bash
|
|
|
087bff |
+
|
|
|
087bff |
+set -e
|
|
|
087bff |
+
|
|
|
087bff |
+$XT_MULTI ip6tables-restore <
|
|
|
087bff |
+*filter
|
|
|
087bff |
+-A FORWARD -s feed:babe::/ffff::0
|
|
|
087bff |
+-A FORWARD -s feed:babe::/ffff:ff00::0
|
|
|
087bff |
+-A FORWARD -s feed:babe::/ffff:fff0::0
|
|
|
087bff |
+-A FORWARD -s feed:babe::/ffff:ffff::0
|
|
|
087bff |
+-A FORWARD -s feed:babe::/0:ffff::0
|
|
|
087bff |
+-A FORWARD -s feed:c0ff::babe:f00/ffff::ffff:0
|
|
|
087bff |
+COMMIT
|
|
|
087bff |
+EOF
|
|
|
087bff |
+
|
|
|
087bff |
+EXPECT='-P FORWARD ACCEPT
|
|
|
087bff |
+-A FORWARD -s feed::/16
|
|
|
087bff |
+-A FORWARD -s feed:ba00::/24
|
|
|
087bff |
+-A FORWARD -s feed:bab0::/28
|
|
|
087bff |
+-A FORWARD -s feed:babe::/32
|
|
|
087bff |
+-A FORWARD -s 0:babe::/0:ffff::
|
|
|
087bff |
+-A FORWARD -s feed::babe:0/ffff::ffff:0'
|
|
|
087bff |
+
|
|
|
087bff |
+diff -u -Z <(echo -e "$EXPECT") <($XT_MULTI ip6tables -S FORWARD)
|
|
|
087bff |
--
|
|
|
087bff |
2.31.1
|
|
|
087bff |
|