On Mon, Mar 20, 2017 at 06:00:28PM +0800, Xin Long wrote:
> This patch is to remove the unnecessary temporary variable 'err' from
> sctp_association_init.
>
> Signed-off-by: Xin Long <lucien....@gmail.com>
> ---
> net/sctp/associola.c | 6 ++----
> 1 file changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/net/sctp/associola.c b/net/sctp/associola.c
> index 2a6835b..0439a1a 100644
> --- a/net/sctp/associola.c
> +++ b/net/sctp/associola.c
> @@ -71,9 +71,8 @@ static struct sctp_association
> *sctp_association_init(struct sctp_association *a
> {
> struct net *net = sock_net(sk);
> struct sctp_sock *sp;
> - int i;
> sctp_paramhdr_t *p;
> - int err;
> + int i;
>
> /* Retrieve the SCTP per socket area. */
> sp = sctp_sk((struct sock *)sk);
> @@ -264,8 +263,7 @@ static struct sctp_association
> *sctp_association_init(struct sctp_association *a
>
> /* AUTH related initializations */
> INIT_LIST_HEAD(&asoc->endpoint_shared_keys);
> - err = sctp_auth_asoc_copy_shkeys(ep, asoc, gfp);
> - if (err)
> + if (sctp_auth_asoc_copy_shkeys(ep, asoc, gfp))
> goto fail_init;
>
> asoc->active_key_id = ep->active_key_id;
> --
> 2.1.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-sctp" in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
Acked-by: Neil Horman <nhor...@tuxdriver.com>
