diff options
| author | Paul Moore <paul@paul-moore.com> | 2023-11-10 18:09:33 +0100 |
|---|---|---|
| committer | Paul Moore <paul@paul-moore.com> | 2023-11-13 04:54:42 +0100 |
| commit | b1a867eeb8ab5e097178728b01cc504c6806acca (patch) | |
| tree | 69e062069760bb4e30fcba6b3238281f64914b08 /security/bpf/hooks.c | |
| parent | lsm: convert security_setselfattr() to use memdup_user() (diff) | |
| download | linux-b1a867eeb8ab5e097178728b01cc504c6806acca.tar.xz linux-b1a867eeb8ab5e097178728b01cc504c6806acca.zip | |
lsm: mark the lsm_id variables are marked as static
As the kernel test robot helpfully reminded us, all of the lsm_id
instances defined inside the various LSMs should be marked as static.
The one exception is Landlock which uses its lsm_id variable across
multiple source files with an extern declaration in a header file.
Reported-by: kernel test robot <lkp@intel.com>
Suggested-by: Casey Schaufler <casey@schaufler-ca.com>
Reviewed-by: Casey Schaufler <casey@schaufler-ca.com>
Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'security/bpf/hooks.c')
| -rw-r--r-- | security/bpf/hooks.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/security/bpf/hooks.c b/security/bpf/hooks.c index 91011e0c361a..57b9ffd53c98 100644 --- a/security/bpf/hooks.c +++ b/security/bpf/hooks.c @@ -16,7 +16,7 @@ static struct security_hook_list bpf_lsm_hooks[] __ro_after_init = { LSM_HOOK_INIT(task_free, bpf_task_storage_free), }; -const struct lsm_id bpf_lsmid = { +static const struct lsm_id bpf_lsmid = { .name = "bpf", .id = LSM_ID_BPF, }; |