Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit fed8d8df authored by Dinghao Liu's avatar Dinghao Liu Committed by Greg Kroah-Hartman
Browse files

net: ethernet: Fix memleak in ethoc_probe 



[ Upstream commit 5d41f9b7ee7a5a5138894f58846a4ffed601498a ]

When mdiobus_register() fails, priv->mdio allocated
by mdiobus_alloc() has not been freed, which leads
to memleak.

Fixes: e7f4dc35 ("mdio: Move allocation of interrupts into core")
Signed-off-by: default avatarDinghao Liu <dinghao.liu@zju.edu.cn>
Reviewed-by: default avatarAndrew Lunn <andrew@lunn.ch>
Link: https://lore.kernel.org/r/20201223110615.31389-1-dinghao.liu@zju.edu.cn


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 8837315f

drivers/net/ethernet/ethoc.c

+2 −1
Original line number Diff line number Diff line
@@ -1190,7 +1190,7 @@ static int ethoc_probe(struct platform_device *pdev) 
	ret = mdiobus_register(priv->mdio);

	if (ret) {

		dev_err(&netdev->dev, "failed to register MDIO bus\n");

		goto free2;

		goto free3;

	}



	ret = ethoc_mdio_probe(netdev);
@@ -1222,6 +1222,7 @@ static int ethoc_probe(struct platform_device *pdev) 
	netif_napi_del(&priv->napi);

error:

	mdiobus_unregister(priv->mdio);

free3:

	mdiobus_free(priv->mdio);

free2:

	if (priv->clk)