From: Marco Varlese <marco.varlese@suse.com>
Date: Wed, 18 Apr 2018 06:50:57 +0000 (+0200)
Subject: SCTP: coverity warnings
X-Git-Tag: v18.07-rc1~449
X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=63777e599af8df1e6bf8c564c638d52007af47d0;p=vpp.git

SCTP: coverity warnings

This patch addresses the warning 182994

Change-Id: I071f3cbdf9965c7b57d444f79430ee75c66c856e
Signed-off-by: Marco Varlese <marco.varlese@suse.com>
---

diff --git a/src/vnet/sctp/sctp_input.c b/src/vnet/sctp/sctp_input.c
index 7c753a2ffd7..ff79840f9fb 100644
--- a/src/vnet/sctp/sctp_input.c
+++ b/src/vnet/sctp/sctp_input.c
@@ -492,7 +492,6 @@ sctp_handle_init_ack (sctp_header_t * sctp_hdr,
 {
   sctp_init_ack_chunk_t *init_ack_chunk =
     (sctp_init_ack_chunk_t *) (sctp_hdr);
-  sctp_state_cookie_param_t state_cookie;
 
   char hostname[FQDN_MAX_LENGTH];
 
@@ -569,8 +568,9 @@ sctp_handle_init_ack (sctp_header_t * sctp_hdr,
 		sctp_state_cookie_param_t *state_cookie_param =
 		  (sctp_state_cookie_param_t *) opt_params_hdr;
 
-		clib_memcpy (&state_cookie, state_cookie_param,
+		clib_memcpy (&(sctp_conn->cookie_param), state_cookie_param,
 			     sizeof (sctp_state_cookie_param_t));
+
 		break;
 	      }
 	    case SCTP_HOSTNAME_ADDRESS_TYPE:
@@ -596,9 +596,6 @@ sctp_handle_init_ack (sctp_header_t * sctp_hdr,
 	}
     }
 
-  clib_memcpy (&(sctp_conn->cookie_param), &state_cookie,
-	       sizeof (sctp_state_cookie_param_t));
-
   sctp_prepare_cookie_echo_chunk (sctp_conn, idx, b0, 1);
 
   /* Start the T1_COOKIE timer */