[PATCH] mtd: spi-nor: aspeed-smc: Add of_node_put()
Tudor.Ambarus at microchip.com
Tudor.Ambarus at microchip.com
Wed Aug 28 20:20:11 AEST 2019
On 08/08/2019 10:51 AM, Nishka Dasgupta wrote:
> External E-Mail
>
>
> Each iteration of for_each_available_child_of_node puts the previous
> node, but in the case of a break from the middle of the loop, there is
> no put, thus causing a memory leak. Upon termination of the loop
> (whether by break or a natural exit), either ret will have a non-zero
> value or child will be NULL. Hence add an of_node_put() that will
> execute only when ret has a non-zero value, as calling of_node_put() on
> a possible NULL value does not cause any further issues.
> Issue found with Coccinelle.
>
> Signed-off-by: Nishka Dasgupta <nishkadg.linux at gmail.com>
> ---
> drivers/mtd/spi-nor/aspeed-smc.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git,
spi-nor/next branch.
Thanks,
ta
More information about the Linux-aspeed
mailing list