diff options
author | David S. Miller <davem@davemloft.net> | 2018-10-28 13:35:12 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-10-28 13:35:12 -0400 |
commit | ece23711dd956cd5053c9cb03e9fe0668f9c8894 (patch) | |
tree | 292446034e76dd6fae7dc66607f8115d6c9aad88 /net/core/dev.c | |
parent | 6788fac82001ed1944b5da976bcec4a7b9accf51 (diff) |
net: Properly unlink GRO packets on overflow.
Just like with normal GRO processing, we have to initialize
skb->next to NULL when we unlink overflow packets from the
GRO hash lists.
Fixes: d4546c2509b1 ("net: Convert GRO SKB handling to list_head.")
Reported-by: Oleksandr Natalenko <oleksandr@natalenko.name>
Tested-by: Oleksandr Natalenko <oleksandr@natalenko.name>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core/dev.c')
-rw-r--r-- | net/core/dev.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/core/dev.c b/net/core/dev.c index 022ad73d6253..77d43ae2a7bb 100644 --- a/net/core/dev.c +++ b/net/core/dev.c | |||
@@ -5457,7 +5457,7 @@ static void gro_flush_oldest(struct list_head *head) | |||
5457 | /* Do not adjust napi->gro_hash[].count, caller is adding a new | 5457 | /* Do not adjust napi->gro_hash[].count, caller is adding a new |
5458 | * SKB to the chain. | 5458 | * SKB to the chain. |
5459 | */ | 5459 | */ |
5460 | list_del(&oldest->list); | 5460 | skb_list_del_init(oldest); |
5461 | napi_gro_complete(oldest); | 5461 | napi_gro_complete(oldest); |
5462 | } | 5462 | } |
5463 | 5463 | ||