diff options
author | 2016-11-30 15:29:50 +0100 | |
---|---|---|
committer | 2016-12-02 10:42:45 -0500 | |
commit | 0a9e22715ee384cf2a714c28f24ce8881b9fd815 (patch) | |
tree | 469f708cd316b7597562218a4281ba64de329f8c /drivers/net | |
parent | net: ethernet: stmmac: dwmac-socfpga: fix use-after-free on probe errors (diff) | |
download | linux-0a9e22715ee384cf2a714c28f24ce8881b9fd815.tar.gz linux-0a9e22715ee384cf2a714c28f24ce8881b9fd815.tar.bz2 linux-0a9e22715ee384cf2a714c28f24ce8881b9fd815.zip |
net: ethernet: stmmac: dwmac-sti: fix probe error path
Make sure to disable clocks before returning on late probe errors.
Fixes: 8387ee21f972 ("stmmac: dwmac-sti: turn setup callback into a
probe function")
Signed-off-by: Johan Hovold <johan@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/ethernet/stmicro/stmmac/dwmac-sti.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/drivers/net/ethernet/stmicro/stmmac/dwmac-sti.c b/drivers/net/ethernet/stmicro/stmmac/dwmac-sti.c index 58c05acc2aab..a1ce018bf844 100644 --- a/drivers/net/ethernet/stmicro/stmmac/dwmac-sti.c +++ b/drivers/net/ethernet/stmicro/stmmac/dwmac-sti.c @@ -365,7 +365,16 @@ static int sti_dwmac_probe(struct platform_device *pdev) if (ret) return ret; - return stmmac_dvr_probe(&pdev->dev, plat_dat, &stmmac_res); + ret = stmmac_dvr_probe(&pdev->dev, plat_dat, &stmmac_res); + if (ret) + goto err_dwmac_exit; + + return 0; + +err_dwmac_exit: + sti_dwmac_exit(pdev, plat_dat->bsp_priv); + + return ret; } static const struct sti_dwmac_of_data stih4xx_dwmac_data = { |