Commit 8bc316cf authored by Eric Dumazet's avatar Eric Dumazet Committed by Jakub Kicinski
Browse files

selftests/net: packetdrill: add tcp_close_no_rst.pkt 



This test makes sure we do send a FIN on close()
if the receive queue contains data that was consumed.

Signed-off-by: default avatarEric Dumazet <edumazet@google.com>
Reviewed-by: default avatarNeal Cardwell <ncardwell@google.com>
Reviewed-by: default avatarKuniyuki Iwashima <kuniyu@google.com>
Reviewed-by: default avatarJason Xing <kerneljasonxing@gmail.com>
Link: https://patch.msgid.link/20250903084720.1168904-3-edumazet@google.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 5f923853

tools/testing/selftests/net/packetdrill/tcp_close_no_rst.pkt

0 → 100644
+32 −0
Original line number Diff line number Diff line 
// SPDX-License-Identifier: GPL-2.0



--mss=1000



`./defaults.sh`



// Initialize connection

    0 socket(..., SOCK_STREAM, IPPROTO_TCP) = 3

   +0 setsockopt(3, SOL_SOCKET, SO_REUSEADDR, [1], 4) = 0

   +0 bind(3, ..., ...) = 0

   +0 listen(3, 1) = 0



   +0 < S 0:0(0) win 32792 <mss 1000,sackOK,nop,nop>

   +0 > S. 0:0(0) ack 1 <mss 1460,nop,nop,sackOK>

  +.1 < . 1:1(0) ack 1 win 32792





   +0 accept(3, ..., ...) = 4

   +0 < . 1:1001(1000) ack 1 win 32792

   +0 > . 1:1(0) ack 1001

   +0 read(4, ..., 1000) = 1000



// resend the payload + a FIN

   +0 < F. 1:1001(1000) ack 1 win 32792

// Why do we have a delay and no dsack ?

   +0~+.04 > . 1:1(0) ack 1002



   +0 close(4) = 0



// According to RFC 2525, section 2.17

// we should _not_ send an RST here, because there was no data to consume.

   +0 > F. 1:1(0) ack 1002