From: Kees Cook <keescook@chromium.org>
Date: Thu, 23 Feb 2017 17:24:24 +0000 (-0800)
Subject: seccomp: Clean up core dump logic
X-Git-Tag: v4.14-rc2~6^2~4^2~2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=131b63515932d18a3b1a60db3958f3c0dd5462bc;p=platform%2Fkernel%2Flinux-exynos.git

seccomp: Clean up core dump logic

This just cleans up the core dumping logic to avoid the braces around
the RET_KILL case.

Signed-off-by: Kees Cook <keescook@chromium.org>
---

diff --git a/kernel/seccomp.c b/kernel/seccomp.c
index 65f6107..fce8388 100644
--- a/kernel/seccomp.c
+++ b/kernel/seccomp.c
@@ -641,11 +641,12 @@ static int __seccomp_filter(int this_syscall, const struct seccomp_data *sd,
 		return 0;
 
 	case SECCOMP_RET_KILL:
-	default: {
-		siginfo_t info;
+	default:
 		audit_seccomp(this_syscall, SIGSYS, action);
 		/* Dump core only if this is the last remaining thread. */
 		if (get_nr_threads(current) == 1) {
+			siginfo_t info;
+
 			/* Show the original registers in the dump. */
 			syscall_rollback(current, task_pt_regs(current));
 			/* Trigger a manual coredump since do_exit skips it. */
@@ -654,7 +655,6 @@ static int __seccomp_filter(int this_syscall, const struct seccomp_data *sd,
 		}
 		do_exit(SIGSYS);
 	}
-	}
 
 	unreachable();