diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2012-03-03 21:58:11 -0500 |
---|---|---|
committer | Eric W. Biederman <ebiederm@xmission.com> | 2012-04-07 19:55:05 -0400 |
commit | 74ba508f60c3650595297b33a4cdfd02e443194e (patch) | |
tree | 6acd9ed909b3ebf4f53c05e7336e41e8082d4f6a /kernel/sys.c | |
parent | 39fda8db80c9da028bc78ef444a51ea56f3cf0ad (diff) |
userns: Remove unnecessary cast to struct user_struct when copying cred->user.
In struct cred the user member is and has always been declared struct user_struct *user.
At most a constant struct cred will have a constant pointer to non-constant user_struct
so remove this unnecessary cast.
Acked-by: Serge Hallyn <serge.hallyn@canonical.com>
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Diffstat (limited to 'kernel/sys.c')
-rw-r--r-- | kernel/sys.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/sys.c b/kernel/sys.c index e7006eb6c1e4..f7a43514ac65 100644 --- a/kernel/sys.c +++ b/kernel/sys.c | |||
@@ -209,7 +209,7 @@ SYSCALL_DEFINE3(setpriority, int, which, int, who, int, niceval) | |||
209 | } while_each_pid_thread(pgrp, PIDTYPE_PGID, p); | 209 | } while_each_pid_thread(pgrp, PIDTYPE_PGID, p); |
210 | break; | 210 | break; |
211 | case PRIO_USER: | 211 | case PRIO_USER: |
212 | user = (struct user_struct *) cred->user; | 212 | user = cred->user; |
213 | if (!who) | 213 | if (!who) |
214 | who = cred->uid; | 214 | who = cred->uid; |
215 | else if ((who != cred->uid) && | 215 | else if ((who != cred->uid) && |
@@ -274,7 +274,7 @@ SYSCALL_DEFINE2(getpriority, int, which, int, who) | |||
274 | } while_each_pid_thread(pgrp, PIDTYPE_PGID, p); | 274 | } while_each_pid_thread(pgrp, PIDTYPE_PGID, p); |
275 | break; | 275 | break; |
276 | case PRIO_USER: | 276 | case PRIO_USER: |
277 | user = (struct user_struct *) cred->user; | 277 | user = cred->user; |
278 | if (!who) | 278 | if (!who) |
279 | who = cred->uid; | 279 | who = cred->uid; |
280 | else if ((who != cred->uid) && | 280 | else if ((who != cred->uid) && |