]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
net/tls: Consume empty data records in tls_sw_read_sock()
authorChuck Lever <cel@kernel.org>
Tue, 30 Jun 2026 19:15:51 +0000 (15:15 -0400)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 7 Jul 2026 08:12:11 +0000 (10:12 +0200)
commit3be28e2c9cd0230cb51fd4967df095273afd3848
tree1817cfb3c25d059cebcd0302e8c18e264f6d8ad0
parent9e05e91a9a847ed57926414bd7c2c5e54d6c56c6
net/tls: Consume empty data records in tls_sw_read_sock()

A peer may send a zero-length TLS application_data record; TLS 1.3
explicitly permits these as a traffic-analysis countermeasure (RFC
8446, Section 5.1). After decryption such a record has full_len ==
0. tls_sw_read_sock() hands it to the read_actor, which has no
payload to consume and returns zero. The loop treats a zero return
as backpressure (used <= 0), requeues the skb at the head of
rx_list, and stops. rx_list is serviced head-first on the next
call, so the empty record is dequeued, fails the same way, and is
requeued again; every later record on the connection is blocked
behind it.

tls_sw_recvmsg() does not stall on this: a zero-length data record
copies nothing and falls through to consume_skb(). Mirror that in
the read_sock() path by recognizing an empty data record before
the actor runs, consuming it, and continuing.

Fixes: 662fbcec32f4 ("net/tls: implement ->read_sock()")
Signed-off-by: Chuck Lever <cel@kernel.org>
Reviewed-by: Sabrina Dubroca <sd@queasysnail.net>
Link: https://patch.msgid.link/20260630191551.875664-1-cel@kernel.org
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
net/tls/tls_sw.c