tls: get psock ref after taking rxlock to avoid leak
authorSabrina Dubroca <sd@queasysnail.net>
Mon, 25 Mar 2024 15:56:48 +0000 (16:56 +0100)
committerJakub Kicinski <kuba@kernel.org>
Wed, 27 Mar 2024 03:48:24 +0000 (20:48 -0700)
commit417e91e856099e9b8a42a2520e2255e6afe024be
treecd7fdfe265a8661485225d94989b4ede3da7f76b
parentdc54b813df63020e946ccdef35b64d4fa99fd622
tls: get psock ref after taking rxlock to avoid leak

At the start of tls_sw_recvmsg, we take a reference on the psock, and
then call tls_rx_reader_lock. If that fails, we return directly
without releasing the reference.

Instead of adding a new label, just take the reference after locking
has succeeded, since we don't need it before.

Fixes: 4cbc325ed6b4 ("tls: rx: allow only one reader at a time")
Signed-off-by: Sabrina Dubroca <sd@queasysnail.net>
Reviewed-by: Simon Horman <horms@kernel.org>
Link: https://lore.kernel.org/r/fe2ade22d030051ce4c3638704ed58b67d0df643.1711120964.git.sd@queasysnail.net
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/tls/tls_sw.c