diff options
| author | KOSAKI Motohiro <[email protected]> | 2010-10-14 19:21:18 +0000 |
|---|---|---|
| committer | James Morris <[email protected]> | 2010-10-20 23:12:44 +0000 |
| commit | b0ae19811375031ae3b3fecc65b702a9c6e5cc28 (patch) | |
| tree | a765b71155fbed1ed3a3cff35c1044ad49a002ae /security/selinux/hooks.c | |
| parent | tpm: change 'tpm_suspend_pcr' to be module parameter (diff) | |
| download | kernel-b0ae19811375031ae3b3fecc65b702a9c6e5cc28.tar.gz kernel-b0ae19811375031ae3b3fecc65b702a9c6e5cc28.zip | |
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 <[email protected]>
Signed-off-by: KOSAKI Motohiro <[email protected]>
Signed-off-by: James Morris <[email protected]>
Diffstat (limited to 'security/selinux/hooks.c')
| -rw-r--r-- | security/selinux/hooks.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c index 4796ddd4e721..db2b331de89a 100644 --- a/security/selinux/hooks.c +++ b/security/selinux/hooks.c @@ -3354,11 +3354,11 @@ static int selinux_task_setrlimit(struct task_struct *p, unsigned int resource, return 0; } -static int selinux_task_setscheduler(struct task_struct *p, int policy, struct sched_param *lp) +static int selinux_task_setscheduler(struct task_struct *p) { int rc; - rc = cap_task_setscheduler(p, policy, lp); + rc = cap_task_setscheduler(p); if (rc) return rc; |