diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-04-03 03:02:28 (GMT) |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-04-07 21:04:17 (GMT) |
commit | 524fba6c14d2892bfa27c01577d544d6380291ba (patch) | |
tree | 058453c4d299208117ab8043b920b2a08889cb98 /net/sctp | |
parent | cfbe800b8b771afc7d5aa113e19f85ec933b7618 (diff) | |
download | linux-fsl-qoriq-524fba6c14d2892bfa27c01577d544d6380291ba.tar.xz |
sctp: fix error return code in __sctp_connect()
Fix to return a negative error code from the error handling
case instead of 0, as returned elsewhere in this function.
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Acked-by: Vlad Yasevich <vyasevich@gmail.com>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sctp')
-rw-r--r-- | net/sctp/socket.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/net/sctp/socket.c b/net/sctp/socket.c index dd21ae3..f631c5f 100644 --- a/net/sctp/socket.c +++ b/net/sctp/socket.c @@ -1119,9 +1119,10 @@ static int __sctp_connect(struct sock* sk, /* Make sure the destination port is correctly set * in all addresses. */ - if (asoc && asoc->peer.port && asoc->peer.port != port) + if (asoc && asoc->peer.port && asoc->peer.port != port) { + err = -EINVAL; goto out_free; - + } /* Check if there already is a matching association on the * endpoint (other than the one created here). |