Commit 74560bb3 authored Mar 28, 2024 by Joel Granados Committed by Paul Moore Apr 15, 2024

lsm: remove the now superfluous sentinel element from ctl_table array

This commit comes at the tail end of a greater effort to remove the
empty elements at the end of the ctl_table arrays (sentinels) which will
reduce the overall build time size of the kernel and run time memory
bloat by ~64 bytes per sentinel (further information Link :
https://lore.kernel.org/all/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/

)

Remove the sentinel from all files under security/ that register a
sysctl table.

Signed-off-by: Joel Granados <j.granados@samsung.com>
Acked-by: Kees Cook <keescook@chromium.org> # loadpin & yama
Tested-by: Valentin Schneider <vschneid@redhat.com>
Reviewed-by: Valentin Schneider <vschneid@redhat.com>
[PM: subject line tweaks]
Signed-off-by: Paul Moore <paul@paul-moore.com>

parent 4cece764

security/apparmor/lsm.c

+0 −1

Original line number	Diff line number	Diff line
		@@ -2064,7 +2064,6 @@ static struct ctl_table apparmor_sysctl_table[] = {
		.mode = 0600,
		.proc_handler = apparmor_dointvec,
		},
		{ }
		};

		static int __init apparmor_init_sysctl(void)

security/keys/sysctl.c

+0 −1

Original line number	Diff line number	Diff line
		@@ -66,7 +66,6 @@ static struct ctl_table key_sysctls[] = {
		.extra2 = (void *) SYSCTL_INT_MAX,
		},
		#endif
		{ }
		};

		static int __init init_security_keys_sysctls(void)

security/loadpin/loadpin.c

+0 −1

Original line number	Diff line number	Diff line
		@@ -63,7 +63,6 @@ static struct ctl_table loadpin_sysctl_table[] = {
		.extra1 = SYSCTL_ONE,
		.extra2 = SYSCTL_ONE,
		},
		{ }
		};

		static void set_sysctl(bool is_writable)

security/yama/yama_lsm.c

+0 −1

Original line number	Diff line number	Diff line
		@@ -463,7 +463,6 @@ static struct ctl_table yama_sysctl_table[] = {
		.extra1 = SYSCTL_ZERO,
		.extra2 = &max_scope,
		},
		{ }
		};
		static void __init yama_init_sysctl(void)
		{