Commit 3ed58f96 authored by Andre Edich's avatar Andre Edich Committed by David S. Miller

smsc95xx: avoid memory leak in smsc95xx_bind

In a case where the ID_REV register read is failed, the memory for a
private data structure has to be freed before returning error from the
function smsc95xx_bind.

Fixes: bbd9f9ee ("smsc95xx: add wol support for more frame types")
Signed-off-by: default avatarAndre Edich <andre.edich@microchip.com>
Signed-off-by: default avatarParthiban Veerasooran <Parthiban.Veerasooran@microchip.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 7c8b1e85
...@@ -1293,7 +1293,8 @@ static int smsc95xx_bind(struct usbnet *dev, struct usb_interface *intf) ...@@ -1293,7 +1293,8 @@ static int smsc95xx_bind(struct usbnet *dev, struct usb_interface *intf)
/* detect device revision as different features may be available */ /* detect device revision as different features may be available */
ret = smsc95xx_read_reg(dev, ID_REV, &val); ret = smsc95xx_read_reg(dev, ID_REV, &val);
if (ret < 0) if (ret < 0)
return ret; goto free_pdata;
val >>= 16; val >>= 16;
pdata->chip_id = val; pdata->chip_id = val;
pdata->mdix_ctrl = get_mdix_status(dev->net); pdata->mdix_ctrl = get_mdix_status(dev->net);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment