diff options
| author | Andrew Goodbody <andrew.goodbody@linaro.org> | 2025-08-01 10:54:04 +0100 |
|---|---|---|
| committer | Michael Trimarchi <michael@amarulasolutions.com> | 2025-08-03 10:22:04 +0200 |
| commit | ebf8aaf7afdeb306b0fb85be2226a00aef18162f (patch) | |
| tree | c5c8f4fbdf9aeb36e8e12a50caa2cfd81ba8ba91 | |
| parent | 56372f834c717402882112ba888943421020b6d5 (diff) | |
mtd: rawnand: cortina_nand: Fix -ENOMEM detection
In init_nand_dma there was code to detect failure to allocate memory but
it had two problems. Firstly the 2nd clause when info->tx_desc was NULL
attempted to free info->tx_desc when it should be freeing info->rx_desc.
Secondly there was no detection of both allocations failing, arguably
the more likely scenario. Refactor the code to simplify it and just fail
as soon as either allocation fails.
This issue was found by Smatch.
Signed-off-by: Andrew Goodbody <andrew.goodbody@linaro.org>
Reviewed-by: Michael Trimarchi <michael@amarulasolutions.com>
| -rw-r--r-- | drivers/mtd/nand/raw/cortina_nand.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/drivers/mtd/nand/raw/cortina_nand.c b/drivers/mtd/nand/raw/cortina_nand.c index 06918a46e93..a27a6994f56 100644 --- a/drivers/mtd/nand/raw/cortina_nand.c +++ b/drivers/mtd/nand/raw/cortina_nand.c @@ -186,14 +186,13 @@ int init_nand_dma(struct nand_chip *nand) info->tx_desc = malloc_cache_aligned((sizeof(struct tx_descriptor_t) * CA_DMA_DESC_NUM)); - info->rx_desc = malloc_cache_aligned((sizeof(struct rx_descriptor_t) * - CA_DMA_DESC_NUM)); - - if (!info->rx_desc && info->tx_desc) { + if (!info->tx_desc) { printf("Fail to alloc DMA descript!\n"); - kfree(info->tx_desc); return -ENOMEM; - } else if (info->rx_desc && !info->tx_desc) { + } + info->rx_desc = malloc_cache_aligned((sizeof(struct rx_descriptor_t) * + CA_DMA_DESC_NUM)); + if (!info->rx_desc) { printf("Fail to alloc DMA descript!\n"); kfree(info->tx_desc); return -ENOMEM; |