misc: fix coverity warnings 85/20485/2
authorDave Barach <dave@barachs.net>
Wed, 3 Jul 2019 19:21:50 +0000 (15:21 -0400)
committerFlorin Coras <florin.coras@gmail.com>
Wed, 3 Jul 2019 20:24:00 +0000 (20:24 +0000)
Type: fix
Ticket: VPP-1649

Change-Id: I93a393eca80065c379035478500e75e855f39b12
Signed-off-by: Dave Barach <dave@barachs.net>
src/plugins/hs_apps/vcl/sock_test_server.c
src/plugins/unittest/test_buffer.c
src/vat/api_format.c
src/vnet/ethernet/node.c

index 0714ccc..110ff3e 100644 (file)
@@ -343,7 +343,8 @@ af_unix_echo (void)
               af_unix_client_fd, nbytes, errno_val);
       goto done;
     }
-
+  /* Make the buffer is NULL-terminated. */
+  buffer[sizeof (buffer) - 1] = 0;
   printf ("SERVER (AF_UNIX): RX (%d bytes) - '%s'\n", rv, buffer);
 
   if (!strncmp (SOCK_TEST_MIXED_EPOLL_DATA, (const char *) buffer, nbytes))
index e5a5643..18938d8 100644 (file)
@@ -55,7 +55,7 @@ linearize_negative_current_data (vlib_main_t * vm)
       b->current_length = 14 + data_size;
     }
 
-  vlib_buffer_chain_linearize (vm, vlib_get_buffer (vm, bi[0]));
+  (void) vlib_buffer_chain_linearize (vm, vlib_get_buffer (vm, bi[0]));
 
   return 0;
 }
index e0d77a4..5bba7ef 100644 (file)
@@ -18921,7 +18921,7 @@ static void
 vl_api_mpls_route_details_t_handler (vl_api_mpls_route_details_t * mp)
 {
   vat_main_t *vam = &vat_main;
-  int count = ntohl (mp->mr_route.mr_n_paths);
+  int count = (int) clib_net_to_host_u32 (mp->mr_route.mr_n_paths);
   vl_api_fib_path_t *fp;
   int i;
 
@@ -18941,7 +18941,7 @@ static void vl_api_mpls_route_details_t_handler_json
   (vl_api_mpls_route_details_t * mp)
 {
   vat_main_t *vam = &vat_main;
-  int count = ntohl (mp->mr_route.mr_n_paths);
+  int count = (int) clib_host_to_net_u32 (mp->mr_route.mr_n_paths);
   vat_json_node_t *node = NULL;
   vl_api_fib_path_t *fp;
   int i;
index 286ec3c..eabcf42 100755 (executable)
@@ -2063,6 +2063,11 @@ ethernet_register_input_type (vlib_main_t * vm,
   }
 
   ti = ethernet_get_type_info (em, type);
+  if (ti == 0)
+    {
+      clib_warning ("type_info NULL for type %d", type);
+      return;
+    }
   ti->node_index = node_index;
   ti->next_index = vlib_node_add_next (vm,
                                       ethernet_input_node.index, node_index);