diff options
author | Todd Kjos <[email protected]> | 2021-10-12 09:56:13 -0700 |
---|---|---|
committer | Paul Moore <[email protected]> | 2021-10-14 20:48:04 -0400 |
commit | 52f88693378a58094c538662ba652aff0253c4fe (patch) | |
tree | b3b18e0e5162c6f0b1f0aeef6870249e800c2d4a /tools/perf/util/trace-event-scripting.c | |
parent | 29bc22ac5e5bc63275e850f0c8fc549e3d0e306b (diff) |
binder: use cred instead of task for selinux checks
Since binder was integrated with selinux, it has passed
'struct task_struct' associated with the binder_proc
to represent the source and target of transactions.
The conversion of task to SID was then done in the hook
implementations. It turns out that there are race conditions
which can result in an incorrect security context being used.
Fix by using the 'struct cred' saved during binder_open and pass
it to the selinux subsystem.
Cc: [email protected] # 5.14 (need backport for earlier stables)
Fixes: 79af73079d75 ("Add security hooks to binder and implement the hooks for SELinux.")
Suggested-by: Jann Horn <[email protected]>
Signed-off-by: Todd Kjos <[email protected]>
Acked-by: Casey Schaufler <[email protected]>
Signed-off-by: Paul Moore <[email protected]>
Diffstat (limited to 'tools/perf/util/trace-event-scripting.c')
0 files changed, 0 insertions, 0 deletions