diff options
| -rw-r--r-- | net/sunrpc/xprtsock.c | 29 |
1 files changed, 18 insertions, 11 deletions
diff --git a/net/sunrpc/xprtsock.c b/net/sunrpc/xprtsock.c index b7cd8cccbe72..2a9675136c68 100644 --- a/net/sunrpc/xprtsock.c +++ b/net/sunrpc/xprtsock.c | |||
| @@ -2293,6 +2293,7 @@ static struct rpc_xprt *xs_setup_udp(struct xprt_create *args) | |||
| 2293 | struct sockaddr *addr = args->dstaddr; | 2293 | struct sockaddr *addr = args->dstaddr; |
| 2294 | struct rpc_xprt *xprt; | 2294 | struct rpc_xprt *xprt; |
| 2295 | struct sock_xprt *transport; | 2295 | struct sock_xprt *transport; |
| 2296 | struct rpc_xprt *ret; | ||
| 2296 | 2297 | ||
| 2297 | xprt = xs_setup_xprt(args, xprt_udp_slot_table_entries); | 2298 | xprt = xs_setup_xprt(args, xprt_udp_slot_table_entries); |
| 2298 | if (IS_ERR(xprt)) | 2299 | if (IS_ERR(xprt)) |
| @@ -2330,8 +2331,8 @@ static struct rpc_xprt *xs_setup_udp(struct xprt_create *args) | |||
| 2330 | xs_format_peer_addresses(xprt, "udp", RPCBIND_NETID_UDP6); | 2331 | xs_format_peer_addresses(xprt, "udp", RPCBIND_NETID_UDP6); |
| 2331 | break; | 2332 | break; |
| 2332 | default: | 2333 | default: |
| 2333 | kfree(xprt); | 2334 | ret = ERR_PTR(-EAFNOSUPPORT); |
| 2334 | return ERR_PTR(-EAFNOSUPPORT); | 2335 | goto out_err; |
| 2335 | } | 2336 | } |
| 2336 | 2337 | ||
| 2337 | if (xprt_bound(xprt)) | 2338 | if (xprt_bound(xprt)) |
| @@ -2346,10 +2347,11 @@ static struct rpc_xprt *xs_setup_udp(struct xprt_create *args) | |||
| 2346 | 2347 | ||
| 2347 | if (try_module_get(THIS_MODULE)) | 2348 | if (try_module_get(THIS_MODULE)) |
| 2348 | return xprt; | 2349 | return xprt; |
| 2349 | 2350 | ret = ERR_PTR(-EINVAL); | |
| 2351 | out_err: | ||
| 2350 | kfree(xprt->slot); | 2352 | kfree(xprt->slot); |
| 2351 | kfree(xprt); | 2353 | kfree(xprt); |
| 2352 | return ERR_PTR(-EINVAL); | 2354 | return ret; |
| 2353 | } | 2355 | } |
| 2354 | 2356 | ||
| 2355 | static const struct rpc_timeout xs_tcp_default_timeout = { | 2357 | static const struct rpc_timeout xs_tcp_default_timeout = { |
| @@ -2368,6 +2370,7 @@ static struct rpc_xprt *xs_setup_tcp(struct xprt_create *args) | |||
| 2368 | struct sockaddr *addr = args->dstaddr; | 2370 | struct sockaddr *addr = args->dstaddr; |
| 2369 | struct rpc_xprt *xprt; | 2371 | struct rpc_xprt *xprt; |
| 2370 | struct sock_xprt *transport; | 2372 | struct sock_xprt *transport; |
| 2373 | struct rpc_xprt *ret; | ||
| 2371 | 2374 | ||
| 2372 | xprt = xs_setup_xprt(args, xprt_tcp_slot_table_entries); | 2375 | xprt = xs_setup_xprt(args, xprt_tcp_slot_table_entries); |
| 2373 | if (IS_ERR(xprt)) | 2376 | if (IS_ERR(xprt)) |
| @@ -2403,8 +2406,8 @@ static struct rpc_xprt *xs_setup_tcp(struct xprt_create *args) | |||
| 2403 | xs_format_peer_addresses(xprt, "tcp", RPCBIND_NETID_TCP6); | 2406 | xs_format_peer_addresses(xprt, "tcp", RPCBIND_NETID_TCP6); |
| 2404 | break; | 2407 | break; |
| 2405 | default: | 2408 | default: |
| 2406 | kfree(xprt); | 2409 | ret = ERR_PTR(-EAFNOSUPPORT); |
| 2407 | return ERR_PTR(-EAFNOSUPPORT); | 2410 | goto out_err; |
| 2408 | } | 2411 | } |
| 2409 | 2412 | ||
| 2410 | if (xprt_bound(xprt)) | 2413 | if (xprt_bound(xprt)) |
| @@ -2420,10 +2423,11 @@ static struct rpc_xprt *xs_setup_tcp(struct xprt_create *args) | |||
| 2420 | 2423 | ||
| 2421 | if (try_module_get(THIS_MODULE)) | 2424 | if (try_module_get(THIS_MODULE)) |
| 2422 | return xprt; | 2425 | return xprt; |
| 2423 | 2426 | ret = ERR_PTR(-EINVAL); | |
| 2427 | out_err: | ||
| 2424 | kfree(xprt->slot); | 2428 | kfree(xprt->slot); |
| 2425 | kfree(xprt); | 2429 | kfree(xprt); |
| 2426 | return ERR_PTR(-EINVAL); | 2430 | return ret; |
| 2427 | } | 2431 | } |
| 2428 | 2432 | ||
| 2429 | /** | 2433 | /** |
| @@ -2437,6 +2441,7 @@ static struct rpc_xprt *xs_setup_bc_tcp(struct xprt_create *args) | |||
| 2437 | struct rpc_xprt *xprt; | 2441 | struct rpc_xprt *xprt; |
| 2438 | struct sock_xprt *transport; | 2442 | struct sock_xprt *transport; |
| 2439 | struct svc_sock *bc_sock; | 2443 | struct svc_sock *bc_sock; |
| 2444 | struct rpc_xprt *ret; | ||
| 2440 | 2445 | ||
| 2441 | xprt = xs_setup_xprt(args, xprt_tcp_slot_table_entries); | 2446 | xprt = xs_setup_xprt(args, xprt_tcp_slot_table_entries); |
| 2442 | if (IS_ERR(xprt)) | 2447 | if (IS_ERR(xprt)) |
| @@ -2476,8 +2481,8 @@ static struct rpc_xprt *xs_setup_bc_tcp(struct xprt_create *args) | |||
| 2476 | RPCBIND_NETID_TCP6); | 2481 | RPCBIND_NETID_TCP6); |
| 2477 | break; | 2482 | break; |
| 2478 | default: | 2483 | default: |
| 2479 | kfree(xprt); | 2484 | ret = ERR_PTR(-EAFNOSUPPORT); |
| 2480 | return ERR_PTR(-EAFNOSUPPORT); | 2485 | goto out_err; |
| 2481 | } | 2486 | } |
| 2482 | 2487 | ||
| 2483 | if (xprt_bound(xprt)) | 2488 | if (xprt_bound(xprt)) |
| @@ -2499,9 +2504,11 @@ static struct rpc_xprt *xs_setup_bc_tcp(struct xprt_create *args) | |||
| 2499 | 2504 | ||
| 2500 | if (try_module_get(THIS_MODULE)) | 2505 | if (try_module_get(THIS_MODULE)) |
| 2501 | return xprt; | 2506 | return xprt; |
| 2507 | ret = ERR_PTR(-EINVAL); | ||
| 2508 | out_err: | ||
| 2502 | kfree(xprt->slot); | 2509 | kfree(xprt->slot); |
| 2503 | kfree(xprt); | 2510 | kfree(xprt); |
| 2504 | return ERR_PTR(-EINVAL); | 2511 | return ret; |
| 2505 | } | 2512 | } |
| 2506 | 2513 | ||
| 2507 | static struct xprt_class xs_udp_transport = { | 2514 | static struct xprt_class xs_udp_transport = { |
