diff options
| author | Phil Sutter <phil@nwl.cc> | 2022-09-29 19:11:55 +0200 |
|---|---|---|
| committer | Phil Sutter <phil@nwl.cc> | 2022-10-01 13:43:59 +0200 |
| commit | eafe731a50058ed59305ee4ab1ea2d63d6c4e86e (patch) | |
| tree | f82c07eb9198fdfb40e732f2be16c3f7b126c7e1 | |
| parent | d0e283469985515146bd158ab374307684f5f363 (diff) | |
extensions: among: Remove pointless fall through
This seems to be a leftover from an earlier version of the switch().
This fall through is never effective as the next case's code will never
apply. So just break instead.
Fixes: 26753888720d8 ("nft: bridge: Rudimental among extension support")
Signed-off-by: Phil Sutter <phil@nwl.cc>
| -rw-r--r-- | extensions/libebt_among.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/extensions/libebt_among.c b/extensions/libebt_among.c index 7eb898f9..c607a775 100644 --- a/extensions/libebt_among.c +++ b/extensions/libebt_among.c @@ -152,10 +152,9 @@ static int bramong_parse(int c, char **argv, int invert, xtables_error(PARAMETER_PROBLEM, "File should only contain one line"); optarg[flen-1] = '\0'; - /* fall through */ + break; case AMONG_DST: - if (c == AMONG_DST) - dst = true; + dst = true; /* fall through */ case AMONG_SRC: break; |