Revert "WORKAROUND: security: smack: Allow ptracing even processes in onlycap set" 16/241916/1 accepted/tizen/unified/20200824.030012 submit/tizen/20200820.145124
authorSeung-Woo Kim <sw0312.kim@samsung.com>
Fri, 21 Aug 2020 01:08:34 +0000 (10:08 +0900)
committerSeung-Woo Kim <sw0312.kim@samsung.com>
Fri, 21 Aug 2020 01:09:33 +0000 (10:09 +0900)
This reverts commit 38b1ae4b960c310b02d7d6935b97d289f0326800.

Now crash-worker has System::Privileged privilege, so not anymore
ptrace related workaround is necessary. So revert the workaround.

Change-Id: I42b535e4152eb742df011a5a08324764b49f989a
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
security/smack/smack_lsm.c

index 7c5adf2..12c0fa8 100644 (file)
@@ -442,8 +442,6 @@ static int smk_ptrace_rule_check(struct task_struct *tracer,
                        rc = 0;
                else if (smack_ptrace_rule == SMACK_PTRACE_DRACONIAN)
                        rc = -EACCES;
-               else if (smack_ptrace_rule == SMACK_PTRACE_EXACT)
-                       rc = capable(CAP_SYS_PTRACE) != 0 ? 0 : -EACCES;
                else if (smack_privileged_cred(CAP_SYS_PTRACE, tracercred))
                        rc = 0;
                else