[PATCH] lsm: fix a missing security_uring_allowed() prototype

[PATCH] lsm: fix a missing security_uring_allowed() prototype

[Paul Moore]

The !CONFIG_SECURITY dummy function was missing an inline attribute which
caused the compiler to complain about a missing prototype.  This patch
adds the missing inline attribute.

Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Closes: https://lore.kernel.org/oe-kbuild-all/202502081912.TeokpAAe-lkp@intel.com/
Fixes: c6ad9fdbd44b ("io_uring,lsm,selinux: add LSM hooks for io_uring_setup()")
Signed-off-by: Paul Moore <paul@paul-moore.com>
---
 include/linux/security.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/security.h b/include/linux/security.h
index 3e68f8468a22..fab843d5e621 100644
--- a/include/linux/security.h
+++ b/include/linux/security.h
@@ -2376,7 +2376,7 @@ static inline int security_uring_cmd(struct io_uring_cmd *ioucmd)
 {
 	return 0;
 }
-extern int security_uring_allowed(void)
+extern inline int security_uring_allowed(void)
 {
 	return 0;
 }
-- 
2.48.1

Re: [PATCH] lsm: fix a missing security_uring_allowed() prototype

[Paul Moore]

On Sun, Feb 9, 2025 at 10:41 PM Paul Moore <paul@paul-moore.com> wrote:
>
> The !CONFIG_SECURITY dummy function was missing an inline attribute which
> caused the compiler to complain about a missing prototype.  This patch
> adds the missing inline attribute.
>
> Reported-by: kernel test robot <lkp@intel.com>
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> Closes: https://lore.kernel.org/oe-kbuild-all/202502081912.TeokpAAe-lkp@intel.com/
> Fixes: c6ad9fdbd44b ("io_uring,lsm,selinux: add LSM hooks for io_uring_setup()")
> Signed-off-by: Paul Moore <paul@paul-moore.com>
> ---
>  include/linux/security.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/security.h b/include/linux/security.h
> index 3e68f8468a22..fab843d5e621 100644
> --- a/include/linux/security.h
> +++ b/include/linux/security.h
> @@ -2376,7 +2376,7 @@ static inline int security_uring_cmd(struct io_uring_cmd *ioucmd)
>  {
>         return 0;
>  }
> -extern int security_uring_allowed(void)
> +extern inline int security_uring_allowed(void)

The extern also needed to be changed out, essentially doing s/extern
int/static inline/.  Regardless, this is now merged into lsm/dev.

>  {
>         return 0;
>  }
> --
> 2.48.1

-- 
paul-moore.com