diff options
| author | Thomas Haller <thaller@redhat.com> | 2023-10-23 19:00:47 +0200 |
|---|---|---|
| committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2023-11-21 15:45:45 +0100 |
| commit | f0653192f47f8989ccd0d5ca62432ea255093b60 (patch) | |
| tree | 4f3584ff5cf3edc36eeb03e870bc7a309f88d6f4 /src/statement.c | |
| parent | e1c813c4137e15e8f28a937e8f9a00d1a6784838 (diff) | |
parser_bison: fix length check for ifname in ifname_expr_alloc()
commit 122dce6b35205a3df419a5cae9acfd6e83e8725a upstream.
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 'src/statement.c')
0 files changed, 0 insertions, 0 deletions