diff options
| author | Woojung Huh <woojung.huh@microchip.com> | 2016-11-23 23:10:33 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-11-25 19:57:57 -0500 |
| commit | a7dac9f9c1695d7470b2f4e7447513e4a71b7c1b (patch) | |
| tree | 1118e18f73b9b8c74b7a049f4c48395ee2b2212a /drivers/net/phy/phy_device.c | |
| parent | f52dffe049ee11ecc02588a118fbe4092672fbaa (diff) | |
| download | lwn-a7dac9f9c1695d7470b2f4e7447513e4a71b7c1b.tar.gz lwn-a7dac9f9c1695d7470b2f4e7447513e4a71b7c1b.zip | |
phy: fix error case of phy_led_triggers_(un)register
When phy_init_hw() fails at phy_attach_direct();
- phy_detach() calls phy_led_triggers_unregister() without
previous call of phy_led_triggers_register().
- still call phy_led_triggers_register() and cause memory leak.
Fixes: 2e0bc452f472 ("net: phy: leds: add support for led triggers on phy link state change")
Signed-off-by: Woojung Huh <woojung.huh@microchip.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/phy/phy_device.c')
| -rw-r--r-- | drivers/net/phy/phy_device.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/net/phy/phy_device.c b/drivers/net/phy/phy_device.c index 9e8f048891bd..ba86c191a13e 100644 --- a/drivers/net/phy/phy_device.c +++ b/drivers/net/phy/phy_device.c @@ -914,15 +914,15 @@ int phy_attach_direct(struct net_device *dev, struct phy_device *phydev, */ err = phy_init_hw(phydev); if (err) - phy_detach(phydev); - else - phy_resume(phydev); + goto error; + phy_resume(phydev); phy_led_triggers_register(phydev); return err; error: + phy_detach(phydev); put_device(d); module_put(bus->owner); return err; |