diff options
| author | Mark Johnston <markj@FreeBSD.org> | 2026-06-15 15:52:24 +0000 |
|---|---|---|
| committer | Mark Johnston <markj@FreeBSD.org> | 2026-06-30 02:32:57 +0000 |
| commit | 7b3373d4eb5f86345e743e1233866fa7b0ff6160 (patch) | |
| tree | 4fcb1970a94cb7cca5074a1fc6c0e5d828abff04 /.github/workflows | |
| parent | b77d19dcc0d5ff8fc704ae255630b92b7a91f05d (diff) | |
rpcsec_tls: Avoid a socket reference underflow in rpctls_server()
The upcall_sockets tree owns a ref on any resident socket. When a
socket is removed after a TLS handshake failure, rpctls_rpc_failed()
thus calls soclose().
rpctls_server() does not acquire an extra ref to compensate for this.
So, if the upcall fails, e.g., because rpc.tlsservd is not running,
we'll call soclose() to drop the reference, but this effectively
releases the xprt layer's reference.
Fix the problem by explicitly acquiring a socket reference when adding
a socket to the upcall tree.
Approved by: so
Security: FreeBSD-EN-26:17.rpcsec_tls
PR: 289734
Reviewed by: rmacklem, glebius
MFC after: 1 week
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D57555
(cherry picked from commit fc7993cf2d6ddba9f94683565838bf8fabc0145c)
(cherry picked from commit f3b14134dec11c2f09980f25c4c60accbfea0f1e)
Diffstat (limited to '.github/workflows')
0 files changed, 0 insertions, 0 deletions
