[RFC PATCH 1/2] security: Remove security_add_mnt_opt() as it's unused

Paul Moore paul at paul-moore.com
Wed Dec 8 18:41:31 UTC 2021


On Wed, Dec 8, 2021 at 1:25 PM David Howells <dhowells at redhat.com> wrote:
>
> Remove the add_mnt_opt LSM hook as it's not actually used.  This makes it
> easier to make the context pointers in selinux_mnt_opts non-const.
>
> Signed-off-by: David Howells <dhowells at redhat.com>
> cc: Alexander Viro <viro at zeniv.linux.org.uk>
> cc: Paul Moore <paul at paul-moore.com>
> cc: Casey Schaufler <casey at schaufler-ca.com>
> cc: selinux at vger.kernel.org
> cc: linux-security-module at vger.kernel.org
> cc: linux-nfs at vger.kernel.org
> cc: linux-cachefs at redhat.com
> ---
>
>  include/linux/lsm_hook_defs.h |    2 --
>  include/linux/lsm_hooks.h     |    2 --
>  include/linux/security.h      |    8 --------
>  security/security.c           |    8 --------
>  security/selinux/hooks.c      |   39 ---------------------------------------
>  5 files changed, 59 deletions(-)

There is already a patch in the selinux/next tree which does this.

  commit 52f982f00b220d097a71a23c149a1d18efc08e63
  Author: Ondrej Mosnacek <omosnace at redhat.com>
  Date:   Mon Dec 6 14:24:06 2021 +0100

   security,selinux: remove security_add_mnt_opt()

   Its last user has been removed in commit f2aedb713c28 ("NFS: Add
   fs_context support.").

   Signed-off-by: Ondrej Mosnacek <omosnace at redhat.com>
   Reviewed-by: Casey Schaufler <casey at schaufler-ca.com>
   Signed-off-by: Paul Moore <paul at paul-moore.com>

-- 
paul moore
www.paul-moore.com



More information about the Linux-security-module-archive mailing list