mss_clamp: coverity test fix 67/32967/3
authorMiklos Tirpak <miklos.tirpak@gmail.com>
Thu, 1 Jul 2021 15:26:34 +0000 (17:26 +0200)
committerDamjan Marion <dmarion@me.com>
Tue, 13 Jul 2021 17:19:51 +0000 (17:19 +0000)
Silence the coverity test with setting the interface index
in a variable before the boundary check.

Type: fix
Change-Id: I9bd6db08bfef93142581dada0b6a7d78b7de91e7
Signed-off-by: Miklos Tirpak <miklos.tirpak@gmail.com>
src/plugins/mss_clamp/mss_clamp_api.c

index 27804d6..c6557ea 100644 (file)
@@ -35,12 +35,15 @@ vl_api_mss_clamp_enable_disable_t_handler (
   mssc_main_t *cm = &mssc_main;
   vl_api_mss_clamp_enable_disable_reply_t *rmp;
   int rv;
+  u32 sw_if_index;
+
+  sw_if_index = ntohl (mp->sw_if_index);
 
   VALIDATE_SW_IF_INDEX (mp);
 
-  rv = mssc_enable_disable (ntohl (mp->sw_if_index), mp->ipv4_direction,
-                           mp->ipv6_direction, ntohs (mp->ipv4_mss),
-                           ntohs (mp->ipv6_mss));
+  rv =
+    mssc_enable_disable (sw_if_index, mp->ipv4_direction, mp->ipv6_direction,
+                        ntohs (mp->ipv4_mss), ntohs (mp->ipv6_mss));
 
   BAD_SW_IF_INDEX_LABEL;
   REPLY_MACRO (VL_API_MSS_CLAMP_ENABLE_DISABLE_REPLY);