728-v6.1-04-net-ethernet-mtk_eth_soc-fix-resource-leak-in-error-.patch 1.2 KB

1234567891011121314151617181920212223242526272829303132333435
  1. From 8110437e59616293228cd781c486d8495a61e36a Mon Sep 17 00:00:00 2001
  2. From: Yan Cangang <[email protected]>
  3. Date: Sun, 20 Nov 2022 13:52:58 +0800
  4. Subject: [PATCH] net: ethernet: mtk_eth_soc: fix resource leak in error path
  5. In mtk_probe(), when mtk_ppe_init() or mtk_eth_offload_init() failed,
  6. mtk_mdio_cleanup() isn't called. Fix it.
  7. Fixes: ba37b7caf1ed ("net: ethernet: mtk_eth_soc: add support for initializing the PPE")
  8. Fixes: 502e84e2382d ("net: ethernet: mtk_eth_soc: add flow offloading support")
  9. Signed-off-by: Yan Cangang <[email protected]>
  10. Reviewed-by: Leon Romanovsky <[email protected]>
  11. Signed-off-by: Jakub Kicinski <[email protected]>
  12. ---
  13. drivers/net/ethernet/mediatek/mtk_eth_soc.c | 4 ++--
  14. 1 file changed, 2 insertions(+), 2 deletions(-)
  15. --- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c
  16. +++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
  17. @@ -4089,13 +4089,13 @@ static int mtk_probe(struct platform_dev
  18. eth->soc->offload_version, i);
  19. if (!eth->ppe[i]) {
  20. err = -ENOMEM;
  21. - goto err_free_dev;
  22. + goto err_deinit_mdio;
  23. }
  24. }
  25. err = mtk_eth_offload_init(eth);
  26. if (err)
  27. - goto err_free_dev;
  28. + goto err_deinit_mdio;
  29. }
  30. for (i = 0; i < MTK_MAX_DEVS; i++) {