diff options
author | Pablo Neira Ayuso <pablo@netfilter.org> | 2019-06-07 13:37:22 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2019-06-07 13:58:07 +0200 |
commit | df2f746fb4cfc68d178dcfcf1fe9db08bd56b660 (patch) | |
tree | 45249111f70f49311e3b2290b8f2873593c3498b /src | |
parent | bbe139fdf5a589499e75d8361efbca6c4add3275 (diff) |
libnftables: keep evaluating until parser_max_errors
Bail out after parser_max_errors has been reached, eg.
# nft -f /tmp/errors.nft
/tmp/errors.nft:1:23-23: Error: syntax error, unexpected newline
filter input tcp dport
^
/tmp/errors.nft:2:24-26: Error: datatype mismatch, expected internet network service, expression has type Internet protocol
filter input tcp dport tcp
~~~~~~~~~ ^^^
/tmp/errors.nft:3:24-26: Error: datatype mismatch, expected internet network service, expression has type Internet protocol
filter input tcp sport udp
~~~~~~~~~ ^^^
Fixes: f211921e25e6 ("src: perform evaluation after parsing")
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src')
-rw-r--r-- | src/libnftables.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/libnftables.c b/src/libnftables.c index eae78e8b..e9dc03cf 100644 --- a/src/libnftables.c +++ b/src/libnftables.c @@ -393,7 +393,8 @@ static int nft_evaluate(struct nft_ctx *nft, struct list_head *msgs, .nft = nft, .msgs = msgs, }; - if (cmd_evaluate(&ectx, cmd) < 0) + if (cmd_evaluate(&ectx, cmd) < 0 && + ++nft->state->nerrs == nft->parser_max_errors) return -1; } |