]> git.ipfire.org Git - thirdparty/nftables.git/commit
main: Fix for misleading error with negative chain priority
authorPhil Sutter <phil@nwl.cc>
Mon, 21 Oct 2019 16:51:14 +0000 (18:51 +0200)
committerPhil Sutter <phil@nwl.cc>
Mon, 21 Oct 2019 20:36:31 +0000 (22:36 +0200)
commit9fc71bc6b602c8706d1214e0100bcd7638c257e3
tree202156a0aa736d8d31f313d324602b67b9d5ec49
parente33b971f35cbfc3634dba9690537bbfb8baeac87
main: Fix for misleading error with negative chain priority

getopt_long() would try to parse the negative priority as an option and
return -1 as it is not known:

| # nft add chain x y { type filter hook input priority -30\; }
| nft: invalid option -- '3'

Fix this by prefixing optstring with a plus character. This instructs
getopt_long() to not collate arguments but just stop after the first
non-option, leaving the rest for manual handling. In fact, this is just
what nft desires: mixing options with nft syntax leads to confusive
command lines anyway.

Signed-off-by: Phil Sutter <phil@nwl.cc>
Acked-by: Pablo Neira Ayuso <pablo@netfilter.org>
src/main.c
tests/shell/testcases/chains/0039negative_priority_0 [new file with mode: 0755]