Message ID | 20190114202835.15878-1-malat@debian.org (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | capabilities:: annotate implicit fall through | expand |
On 1/14/19 2:28 PM, Mathieu Malaterre wrote: > There is a plan to build the kernel with -Wimplicit-fallthrough and > this place in the code produced a warning (W=1). > > In this particular case change put the fall through comment on a single > line so as to match the regular expression expected by GCC. > > This commit remove the following warning: > > kernel/capability.c:95:3: warning: this statement may fall through [-Wimplicit-fallthrough=] > > Signed-off-by: Mathieu Malaterre <malat@debian.org> Acked-by: Gustavo A. R. Silva <gustavo@embeddedor.com> Thanks > --- > kernel/capability.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/kernel/capability.c b/kernel/capability.c > index 1e1c0236f55b..e60be18b98e2 100644 > --- a/kernel/capability.c > +++ b/kernel/capability.c > @@ -93,9 +93,7 @@ static int cap_validate_magic(cap_user_header_t header, unsigned *tocopy) > break; > case _LINUX_CAPABILITY_VERSION_2: > warn_deprecated_v2(); > - /* > - * fall through - v3 is otherwise equivalent to v2. > - */ > + /* fall through - v3 is otherwise equivalent to v2. */ > case _LINUX_CAPABILITY_VERSION_3: > *tocopy = _LINUX_CAPABILITY_U32S_3; > break; >
On Mon, 14 Jan 2019, Mathieu Malaterre wrote: > There is a plan to build the kernel with -Wimplicit-fallthrough and > this place in the code produced a warning (W=1). > > In this particular case change put the fall through comment on a single > line so as to match the regular expression expected by GCC. > > This commit remove the following warning: > > kernel/capability.c:95:3: warning: this statement may fall through [-Wimplicit-fallthrough=] > > Signed-off-by: Mathieu Malaterre <malat@debian.org> Applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general
diff --git a/kernel/capability.c b/kernel/capability.c index 1e1c0236f55b..e60be18b98e2 100644 --- a/kernel/capability.c +++ b/kernel/capability.c @@ -93,9 +93,7 @@ static int cap_validate_magic(cap_user_header_t header, unsigned *tocopy) break; case _LINUX_CAPABILITY_VERSION_2: warn_deprecated_v2(); - /* - * fall through - v3 is otherwise equivalent to v2. - */ + /* fall through - v3 is otherwise equivalent to v2. */ case _LINUX_CAPABILITY_VERSION_3: *tocopy = _LINUX_CAPABILITY_U32S_3; break;
There is a plan to build the kernel with -Wimplicit-fallthrough and this place in the code produced a warning (W=1). In this particular case change put the fall through comment on a single line so as to match the regular expression expected by GCC. This commit remove the following warning: kernel/capability.c:95:3: warning: this statement may fall through [-Wimplicit-fallthrough=] Signed-off-by: Mathieu Malaterre <malat@debian.org> --- kernel/capability.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)