SUNRPC: Fix a potential race in xprt_connect()
authorTrond Myklebust <trond.myklebust@hammerspace.com>
Sun, 2 Dec 2018 04:18:00 +0000 (23:18 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 21 Dec 2018 13:11:36 +0000 (14:11 +0100)
commit5ba8d8b5a27c3d50f6544d1ce8c6732c8b36aedc
treedd06c86a7658e274a0594396d673e667f57d02bc
parentde42cd219802457caa61c001776281fe42deae9d
SUNRPC: Fix a potential race in xprt_connect()

[ Upstream commit 0a9a4304f3614e25d9de9b63502ca633c01c0d70 ]

If an asynchronous connection attempt completes while another task is
in xprt_connect(), then the call to rpc_sleep_on() could end up
racing with the call to xprt_wake_pending_tasks().
So add a second test of the connection state after we've put the
task to sleep and set the XPRT_CONNECTING flag, when we know that there
can be no asynchronous connection attempts still in progress.

Fixes: 0b9e79431377d ("SUNRPC: Move the test for XPRT_CONNECTING into...")
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
net/sunrpc/xprt.c