aboutsummaryrefslogtreecommitdiffstats
path: root/net/sunrpc/xprtsock.c
diff options
context:
space:
mode:
authorTrond Myklebust <Trond.Myklebust@netapp.com>2010-05-12 17:50:23 -0400
committerTrond Myklebust <Trond.Myklebust@netapp.com>2010-05-14 15:09:33 -0400
commit712a4338669d7d57f952244abb608e6ac07e39da (patch)
tree97875575e55a85ac02acc39bb7fe6a6139c71885 /net/sunrpc/xprtsock.c
parentff8399709e41bf72b4cb145612a0f9a9f7283c83 (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.c3
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;