Skip to content
Commit 63f2a9bd authored by Alexander Ovechkin's avatar Alexander Ovechkin Committed by Greg Kroah-Hartman
Browse files

tcp: relookup sock for RST+ACK packets handled by obsolete req sock

[ Upstream commit 7233da86 ]

Currently tcp_check_req can be called with obsolete req socket for which big
socket have been already created (because of CPU race or early demux
assigning req socket to multiple packets in gro batch).

Commit e0f9759f ("tcp: try to keep packet if SYN_RCV race
is lost") added retry in case when tcp_check_req is called for PSH|ACK packet.
But if client sends RST+ACK immediatly after connection being
established (it is performing healthcheck, for example) retry does not
occur. In that case tcp_check_req tries to close req socket,
leaving big socket active.

Fixes: e0f9759f

 ("tcp: try to keep packet if SYN_RCV race is lost")
Signed-off-by: default avatarAlexander Ovechkin <ovov@yandex-team.ru>
Reported-by: default avatarOleg Senin <olegsenin@yandex-team.ru>
Reviewed-by: default avatarEric Dumazet <edumazet@google.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent 50f41f2e
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment