diff options
author | David S. Miller <davem@davemloft.net> | 2006-03-10 18:08:09 -0800 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2006-03-11 13:25:17 -0500 |
commit | 9e927fb6186b6913d47d33068160088a084d568e (patch) | |
tree | 4ecd5584f82fbda06cfe6c162505b59ba4391915 /drivers/net/dl2k.c | |
parent | 5ee33b7adc178dbeb16b4d8303ae3ab69a334518 (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>
Diffstat (limited to 'drivers/net/dl2k.c')
0 files changed, 0 insertions, 0 deletions