SUNRPC: Fix xs_setup_bc_tcp()
authorTrond Myklebust <Trond.Myklebust@netapp.com>
Wed, 12 May 2010 21:50:23 +0000 (17:50 -0400)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Fri, 14 May 2010 19:09:33 +0000 (15:09 -0400)
It is a BUG for anybody to call this function without setting
args->bc_xprt. Trying to return an error value is just wrong, since the
user cannot fix this: it is a programming error, not a user error.

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
net/sunrpc/xprtsock.c

index 3d1dcdf2aef10f6cf96a4cd9f6945d25a2f47ddf..beefa7a3a90e519421fb617382fe6d757e778116 100644 (file)
@@ -2442,9 +2442,6 @@ static struct rpc_xprt *xs_setup_bc_tcp(struct xprt_create *args)
        struct sock_xprt *transport;
        struct svc_sock *bc_sock;
 
-       if (!args->bc_xprt)
-               ERR_PTR(-EINVAL);
-
        xprt = xs_setup_xprt(args, xprt_tcp_slot_table_entries);
        if (IS_ERR(xprt))
                return xprt;