diff options
author | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2019-07-28 20:11:11 -0400 |
---|---|---|
committer | Gustavo A. R. Silva <gustavo@embeddedor.com> | 2019-08-09 20:47:15 -0400 |
commit | e9d81fc5b2014eb17d45ba4940bfb603d57bbcb3 (patch) | |
tree | 8fc28eef74b471da77b42d57c844f4136e48f7eb /arch/arm/kernel/signal.c | |
parent | 9039782047e769bd446667a9c14c0d3b8948ae51 (diff) |
ARM: signal: Mark expected switch fall-through
Mark switch cases where we are expecting to fall through.
This patch fixes the following warning:
arch/arm/kernel/signal.c: In function 'do_signal':
arch/arm/kernel/signal.c:598:12: warning: this statement may fall through [-Wimplicit-fallthrough=]
restart -= 2;
~~~~~~~~^~~~
arch/arm/kernel/signal.c:599:3: note: here
case -ERESTARTNOHAND:
^~~~
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Reviewed-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Gustavo A. R. Silva <gustavo@embeddedor.com>
Diffstat (limited to 'arch/arm/kernel/signal.c')
-rw-r--r-- | arch/arm/kernel/signal.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/arch/arm/kernel/signal.c b/arch/arm/kernel/signal.c index 09f6fdd41974..ab2568996ddb 100644 --- a/arch/arm/kernel/signal.c +++ b/arch/arm/kernel/signal.c | |||
@@ -596,6 +596,7 @@ static int do_signal(struct pt_regs *regs, int syscall) | |||
596 | switch (retval) { | 596 | switch (retval) { |
597 | case -ERESTART_RESTARTBLOCK: | 597 | case -ERESTART_RESTARTBLOCK: |
598 | restart -= 2; | 598 | restart -= 2; |
599 | /* Fall through */ | ||
599 | case -ERESTARTNOHAND: | 600 | case -ERESTARTNOHAND: |
600 | case -ERESTARTSYS: | 601 | case -ERESTARTSYS: |
601 | case -ERESTARTNOINTR: | 602 | case -ERESTARTNOINTR: |