selinux: Remove unused function selinux_policycap_netif_wildcard()

[PATCH -next] selinux: Remove unused function selinux_policycap_netif_wildcard()

Posted by Yue Haibing 2 months, 2 weeks ago

This is unused since commit a3d3043ef24a ("selinux: get netif_wildcard
policycap from policy instead of cache").

Signed-off-by: Yue Haibing <yuehaibing@huawei.com>
---
 security/selinux/include/security.h | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/security/selinux/include/security.h b/security/selinux/include/security.h
index 8201e6a3ac0f..7f19972f7922 100644
--- a/security/selinux/include/security.h
+++ b/security/selinux/include/security.h
@@ -203,12 +203,6 @@ static inline bool selinux_policycap_netlink_xperm(void)
 		selinux_state.policycap[POLICYDB_CAP_NETLINK_XPERM]);
 }
 
-static inline bool selinux_policycap_netif_wildcard(void)
-{
-	return READ_ONCE(
-		selinux_state.policycap[POLICYDB_CAP_NETIF_WILDCARD]);
-}
-
 struct selinux_policy_convert_data;
 
 struct selinux_load_state {
-- 
2.34.1

Re: [PATCH] selinux: Remove unused function selinux_policycap_netif_wildcard()

Posted by Paul Moore 2 months ago

On Jul 23, 2025 Yue Haibing <yuehaibing@huawei.com> wrote:
> 
> This is unused since commit a3d3043ef24a ("selinux: get netif_wildcard
> policycap from policy instead of cache").
> 
> Signed-off-by: Yue Haibing <yuehaibing@huawei.com>
> Acked-by: Stephen Smalley <stephen.smalley.work@gmail.com>
> ---
>  security/selinux/include/security.h | 6 ------
>  1 file changed, 6 deletions(-)

Merged into selinux/dev-staging with plans to move it to selinux/dev
once the merge window closes.

--
paul-moore.com

Re: [PATCH] selinux: Remove unused function selinux_policycap_netif_wildcard()

Posted by Paul Moore 1 month, 3 weeks ago

On Tue, Aug 5, 2025 at 5:22 PM Paul Moore <paul@paul-moore.com> wrote:
> On Jul 23, 2025 Yue Haibing <yuehaibing@huawei.com> wrote:
> >
> > This is unused since commit a3d3043ef24a ("selinux: get netif_wildcard
> > policycap from policy instead of cache").
> >
> > Signed-off-by: Yue Haibing <yuehaibing@huawei.com>
> > Acked-by: Stephen Smalley <stephen.smalley.work@gmail.com>
> > ---
> >  security/selinux/include/security.h | 6 ------
> >  1 file changed, 6 deletions(-)
>
> Merged into selinux/dev-staging with plans to move it to selinux/dev
> once the merge window closes.

Now merged into selinux/dev, thanks!

-- 
paul-moore.com

Re: [PATCH -next] selinux: Remove unused function selinux_policycap_netif_wildcard()

Posted by Stephen Smalley 2 months, 1 week ago

On Wed, Jul 23, 2025 at 8:01 AM Yue Haibing <yuehaibing@huawei.com> wrote:
>
> This is unused since commit a3d3043ef24a ("selinux: get netif_wildcard
> policycap from policy instead of cache").
>
> Signed-off-by: Yue Haibing <yuehaibing@huawei.com>

Acked-by: Stephen Smalley <stephen.smalley.work@gmail.com>

> ---
>  security/selinux/include/security.h | 6 ------
>  1 file changed, 6 deletions(-)
>
> diff --git a/security/selinux/include/security.h b/security/selinux/include/security.h
> index 8201e6a3ac0f..7f19972f7922 100644
> --- a/security/selinux/include/security.h
> +++ b/security/selinux/include/security.h
> @@ -203,12 +203,6 @@ static inline bool selinux_policycap_netlink_xperm(void)
>                 selinux_state.policycap[POLICYDB_CAP_NETLINK_XPERM]);
>  }
>
> -static inline bool selinux_policycap_netif_wildcard(void)
> -{
> -       return READ_ONCE(
> -               selinux_state.policycap[POLICYDB_CAP_NETIF_WILDCARD]);
> -}
> -
>  struct selinux_policy_convert_data;
>
>  struct selinux_load_state {
> --
> 2.34.1
>