renegade-project/linux-next
2
0
Fork 0
mirror of https://github.com/edk2-porting/linux-next.git synced 2024-12-18 02:04:05 +08:00
Code

ibm_newemac: use free_netdev(netdev) instead of kfree()

Browse Source 
Freeing netdev without free_netdev() leads to net, tx leaks.
I might lead to dereferencing freed pointer.

The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)

@@
struct net_device* dev;
@@

-kfree(dev)
+free_netdev(dev)

Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Kulikov Vasiliy 2010-09-25 23:58:00 +00:00 committed by David S. Miller
parent 62038e4a14
commit 52933f0521
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
drivers/net/ibm_newemac/core.c
View File

@ -2928,7 +2928,7 @@ static int __devinit emac_probe(struct platform_device *ofdev,
if (dev->emac_irq != NO_IRQ) if (dev->emac_irq != NO_IRQ)
irq_dispose_mapping(dev->emac_irq); irq_dispose_mapping(dev->emac_irq);
err_free: err_free:
kfree(ndev); free_netdev(ndev);
err_gone: err_gone:
/* if we were on the bootlist, remove us as we won't show up and /* if we were on the bootlist, remove us as we won't show up and
* wake up all waiters to notify them in case they were waiting * wake up all waiters to notify them in case they were waiting
@ -2971,7 +2971,7 @@ static int __devexit emac_remove(struct platform_device *ofdev)
if (dev->emac_irq != NO_IRQ) if (dev->emac_irq != NO_IRQ)
irq_dispose_mapping(dev->emac_irq); irq_dispose_mapping(dev->emac_irq);
kfree(dev->ndev); free_netdev(dev->ndev);
return 0; return 0;
} }