Re: [PATCH] mmc: mmc_spi: Adjust error handling in mmc_spi_probe()

From: Andy Shevchenko
Date: Wed Dec 27 2023 - 11:57:04 EST


On Wed, Dec 27, 2023 at 12:50:50PM +0100, Markus Elfring wrote:
> From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
> Date: Wed, 27 Dec 2023 12:23:20 +0100
>
> The kfree() function was called in one case by
> the mmc_spi_probe() function during error handling
> even if the passed variable contained a null pointer.
> This issue was detected by using the Coccinelle software.

> * Thus return directly after a call of the function “kmalloc” failed
> at the beginning.
>
> * Move an error code assignment into an if branch.

How is this one better?

--
With Best Regards,
Andy Shevchenko