diff options
| author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2024-11-12 17:20:23 +0100 |
|---|---|---|
| committer | Paul E. McKenney <paulmck@kernel.org> | 2024-11-14 16:09:51 -0800 |
| commit | f946cae86d088d02a2f9c0ae0bf8a80359d3f454 (patch) | |
| tree | 57c59bb66311641766e43d793fa52426f5f18ae6 /kernel | |
| parent | 4788c861ad7e97d611e9b94596c896f5d0d072a6 (diff) | |
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')
| -rw-r--r-- | kernel/scftorture.c | 2 |
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); |