diff options
author | Khadija Kamran <kamrankhadijadj@gmail.com> | 2023-08-23 11:16:40 +0300 |
---|---|---|
committer | Paul Moore <paul@paul-moore.com> | 2023-09-14 22:27:29 +0300 |
commit | a721f7b8c3548e943e514a957f2a37f4763b9888 (patch) | |
tree | ab176d981cc00bf0efb733183276c601e4967cfb /security/selinux/hooks.c | |
parent | 64fc9526147c7fc14535134d8ea79b9c8dc549a7 (diff) | |
download | linux-a721f7b8c3548e943e514a957f2a37f4763b9888.tar.xz |
lsm: constify 'bprm' parameter in security_bprm_committed_creds()
Three LSMs register the implementations for the 'bprm_committed_creds()'
hook: AppArmor, SELinux and tomoyo. Looking at the function
implementations we may observe that the 'bprm' parameter is not changing.
Mark the 'bprm' parameter of LSM hook security_bprm_committed_creds() as
'const' since it will not be changing in the LSM hook.
Signed-off-by: Khadija Kamran <kamrankhadijadj@gmail.com>
[PM: minor merge fuzzing due to other constification patches]
Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'security/selinux/hooks.c')
-rw-r--r-- | security/selinux/hooks.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c index f42a1b78bc43..e9ee008a9537 100644 --- a/security/selinux/hooks.c +++ b/security/selinux/hooks.c @@ -2501,7 +2501,7 @@ static void selinux_bprm_committing_creds(const struct linux_binprm *bprm) * Clean up the process immediately after the installation of new credentials * due to exec */ -static void selinux_bprm_committed_creds(struct linux_binprm *bprm) +static void selinux_bprm_committed_creds(const struct linux_binprm *bprm) { const struct task_security_struct *tsec = selinux_cred(current_cred()); u32 osid, sid; |