diff options
| author | Kulikov Vasiliy <segooon@gmail.com> | 2010-09-25 23:58:06 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-09-26 18:54:34 -0700 |
| commit | 8d879de89807d82bc4cc3e9d73609b874fa9458c (patch) | |
| tree | 4c654cab6a0dc46f4e5f7e0330bc22c4747acbc2 | |
| parent | 22138d307329e1968fc698821095b87c2fd5de12 (diff) | |
| download | lwn-8d879de89807d82bc4cc3e9d73609b874fa9458c.tar.gz lwn-8d879de89807d82bc4cc3e9d73609b874fa9458c.zip | |
sgiseeq: use free_netdev(netdev) instead of kfree()
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>
| -rw-r--r-- | drivers/net/sgiseeq.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/sgiseeq.c b/drivers/net/sgiseeq.c index cc4bd8c65f8b..9265315baa0b 100644 --- a/drivers/net/sgiseeq.c +++ b/drivers/net/sgiseeq.c @@ -804,7 +804,7 @@ static int __devinit sgiseeq_probe(struct platform_device *pdev) err_out_free_page: free_page((unsigned long) sp->srings); err_out_free_dev: - kfree(dev); + free_netdev(dev); err_out: return err; |