Djam/u-boot
1
0
Fork 0
mirror of https://github.com/u-boot/u-boot.git synced 2025-04-21 04:14:34 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

ata: ahci: remove bad free

Browse source 
In the case of a memory allocation error, the ahci_port_start() function
tries to free the `pp' pointer.
This pointer was not dynamically allocated but does in fact point to an
element of the port[] array member of the struct ahci_uc_priv.
Remove the erroneous call to free() to fix this.

Fixes: 4782ac80b0 ("Add AHCI support to u-boot")
Signed-off-by: Vincent Stehlé <vincent.stehle@arm.com>
Cc: Tom Rini <trini@konsulko.com>
Cc: Jason Jin <jason.jin@freescale.com>
Reviewed-by: Andre Przywara <andre.przywara@arm.com>
This commit is contained in:
Vincent Stehlé 2025-03-24 09:34:06 +01:00 • committed by Tom Rini
parent 65504478fe
commit a345f44a60
1 changed files with 0 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
drivers/ata/ahci.c
View file

@ -463,7 +463,6 @@ static int ahci_port_start(struct ahci_uc_priv *uc_priv, u8 port)
mem = memalign(2048, AHCI_PORT_PRIV_DMA_SZ); mem = memalign(2048, AHCI_PORT_PRIV_DMA_SZ);
if (!mem) { if (!mem) {
free(pp);
printf("%s: No mem for table!\n", __func__); printf("%s: No mem for table!\n", __func__);
return -ENOMEM; return -ENOMEM;
} }