summaryrefslogtreecommitdiffstats
path: root/kernel/scftorture.c
diff options
context:
space:
mode:
authorSebastian Andrzej Siewior <bigeasy@linutronix.de>2024-11-12 17:20:23 +0100
committerPaul E. McKenney <paulmck@kernel.org>2024-11-15 01:09:51 +0100
commitf946cae86d088d02a2f9c0ae0bf8a80359d3f454 (patch)
tree57c59bb66311641766e43d793fa52426f5f18ae6 /kernel/scftorture.c
parentscftorture: Use a lock-less list to free memory. (diff)
downloadlinux-f946cae86d088d02a2f9c0ae0bf8a80359d3f454.tar.xz
linux-f946cae86d088d02a2f9c0ae0bf8a80359d3f454.zip
scftorture: Handle NULL argument passed to scf_add_to_free_list().
Dan reported that after the rework the newly introduced scf_add_to_free_list() may get a NULL pointer passed. This replaced kfree() which was fine with a NULL pointer but scf_add_to_free_list() isn't. Let scf_add_to_free_list() handle NULL pointer. Reported-by: Dan Carpenter <dan.carpenter@linaro.org> Closes: https://lore.kernel.org/all/2375aa2c-3248-4ffa-b9b0-f0a24c50f237@stanley.mountain Fixes: 4788c861ad7e9 ("scftorture: Use a lock-less list to free memory.") Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de> Signed-off-by: Paul E. McKenney <paulmck@kernel.org>
Diffstat (limited to 'kernel/scftorture.c')
-rw-r--r--kernel/scftorture.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/kernel/scftorture.c b/kernel/scftorture.c
index eeafd3fc1682..d86d2d9c4624 100644
--- a/kernel/scftorture.c
+++ b/kernel/scftorture.c
@@ -155,6 +155,8 @@ static void scf_add_to_free_list(struct scf_check *scfcp)
struct llist_head *pool;
unsigned int cpu;
+ if (!scfcp)
+ return;
cpu = raw_smp_processor_id() % nthreads;
pool = &per_cpu(scf_free_pool, cpu);
llist_add(&scfcp->scf_node, pool);