On Mon, Jun 13, 2016 at 11:08 PM,  <weiyj...@163.com> wrote:
> From: Wei Yongjun <yongjun_...@trendmicro.com.cn>
>
> Fix to return a negative error code from the error handling
> case instead of 0, as done elsewhere in this function.
>
> Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn>
> ---
>  net/sctp/protocol.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/net/sctp/protocol.c b/net/sctp/protocol.c
> index 40022ee..3b56ae5 100644
> --- a/net/sctp/protocol.c
> +++ b/net/sctp/protocol.c
> @@ -1479,7 +1479,8 @@ static __init int sctp_init(void)
>                 INIT_HLIST_HEAD(&sctp_port_hashtable[i].chain);
>         }
>
> -       if (sctp_transport_hashtable_init())
> +       status = sctp_transport_hashtable_init();
> +       if (status)
>                 goto err_thash_alloc;
>
>         pr_info("Hash tables configured (bind %d/%d)\n", sctp_port_hashsize,
>
>
Acked-by: Xin Long <lucien....@gmail.com>

Reply via email to