diff options
author | Stanislav Kinsbursky <skinsbursky@parallels.com> | 2011-11-10 06:33:23 -0500 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2011-11-10 14:50:07 -0500 |
commit | 2aa13531bbbc6582874bedfcd853e1058b0fb4f9 (patch) | |
tree | 6dc0ad696b49bdf6f68382900775a19fb289bf25 /net/sunrpc/xprtsock.c | |
parent | a6f498a891c730327645a7afa10c5ae977de6fd8 (diff) |
SUNRPC: destroy freshly allocated transport in case of sockaddr init error
Otherwise we will leak xprt structure and struct net reference.
Signed-off-by: Stanislav Kinsbursky <skinsbursky@parallels.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'net/sunrpc/xprtsock.c')
-rw-r--r-- | net/sunrpc/xprtsock.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net/sunrpc/xprtsock.c b/net/sunrpc/xprtsock.c index d7f97ef26590..2d78d95955ab 100644 --- a/net/sunrpc/xprtsock.c +++ b/net/sunrpc/xprtsock.c | |||
@@ -2530,8 +2530,10 @@ static struct rpc_xprt *xs_setup_xprt(struct xprt_create *args, | |||
2530 | int err; | 2530 | int err; |
2531 | err = xs_init_anyaddr(args->dstaddr->sa_family, | 2531 | err = xs_init_anyaddr(args->dstaddr->sa_family, |
2532 | (struct sockaddr *)&new->srcaddr); | 2532 | (struct sockaddr *)&new->srcaddr); |
2533 | if (err != 0) | 2533 | if (err != 0) { |
2534 | xprt_free(xprt); | ||
2534 | return ERR_PTR(err); | 2535 | return ERR_PTR(err); |
2536 | } | ||
2535 | } | 2537 | } |
2536 | 2538 | ||
2537 | return xprt; | 2539 | return xprt; |