diff options
author | David S. Miller <davem@davemloft.net> | 2008-09-23 23:47:22 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-09-23 23:47:22 -0400 |
commit | d87798450a7635ab1bcc80271a13ce4a53b016a9 (patch) | |
tree | d60f6a09089f856073e9a02df92dbdf54c3b62fb /drivers/block/aoe/aoenet.c | |
parent | e5d2304802a63304a54cff010209c4a717a2509f (diff) |
aoe: Fix OOPS after SKB queue changes.
Reported by Thomas Graf.
If we don't unlink the SKB from the queue when we send it
out in aoenet_xmit(), dev_hard_start_xmit() will see skb->next
as non-NULL and interpret this to mean the SKB is part of a
GSO segment list.
Add __skb_unlink() call to fix that.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/block/aoe/aoenet.c')
-rw-r--r-- | drivers/block/aoe/aoenet.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/block/aoe/aoenet.c b/drivers/block/aoe/aoenet.c index 8fb26030eba8..9157d64270cb 100644 --- a/drivers/block/aoe/aoenet.c +++ b/drivers/block/aoe/aoenet.c | |||
@@ -99,8 +99,10 @@ aoenet_xmit(struct sk_buff_head *queue) | |||
99 | { | 99 | { |
100 | struct sk_buff *skb, *tmp; | 100 | struct sk_buff *skb, *tmp; |
101 | 101 | ||
102 | skb_queue_walk_safe(queue, skb, tmp) | 102 | skb_queue_walk_safe(queue, skb, tmp) { |
103 | __skb_unlink(skb, queue); | ||
103 | dev_queue_xmit(skb); | 104 | dev_queue_xmit(skb); |
105 | } | ||
104 | } | 106 | } |
105 | 107 | ||
106 | /* | 108 | /* |