diff options
author | Dinghao Liu <dinghao.liu@zju.edu.cn> | 2020-12-23 12:06:12 +0100 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2020-12-23 21:28:53 +0100 |
commit | 5d41f9b7ee7a5a5138894f58846a4ffed601498a (patch) | |
tree | be65b2a5398e5abf061eca0de232d8ba757d55d7 /drivers/net/ethernet/ethoc.c | |
parent | net/ncsi: Use real net-device for response handler (diff) | |
download | linux-5d41f9b7ee7a5a5138894f58846a4ffed601498a.tar.xz linux-5d41f9b7ee7a5a5138894f58846a4ffed601498a.zip |
net: ethernet: Fix memleak in ethoc_probe
When mdiobus_register() fails, priv->mdio allocated
by mdiobus_alloc() has not been freed, which leads
to memleak.
Fixes: e7f4dc3536a4 ("mdio: Move allocation of interrupts into core")
Signed-off-by: Dinghao Liu <dinghao.liu@zju.edu.cn>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Link: https://lore.kernel.org/r/20201223110615.31389-1-dinghao.liu@zju.edu.cn
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'drivers/net/ethernet/ethoc.c')
-rw-r--r-- | drivers/net/ethernet/ethoc.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/ethernet/ethoc.c b/drivers/net/ethernet/ethoc.c index 0981fe9652e5..3d9b0b161e24 100644 --- a/drivers/net/ethernet/ethoc.c +++ b/drivers/net/ethernet/ethoc.c @@ -1211,7 +1211,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); @@ -1243,6 +1243,7 @@ error2: netif_napi_del(&priv->napi); error: mdiobus_unregister(priv->mdio); +free3: mdiobus_free(priv->mdio); free2: clk_disable_unprepare(priv->clk); |