summaryrefslogtreecommitdiffstats
path: root/iptables/nft-shared.c
diff options
context:
space:
mode:
authorPhil Sutter <phil@nwl.cc>2018-09-19 15:16:45 +0200
committerFlorian Westphal <fw@strlen.de>2018-09-24 11:23:49 +0200
commit1788f545aae285fa3cd6595d5d25b2ae1b215282 (patch)
tree18856f67dce467c5f8598947506abb35a1187e39 /iptables/nft-shared.c
parent31f1434dfe3770ecbdac1bacb8e0fc4a17b3d671 (diff)
Mark fall through cases in switch() statements
Typical covscan complaint, non-empty fall throughs should be marked as such. There was but a single case which should break instead, namely in libebt_log.c: It is not critical, since the next case merely asserts 'invert' being zero (which can't be as it was checked before). But while being at it, introduce log_chk_inv() to consolidate the semantically equal cases for the various log types. Signed-off-by: Phil Sutter <phil@nwl.cc> Signed-off-by: Florian Westphal <fw@strlen.de>
Diffstat (limited to 'iptables/nft-shared.c')
-rw-r--r--iptables/nft-shared.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/iptables/nft-shared.c b/iptables/nft-shared.c
index c8414294..a38ffd3a 100644
--- a/iptables/nft-shared.c
+++ b/iptables/nft-shared.c
@@ -547,6 +547,7 @@ void nft_parse_immediate(struct nft_xt_ctx *ctx, struct nftnl_expr *e)
break;;
case NFT_GOTO:
nft_goto = true;
+ /* fall through */
case NFT_JUMP:
jumpto = chain;
break;