diff options
author | Eric Dumazet <edumazet@google.com> | 2017-11-03 02:10:03 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-11-03 08:02:56 +0100 |
commit | f67971e683e81d7ba4739728511ae6e52a1b6321 (patch) | |
tree | c960635bc9e1813044d72d838df112acc04fc9b2 /net/ipv4/tcp_htcp.c | |
parent | mISDN: hfcpci: Convert timers to use timer_setup() (diff) | |
download | linux-f67971e683e81d7ba4739728511ae6e52a1b6321.tar.xz linux-f67971e683e81d7ba4739728511ae6e52a1b6321.zip |
tcp: tcp_fragment() should not assume rtx skbs
While stress testing MTU probing, we had crashes in list_del() that we root-caused
to the fact that tcp_fragment() is unconditionally inserting the freshly allocated
skb into tsorted_sent_queue list.
But this list is supposed to contain skbs that were sent.
This was mostly harmless until MTU probing was enabled.
Fortunately we can use the tcp_queue enum added later (but in same linux version)
for rtx-rb-tree to fix the bug.
Fixes: e2080072ed2d ("tcp: new list for sent but unacked skbs for RACK recovery")
Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: Yuchung Cheng <ycheng@google.com>
Cc: Neal Cardwell <ncardwell@google.com>
Cc: Soheil Hassas Yeganeh <soheil@google.com>
Cc: Alexei Starovoitov <ast@kernel.org>
Cc: Priyaranjan Jha <priyarjha@google.com>
Acked-by: Neal Cardwell <ncardwell@google.com>
Acked-by: Soheil Hassas Yeganeh <soheil@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/tcp_htcp.c')
0 files changed, 0 insertions, 0 deletions