]> git.baikalelectronics.ru Git - kernel.git/commit
projects / kernel.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: d69373b) | patch
[PATCH] Wrong return value corrupts free object in e1000 driver
authorDavid S. Miller <davem@davemloft.net>
Sat, 11 Mar 2006 02:08:09 +0000 (18:08 -0800)
committerJeff Garzik <jeff@garzik.org>
Sat, 11 Mar 2006 18:25:17 +0000 (13:25 -0500)
commit60a6740c596da1df52189f998bf8d21e81ad9cc8
tree4ecd5584f82fbda06cfe6c162505b59ba4391915tree | snapshot
parentd69373b296f836b9f666f367fd7a6ab902af5656commit | diff
[PATCH] Wrong return value corrupts free object in e1000 driver

For some reason, E1000's ->hard_start_xmit() routine returns -EFAULT
instead of one of the NETDEV_TX_* error codes.  In fact, it frees up
the SKB before returning this.  This makes the queueing layer think
the packet should be requeued and subsequently we corrupt a freed
object.

Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
drivers/net/e1000/e1000_main.c diff | blob | history
Linux Kernel Source Tree.