diff options
| author | Phil Sutter <phil@nwl.cc> | 2018-09-19 15:16:45 +0200 |
|---|---|---|
| committer | Florian Westphal <fw@strlen.de> | 2018-09-24 11:23:49 +0200 |
| commit | 1788f545aae285fa3cd6595d5d25b2ae1b215282 (patch) | |
| tree | 18856f67dce467c5f8598947506abb35a1187e39 /extensions/libxt_set.c | |
| parent | 31f1434dfe3770ecbdac1bacb8e0fc4a17b3d671 (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 'extensions/libxt_set.c')
| -rw-r--r-- | extensions/libxt_set.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/extensions/libxt_set.c b/extensions/libxt_set.c index 679c04c7..16921023 100644 --- a/extensions/libxt_set.c +++ b/extensions/libxt_set.c @@ -60,6 +60,7 @@ set_parse_v0(int c, char **argv, int invert, unsigned int *flags, case '2': fprintf(stderr, "--set option deprecated, please use --match-set\n"); + /* fall through */ case '1': /* --match-set <set> <flag>[,<flag> */ if (info->u.flags[0]) xtables_error(PARAMETER_PROBLEM, @@ -140,6 +141,7 @@ set_parse_v1(int c, char **argv, int invert, unsigned int *flags, case '2': fprintf(stderr, "--set option deprecated, please use --match-set\n"); + /* fall through */ case '1': /* --match-set <set> <flag>[,<flag> */ if (info->dim) xtables_error(PARAMETER_PROBLEM, @@ -238,6 +240,7 @@ set_parse_v2(int c, char **argv, int invert, unsigned int *flags, case '2': fprintf(stderr, "--set option deprecated, please use --match-set\n"); + /* fall through */ case '1': /* --match-set <set> <flag>[,<flag> */ if (info->dim) xtables_error(PARAMETER_PROBLEM, @@ -415,6 +418,7 @@ set_parse_v3(int c, char **argv, int invert, unsigned int *flags, case '2': fprintf(stderr, "--set option deprecated, please use --match-set\n"); + /* fall through */ case '1': /* --match-set <set> <flag>[,<flag> */ if (info->match_set.dim) xtables_error(PARAMETER_PROBLEM, @@ -583,6 +587,7 @@ set_parse_v4(int c, char **argv, int invert, unsigned int *flags, case '2': fprintf(stderr, "--set option deprecated, please use --match-set\n"); + /* fall through */ case '1': /* --match-set <set> <flag>[,<flag> */ if (info->match_set.dim) xtables_error(PARAMETER_PROBLEM, |