diff options
author | KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com> | 2010-10-14 15:21:18 -0400 |
---|---|---|
committer | James Morris <jmorris@namei.org> | 2010-10-20 19:12:44 -0400 |
commit | b0ae19811375031ae3b3fecc65b702a9c6e5cc28 (patch) | |
tree | a765b71155fbed1ed3a3cff35c1044ad49a002ae /kernel/sched.c | |
parent | 9b3056cca09529d34af2d81305b2a9c6b622ca1b (diff) |
security: remove unused parameter from security_task_setscheduler()
All security modules shouldn't change sched_param parameter of
security_task_setscheduler(). This is not only meaningless, but also
make a harmful result if caller pass a static variable.
This patch remove policy and sched_param parameter from
security_task_setscheduler() becuase none of security module is
using it.
Cc: James Morris <jmorris@namei.org>
Signed-off-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com>
Signed-off-by: James Morris <jmorris@namei.org>
Diffstat (limited to 'kernel/sched.c')
-rw-r--r-- | kernel/sched.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/sched.c b/kernel/sched.c index dc85ceb90832..df6579d9b4df 100644 --- a/kernel/sched.c +++ b/kernel/sched.c | |||
@@ -4645,7 +4645,7 @@ recheck: | |||
4645 | } | 4645 | } |
4646 | 4646 | ||
4647 | if (user) { | 4647 | if (user) { |
4648 | retval = security_task_setscheduler(p, policy, param); | 4648 | retval = security_task_setscheduler(p); |
4649 | if (retval) | 4649 | if (retval) |
4650 | return retval; | 4650 | return retval; |
4651 | } | 4651 | } |
@@ -4887,7 +4887,7 @@ long sched_setaffinity(pid_t pid, const struct cpumask *in_mask) | |||
4887 | if (!check_same_owner(p) && !capable(CAP_SYS_NICE)) | 4887 | if (!check_same_owner(p) && !capable(CAP_SYS_NICE)) |
4888 | goto out_unlock; | 4888 | goto out_unlock; |
4889 | 4889 | ||
4890 | retval = security_task_setscheduler(p, 0, NULL); | 4890 | retval = security_task_setscheduler(p); |
4891 | if (retval) | 4891 | if (retval) |
4892 | goto out_unlock; | 4892 | goto out_unlock; |
4893 | 4893 | ||