diff options
| author | Subasri S <subasris1210@gmail.com> | 2026-06-25 19:01:24 +0530 |
|---|---|---|
| committer | Florian Westphal <fw@strlen.de> | 2026-07-02 12:17:14 +0200 |
| commit | 5efbced92ec19514528ce6b18ae8e1b755cc4552 (patch) | |
| tree | d48b1cd5a22577a6ad89895ec94d09274cc005e0 | |
| parent | 1501ab0701fdd4571658e5829a2178f1af1f3917 (diff) | |
| download | linux-stable-5efbced92ec19514528ce6b18ae8e1b755cc4552.tar.gz linux-stable-5efbced92ec19514528ce6b18ae8e1b755cc4552.zip | |
netfilter: remove redundant null check before kvfree()
kvfree() internally performs NULL check on the pointer
handed to it and takes no action if it indeed is NULL.
Hence there is no need for a pre-check of the memory
pointer before handing it to kvfree().
Issue reported by ifnullfree.cocci Coccinelle semantic
patch script.
Signed-off-by: Subasri S <subasris1210@gmail.com>
Signed-off-by: Florian Westphal <fw@strlen.de>
| -rw-r--r-- | net/netfilter/nft_set_rbtree.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/netfilter/nft_set_rbtree.c b/net/netfilter/nft_set_rbtree.c index 018bbb6df4ce..efc25e788a1c 100644 --- a/net/netfilter/nft_set_rbtree.c +++ b/net/netfilter/nft_set_rbtree.c @@ -544,8 +544,7 @@ static int nft_array_intervals_alloc(struct nft_array *array, u32 max_intervals) if (!intervals) return -ENOMEM; - if (array->intervals) - kvfree(array->intervals); + kvfree(array->intervals); array->intervals = intervals; array->max_intervals = max_intervals; |