diff options
author | Oleg Nesterov <oleg@redhat.com> | 2013-10-08 05:46:03 -0700 |
---|---|---|
committer | John Johansen <john.johansen@canonical.com> | 2013-10-29 21:34:18 -0700 |
commit | 51775fe736f053329faf0f5de7c679ee4cb0023d (patch) | |
tree | 03c7edaa4b4e6b3d78528769202661ce4861a832 /ipc/msgutil.c | |
parent | 4a7fc3018f05f4305723b508b12f3be13b7c4875 (diff) |
apparmor: remove the "task" arg from may_change_ptraced_domain()
Unless task == current ptrace_parent(task) is not safe even under
rcu_read_lock() and most of the current users are not right.
So may_change_ptraced_domain(task) looks wrong as well. However it
is always called with task == current so the code is actually fine.
Remove this argument to make this fact clear.
Note: perhaps we should simply kill ptrace_parent(), it buys almost
nothing. And it is obviously racy, perhaps this should be fixed.
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: John Johansen <john.johansen@canonical.com>
Diffstat (limited to 'ipc/msgutil.c')
0 files changed, 0 insertions, 0 deletions