diff options
author | Thomas Haller <thaller@redhat.com> | 2023-10-23 19:00:47 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2023-10-24 11:31:02 +0200 |
commit | 122dce6b35205a3df419a5cae9acfd6e83e8725a (patch) | |
tree | bce5908adbb4606c3b266da440b3de77afb0f9de /tests | |
parent | f773041121d6d0d112fa9cb003fd791eacd6e43d (diff) |
parser_bison: fix length check for ifname in ifname_expr_alloc()
IFNAMSIZ is 16, and the allowed byte length of the name is one less than
that. Fix the length check and adjust a test for covering the longest
allowed interface name.
This is obviously a change in behavior, because previously interface
names with length 16 were accepted and were silently truncated along the
way. Now they are rejected as invalid.
Fixes: fa52bc225806 ("parser: reject zero-length interface names")
Signed-off-by: Thomas Haller <thaller@redhat.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'tests')
-rwxr-xr-x | tests/shell/testcases/chains/0042chain_variable_0 | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/tests/shell/testcases/chains/0042chain_variable_0 b/tests/shell/testcases/chains/0042chain_variable_0 index 739dc05a..a4b929f7 100755 --- a/tests/shell/testcases/chains/0042chain_variable_0 +++ b/tests/shell/testcases/chains/0042chain_variable_0 @@ -26,18 +26,13 @@ table netdev filter2 { rc=0 $NFT -f - <<< $EXPECTED || rc=$? -test "$rc" = 0 +test "$rc" = 1 cat <<EOF | $DIFF -u <($NFT list ruleset) - table netdev filter1 { chain Main_Ingress1 { type filter hook ingress device "lo" priority -500; policy accept; } } -table netdev filter2 { - chain Main_Ingress2 { - type filter hook ingress devices = { d23456789012345, lo } priority -500; policy accept; - } -} EOF |