mirrors/passt
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

tcp: Don't mistake a FIN segment with no data for a Fast Retransmit request

Browse source 
It carries no data and usually duplicates the previous ACK sequence,
but it's just a FIN.

Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
This commit is contained in:
Stefano Brivio 2021-09-14 16:50:09 +02:00
parent c162f1e801
commit 7c82ea4dd9
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
tcp.c
View file

@ -2007,7 +2007,8 @@ static void tcp_data_from_tap(struct ctx *c, struct tcp_tap_conn *conn,
ack_seq - max_ack_seq < MAX_WINDOW) { ack_seq - max_ack_seq < MAX_WINDOW) {
/* Fast re-transmit */ /* Fast re-transmit */
retr = !len && ack_seq == max_ack_seq && retr = !len && !th->fin &&
ack_seq == max_ack_seq &&
max_ack_seq_wnd == ntohs(th->window); max_ack_seq_wnd == ntohs(th->window);
max_ack_seq_wnd = ntohs(th->window); max_ack_seq_wnd = ntohs(th->window);