From: Serge E. Hallyn <serue@us.ibm.com>
Date: Sun, 24 Feb 2008 02:10:07 +0000 (+0000)
Subject: file capabilities: simplify signal check
X-Git-Tag: v2.6.24.4~45
X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=2628814b59f33d6a1aae535adc7ef44359aafe98;p=karo-tx-linux.git

file capabilities: simplify signal check

commit: 094972840f2e7c1c6fc9e1a97d817cc17085378e

Simplify the uid equivalence check in cap_task_kill().  Anyone can kill a
process owned by the same uid.

Without this patch wireshark is reported to fail.

Signed-off-by: Serge E. Hallyn <serue@us.ibm.com>
Signed-off-by: Andrew G. Morgan <morgan@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Chris Wright <chrisw@sous-sol.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
---

diff --git a/security/commoncap.c b/security/commoncap.c
index ea61bc73f6d3..e87422e766dd 100644
--- a/security/commoncap.c
+++ b/security/commoncap.c
@@ -539,7 +539,7 @@ int cap_task_kill(struct task_struct *p, struct siginfo *info,
 	 * allowed.
 	 * We must preserve legacy signal behavior in this case.
 	 */
-	if (p->euid == 0 && p->uid == current->uid)
+	if (p->uid == current->uid)
 		return 0;
 
 	/* sigcont is permitted within same session */