Skip to content

Commit

Permalink
adm8211: fix memory leak
Browse files Browse the repository at this point in the history
We must free priv->eeprom allocated in adm8211_read_eeprom().

Signed-off-by: Kulikov Vasiliy <segooon@gmail.com>
Acked-by: Luis R. Rodriguez <lrodriguez@atheros.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
  • Loading branch information
Kulikov Vasiliy authored and John W. Linville committed Jul 12, 2010
1 parent 4c92831 commit b85aeb5
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion drivers/net/wireless/adm8211.c
Original file line number Diff line number Diff line change
Expand Up @@ -1903,7 +1903,7 @@ static int __devinit adm8211_probe(struct pci_dev *pdev,
if (err) {
printk(KERN_ERR "%s (adm8211): Cannot register device\n",
pci_name(pdev));
goto err_free_desc;
goto err_free_eeprom;
}

printk(KERN_INFO "%s: hwaddr %pM, Rev 0x%02x\n",
Expand All @@ -1912,6 +1912,9 @@ static int __devinit adm8211_probe(struct pci_dev *pdev,

return 0;

err_free_eeprom:
kfree(priv->eeprom);

err_free_desc:
pci_free_consistent(pdev,
sizeof(struct adm8211_desc) * priv->rx_ring_size +
Expand Down

0 comments on commit b85aeb5

Please sign in to comment.