diff options
author | Gustavo A. R. Silva <gustavoars@kernel.org> | 2020-11-19 14:11:44 +0100 |
---|---|---|
committer | Gustavo A. R. Silva <gustavoars@kernel.org> | 2020-11-19 14:23:47 +0100 |
commit | 36f9ff9e03de89691274a6aec45aa079bd3ae405 (patch) | |
tree | 79c1e0a1bae6c0463276f4294c4440e0d8c6d78d /lib/nlattr.c | |
parent | powerpc: fix -Wimplicit-fallthrough (diff) | |
download | linux-36f9ff9e03de89691274a6aec45aa079bd3ae405.tar.xz linux-36f9ff9e03de89691274a6aec45aa079bd3ae405.zip |
lib: Fix fall-through warnings for Clang
In preparation to enable -Wimplicit-fallthrough for Clang, fix multiple
warnings by explicitly adding multiple break statements instead of
letting the code fall through to the next case, and by replacing a
number of /* fall through */ comments with the new pseudo-keyword
macro fallthrough.
Notice that Clang doesn't recognize /* Fall through */ comments as
implicit fall-through markings.
Link: https://github.com/KSPP/linux/issues/115
Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Diffstat (limited to 'lib/nlattr.c')
-rw-r--r-- | lib/nlattr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/nlattr.c b/lib/nlattr.c index 74019c8ebf6b..19372355557a 100644 --- a/lib/nlattr.c +++ b/lib/nlattr.c @@ -432,7 +432,7 @@ static int validate_nla(const struct nlattr *nla, int maxtype, err = -EINVAL; goto out_err; } - /* fall through */ + fallthrough; case NLA_STRING: if (attrlen < 1) |