aboutsummaryrefslogtreecommitdiffstatshomepage
diff options
context:
space:
mode:
authorMohammad Shahid <mdshahid03@gmail.com>2026-07-03 20:18:51 +0530
committerAlison Schofield <alison.schofield@intel.com>2026-07-14 16:11:46 -0700
commit61f4eef68feee5076674c2d36749184afc3c9ffe (patch)
tree54189be5162ae1ceb407f306ec0e6d2aea89626b
parentLinux 7.2-rc3 (diff)
nvdimm: nfit: remove redundant NULL check before vfree()
vfree() safely handles NULL pointers, so the explicit NULL check before calling vfree() is unnecessary. This issue was reported by ifnullfree.cocci. Signed-off-by: Mohammad Shahid <mdshahid03@gmail.com> Reviewed-by: Dave Jiang <dave.jiang@intel.com> Link: https://patch.msgid.link/20260703144851.80309-1-mdshahid03@gmail.com Signed-off-by: Alison Schofield <alison.schofield@intel.com>
-rw-r--r--tools/testing/nvdimm/test/nfit.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/tools/testing/nvdimm/test/nfit.c b/tools/testing/nvdimm/test/nfit.c
index f87e9f251d13..009fe107b0d7 100644
--- a/tools/testing/nvdimm/test/nfit.c
+++ b/tools/testing/nvdimm/test/nfit.c
@@ -1644,8 +1644,7 @@ static void *__test_alloc(struct nfit_test *t, size_t size, dma_addr_t *dma,
err:
if (*dma && size >= DIMM_SIZE)
gen_pool_free(nfit_pool, *dma, size);
- if (buf)
- vfree(buf);
+ vfree(buf);
kfree(nfit_res);
return NULL;
}