Re: [PATCH] net: xfrm: fix bug in ipcomp_free_scratches

From: Herbert Xu
Date: Mon Aug 16 2021 - 03:54:08 EST


On Mon, Aug 16, 2021 at 03:38:29PM +0800, Dongliang Mu wrote:
>
> - for_each_possible_cpu(i)
> - vfree(*per_cpu_ptr(scratches, i));
> + for_each_possible_cpu(i) {
> + void *scratch = *per_cpu_ptr(scratches, i);
> + if (!scratch)
> + vfree(scratch);
> + }

This patch is unnecessary. Please check the implementation of
vfree, it already checks for NULL pointers just like most of our
free primitives.

Cheers,
--
Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt