Commit d8779845 authored Sep 23, 2008 by David S. Miller

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>

parent e5d23048

drivers/block/aoe/aoenet.c

+3 −1

Original line number	Diff line number	Diff line
		@@ -99,9 +99,11 @@ aoenet_xmit(struct sk_buff_head *queue)
		{
		struct sk_buff skb, tmp;

		skb_queue_walk_safe(queue, skb, tmp)
		skb_queue_walk_safe(queue, skb, tmp) {
		__skb_unlink(skb, queue);
		dev_queue_xmit(skb);
		}
		}

		/*
		* (1) len doesn't include the header by default. I want this.