lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 15 Aug 2018 15:13:28 +0100 From: Daniel Thompson <daniel.thompson@...aro.org> To: "Gustavo A. R. Silva" <gustavo@...eddedor.com>, Jason Wessel <jason.wessel@...driver.com> Cc: kgdb-bugreport@...ts.sourceforge.net, linux-kernel@...r.kernel.org Subject: Re: [PATCH] kdb: kdb_keyboard: mark expected switch fall-throughs On 05/08/18 05:18, Gustavo A. R. Silva wrote: > In preparation to enabling -Wimplicit-fallthrough, mark switch cases > where we are expecting to fall through. > > Notice that in this particular case, I replaced the code comments with > a proper "fall through" annotation, which is what GCC is expecting > to find. > > Signed-off-by: Gustavo A. R. Silva <gustavo@...eddedor.com> Reviewed-by: Daniel Thompson <daniel.thompson@...aro.org> > --- > kernel/debug/kdb/kdb_keyboard.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/kernel/debug/kdb/kdb_keyboard.c b/kernel/debug/kdb/kdb_keyboard.c > index 118527a..750497b 100644 > --- a/kernel/debug/kdb/kdb_keyboard.c > +++ b/kernel/debug/kdb/kdb_keyboard.c > @@ -173,11 +173,11 @@ int kdb_get_kbd_char(void) > case KT_LATIN: > if (isprint(keychar)) > break; /* printable characters */ > - /* drop through */ > + /* fall through */ > case KT_SPEC: > if (keychar == K_ENTER) > break; > - /* drop through */ > + /* fall through */ > default: > return -1; /* ignore unprintables */ > } >
Powered by blists - more mailing lists