diff options
author | Neale Ranns <nranns@cisco.com> | 2020-09-14 13:28:42 +0000 |
---|---|---|
committer | Damjan Marion <dmarion@me.com> | 2020-09-14 18:58:03 +0000 |
commit | 76770fd659420c23e43422d672a55e268f042129 (patch) | |
tree | 16fbc05bc2be95ac9fb142c19130cf2ac06df346 | |
parent | 935ce75cb060da34991e6b02bec5680445f85323 (diff) |
wireguard: coverity fixes
Type: fix
Signed-off-by: Neale Ranns <nranns@cisco.com>
Change-Id: Ib1eabbc87a573c660ac251602d631f167928259b
-rwxr-xr-x | src/plugins/wireguard/wireguard_api.c | 4 | ||||
-rw-r--r-- | src/plugins/wireguard/wireguard_if.c | 12 |
2 files changed, 10 insertions, 6 deletions
diff --git a/src/plugins/wireguard/wireguard_api.c b/src/plugins/wireguard/wireguard_api.c index e107cb56b4b..8bbacddaf45 100755 --- a/src/plugins/wireguard/wireguard_api.c +++ b/src/plugins/wireguard/wireguard_api.c @@ -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; diff --git a/src/plugins/wireguard/wireguard_if.c b/src/plugins/wireguard/wireguard_if.c index ed90146a800..c91667bb234 100644 --- a/src/plugins/wireguard/wireguard_if.c +++ b/src/plugins/wireguard/wireguard_if.c @@ -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); |