diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-05-12 17:50:23 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2010-05-14 15:09:33 -0400 |
commit | 712a4338669d7d57f952244abb608e6ac07e39da (patch) | |
tree | 97875575e55a85ac02acc39bb7fe6a6139c71885 /net/sunrpc/xprtsock.c | |
parent | ff8399709e41bf72b4cb145612a0f9a9f7283c83 (diff) |
SUNRPC: Fix xs_setup_bc_tcp()
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>
Diffstat (limited to 'net/sunrpc/xprtsock.c')
-rw-r--r-- | net/sunrpc/xprtsock.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/net/sunrpc/xprtsock.c b/net/sunrpc/xprtsock.c index 3d1dcdf2aef1..beefa7a3a90e 100644 --- a/net/sunrpc/xprtsock.c +++ b/net/sunrpc/xprtsock.c | |||
@@ -2442,9 +2442,6 @@ static struct rpc_xprt *xs_setup_bc_tcp(struct xprt_create *args) | |||
2442 | struct sock_xprt *transport; | 2442 | struct sock_xprt *transport; |
2443 | struct svc_sock *bc_sock; | 2443 | struct svc_sock *bc_sock; |
2444 | 2444 | ||
2445 | if (!args->bc_xprt) | ||
2446 | ERR_PTR(-EINVAL); | ||
2447 | |||
2448 | xprt = xs_setup_xprt(args, xprt_tcp_slot_table_entries); | 2445 | xprt = xs_setup_xprt(args, xprt_tcp_slot_table_entries); |
2449 | if (IS_ERR(xprt)) | 2446 | if (IS_ERR(xprt)) |
2450 | return xprt; | 2447 | return xprt; |