Messages in this thread Patch in this message | | | From | Christophe JAILLET <> | Subject | [PATCH] net: bcmgenet: Fix a resource leak in an error handling path in the probe functin | Date | Sat, 12 Dec 2020 19:20:05 +0100 |
| |
If the 'register_netdev()' call fails, we must undo a previous 'bcmgenet_mii_init()' call.
Fixes: 1c1008c793fa ("net: bcmgenet: add main driver file") Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> --- The missing 'bcmgenet_mii_exit()' call is added here, instead of in the error handling path in order to avoid some goto spaghetti code. --- drivers/net/ethernet/broadcom/genet/bcmgenet.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/broadcom/genet/bcmgenet.c b/drivers/net/ethernet/broadcom/genet/bcmgenet.c index be85dad2e3bc..fcca023f22e5 100644 --- a/drivers/net/ethernet/broadcom/genet/bcmgenet.c +++ b/drivers/net/ethernet/broadcom/genet/bcmgenet.c @@ -4069,8 +4069,10 @@ static int bcmgenet_probe(struct platform_device *pdev) clk_disable_unprepare(priv->clk); err = register_netdev(dev); - if (err) + if (err) { + bcmgenet_mii_exit(dev); goto err; + } return err; -- 2.27.0
| |