Revert "SUNRPC: attempt AF_LOCAL connect on setup"
authorTrond Myklebust <trond.myklebust@hammerspace.com>
Fri, 29 Apr 2022 16:27:30 +0000 (12:27 -0400)
committerTrond Myklebust <trond.myklebust@hammerspace.com>
Sat, 30 Apr 2022 00:38:27 +0000 (20:38 -0400)
This reverts commit 7073ea8799a8cf73db60270986f14e4aae20fa80.

We must not try to connect the socket while the transport is under
construction, because the mechanisms to safely tear it down are not in
place. As the code stands, we end up leaking the sockets on a connection
error.

Reported-by: wanghai (M) <wanghai38@huawei.com>
Cc: stable@vger.kernel.org
Signed-off-by: Trond Myklebust <trond.myklebust@hammerspace.com>
net/sunrpc/xprtsock.c

index 25b8a8e..650102a 100644 (file)
@@ -2875,9 +2875,6 @@ static struct rpc_xprt *xs_setup_local(struct xprt_create *args)
                }
                xprt_set_bound(xprt);
                xs_format_peer_addresses(xprt, "local", RPCBIND_NETID_LOCAL);
-               ret = ERR_PTR(xs_local_setup_socket(transport));
-               if (ret)
-                       goto out_err;
                break;
        default:
                ret = ERR_PTR(-EAFNOSUPPORT);