Re: [PATCH 1/1] Smack:- Remove redundant inode_smack cache since now new cache is being used for inode_smack structure allocations

From: Casey Schaufler
Date: Thu May 07 2020 - 18:26:43 EST


On 4/26/2020 9:12 PM, Vishal Goel wrote:
> Signed-off-by: Vishal Goel <vishal.goel@xxxxxxxxxxx>

I've taken this after adding a commit message.
Please include a commit message, even if you believe
the "subject line says it all" in the future.

> ---
> security/smack/smack_lsm.c | 9 +--------
> 1 file changed, 1 insertion(+), 8 deletions(-)
>
> diff --git a/security/smack/smack_lsm.c b/security/smack/smack_lsm.c
> index 2862fc3..d0f87d4 100644
> --- a/security/smack/smack_lsm.c
> +++ b/security/smack/smack_lsm.c
> @@ -55,7 +55,6 @@
> DEFINE_MUTEX(smack_ipv6_lock);
> static LIST_HEAD(smk_ipv6_port_list);
> #endif
> -static struct kmem_cache *smack_inode_cache;
> struct kmem_cache *smack_rule_cache;
> int smack_enabled;
>
> @@ -4841,15 +4840,9 @@ static __init int smack_init(void)
> struct cred *cred = (struct cred *) current->cred;
> struct task_smack *tsp;
>
> - smack_inode_cache = KMEM_CACHE(inode_smack, 0);
> - if (!smack_inode_cache)
> - return -ENOMEM;
> -
> smack_rule_cache = KMEM_CACHE(smack_rule, 0);
> - if (!smack_rule_cache) {
> - kmem_cache_destroy(smack_inode_cache);
> + if (!smack_rule_cache)
> return -ENOMEM;
> - }
>
> /*
> * Set the security state for the initial task.