wireguard: coverity fixes 24/28824/3
authorNeale Ranns <nranns@cisco.com>
Mon, 14 Sep 2020 13:28:42 +0000 (13:28 +0000)
committerDamjan Marion <dmarion@me.com>
Mon, 14 Sep 2020 18:58:03 +0000 (18:58 +0000)
Type: fix

Signed-off-by: Neale Ranns <nranns@cisco.com>
Change-Id: Ib1eabbc87a573c660ac251602d631f167928259b

src/plugins/wireguard/wireguard_api.c
src/plugins/wireguard/wireguard_if.c

index e107cb5..8bbacdd 100755 (executable)
@@ -39,7 +39,7 @@ static void
   wg_main_t *wmp = &wg_main;
   u8 private_key[NOISE_PUBLIC_KEY_LEN];
   ip_address_t src;
-  u32 sw_if_index;
+  u32 sw_if_index = ~0;
   int rv = 0;
 
   ip_address_decode2 (&mp->interface.src_ip, &src);
@@ -140,7 +140,7 @@ vl_api_wireguard_peer_add_t_handler (vl_api_wireguard_peer_add_t * mp)
 {
   vl_api_wireguard_peer_add_reply_t *rmp;
   wg_main_t *wmp = &wg_main;
-  index_t peeri;
+  index_t peeri = INDEX_INVALID;
   int ii, rv = 0;
 
   ip_address_t endpoint;
index ed90146..c91667b 100644 (file)
@@ -251,10 +251,14 @@ wg_if_create (u32 user_instance,
   wg_if_index_by_port[port] = wg_if - wg_if_pool;
 
   wg_if->port = port;
-  struct noise_upcall upcall;
-  upcall.u_remote_get = wg_remote_get;
-  upcall.u_index_set = wg_index_set;
-  upcall.u_index_drop = wg_index_drop;
+
+  /* *INDENT-OFF* */
+  struct noise_upcall upcall =  {
+    .u_remote_get = wg_remote_get,
+    .u_index_set = wg_index_set,
+    .u_index_drop = wg_index_drop,
+  };
+  /* *INDENT-ON* */
 
   noise_local_init (&wg_if->local, &upcall);
   noise_local_set_private (&wg_if->local, private_key);