* [PATCH] MIPS: CPS: Fix potential NULL pointer dereferences in cps_prepare_cpus()
@ 2025-04-07 16:32 Thorsten Blum
2025-04-27 8:15 ` Thomas Bogendoerfer
0 siblings, 1 reply; 2+ messages in thread
From: Thorsten Blum @ 2025-04-07 16:32 UTC (permalink / raw)
To: Thomas Bogendoerfer, Gregory CLEMENT, Dragan Mladjenovic,
Aleksandar Rikalo, Paul Burton, Chao-ying Fu
Cc: Thorsten Blum, stable, linux-mips, linux-kernel
Check the return values of kcalloc() and exit early to avoid potential
NULL pointer dereferences.
Compile-tested only.
Cc: stable@vger.kernel.org
Fixes: 75fa6a583882e ("MIPS: CPS: Introduce struct cluster_boot_config")
Fixes: 0856c143e1cd3 ("MIPS: CPS: Boot CPUs in secondary clusters")
Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
---
arch/mips/kernel/smp-cps.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/mips/kernel/smp-cps.c b/arch/mips/kernel/smp-cps.c
index e85bd087467e..cc26d56f3ab6 100644
--- a/arch/mips/kernel/smp-cps.c
+++ b/arch/mips/kernel/smp-cps.c
@@ -332,6 +332,8 @@ static void __init cps_prepare_cpus(unsigned int max_cpus)
mips_cps_cluster_bootcfg = kcalloc(nclusters,
sizeof(*mips_cps_cluster_bootcfg),
GFP_KERNEL);
+ if (!mips_cps_cluster_bootcfg)
+ goto err_out;
if (nclusters > 1)
mips_cm_update_property();
@@ -348,6 +350,8 @@ static void __init cps_prepare_cpus(unsigned int max_cpus)
mips_cps_cluster_bootcfg[cl].core_power =
kcalloc(BITS_TO_LONGS(ncores), sizeof(unsigned long),
GFP_KERNEL);
+ if (!mips_cps_cluster_bootcfg[cl].core_power)
+ goto err_out;
/* Allocate VPE boot configuration structs */
for (c = 0; c < ncores; c++) {
--
2.49.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] MIPS: CPS: Fix potential NULL pointer dereferences in cps_prepare_cpus()
2025-04-07 16:32 [PATCH] MIPS: CPS: Fix potential NULL pointer dereferences in cps_prepare_cpus() Thorsten Blum
@ 2025-04-27 8:15 ` Thomas Bogendoerfer
0 siblings, 0 replies; 2+ messages in thread
From: Thomas Bogendoerfer @ 2025-04-27 8:15 UTC (permalink / raw)
To: Thorsten Blum
Cc: Gregory CLEMENT, Dragan Mladjenovic, Aleksandar Rikalo,
Paul Burton, Chao-ying Fu, stable, linux-mips, linux-kernel
On Mon, Apr 07, 2025 at 06:32:21PM +0200, Thorsten Blum wrote:
> Check the return values of kcalloc() and exit early to avoid potential
> NULL pointer dereferences.
>
> Compile-tested only.
>
> Cc: stable@vger.kernel.org
> Fixes: 75fa6a583882e ("MIPS: CPS: Introduce struct cluster_boot_config")
> Fixes: 0856c143e1cd3 ("MIPS: CPS: Boot CPUs in secondary clusters")
> Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
> ---
> arch/mips/kernel/smp-cps.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/arch/mips/kernel/smp-cps.c b/arch/mips/kernel/smp-cps.c
> index e85bd087467e..cc26d56f3ab6 100644
> --- a/arch/mips/kernel/smp-cps.c
> +++ b/arch/mips/kernel/smp-cps.c
> @@ -332,6 +332,8 @@ static void __init cps_prepare_cpus(unsigned int max_cpus)
> mips_cps_cluster_bootcfg = kcalloc(nclusters,
> sizeof(*mips_cps_cluster_bootcfg),
> GFP_KERNEL);
> + if (!mips_cps_cluster_bootcfg)
> + goto err_out;
>
> if (nclusters > 1)
> mips_cm_update_property();
> @@ -348,6 +350,8 @@ static void __init cps_prepare_cpus(unsigned int max_cpus)
> mips_cps_cluster_bootcfg[cl].core_power =
> kcalloc(BITS_TO_LONGS(ncores), sizeof(unsigned long),
> GFP_KERNEL);
> + if (!mips_cps_cluster_bootcfg[cl].core_power)
> + goto err_out;
>
> /* Allocate VPE boot configuration structs */
> for (c = 0; c < ncores; c++) {
> --
> 2.49.0
applied to mips-fixes.
Thomas.
--
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea. [ RFC1925, 2.3 ]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-04-27 8:19 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-04-07 16:32 [PATCH] MIPS: CPS: Fix potential NULL pointer dereferences in cps_prepare_cpus() Thorsten Blum
2025-04-27 8:15 ` Thomas Bogendoerfer
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).