commit | 9e927fb6186b6913d47d33068160088a084d568e | [log] [tgz] |
---|---|---|
author | David S. Miller <davem@davemloft.net> | Fri Mar 10 18:08:09 2006 -0800 |
committer | Jeff Garzik <jeff@garzik.org> | Sat Mar 11 13:25:17 2006 -0500 |
tree | 4ecd5584f82fbda06cfe6c162505b59ba4391915 | |
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>