fix isssues discovered by coverity scan 08/3708/2
authorDamjan Marion <damarion@cisco.com>
Mon, 7 Nov 2016 13:05:34 +0000 (14:05 +0100)
committerChris Luke <chris_luke@comcast.com>
Mon, 7 Nov 2016 14:51:46 +0000 (14:51 +0000)
Change-Id: I62deaaa47a9259c886efa702eb4279c1c481585f
Signed-off-by: Damjan Marion <damarion@cisco.com>
vnet/vnet/devices/netmap/node.c
vnet/vnet/feature/feature.c

index 1e2a6a2..54d0aa1 100644 (file)
@@ -153,7 +153,7 @@ netmap_device_input_fn (vlib_main_t * vm, vlib_node_runtime_t * node,
 
          while (r && n_left_to_next)
            {
-             vlib_buffer_t *b0 = 0, *first_b0 = 0;
+             vlib_buffer_t *first_b0 = 0;
              u32 offset = 0;
              u32 bi0 = 0, first_bi0 = 0, prev_bi0;
              u32 next_slot_index = (cur_slot_index + 1) % ring->num_slots;
@@ -171,6 +171,7 @@ netmap_device_input_fn (vlib_main_t * vm, vlib_node_runtime_t * node,
 
              while (data_len && n_free_bufs)
                {
+                 vlib_buffer_t *b0;
                  /* grab free buffer */
                  u32 last_empty_buffer =
                    vec_len (nm->rx_buffers[cpu_index]) - 1;
@@ -233,7 +234,7 @@ netmap_device_input_fn (vlib_main_t * vm, vlib_node_runtime_t * node,
 
              /* redirect if feature path enabled */
              vnet_feature_start_device_input_x1 (nif->sw_if_index, &next0,
-                                                 b0, 0);
+                                                 first_b0, 0);
 
              /* enque and take next packet */
              vlib_validate_buffer_enqueue_x1 (vm, node, next_index, to_next,
index c19b3e0..c518cc6 100644 (file)
@@ -175,7 +175,7 @@ vnet_feature_enable_disable (const char *arc_name, const char *node_name,
 
   arc_index = vnet_get_feature_arc_index (arc_name);
 
-  if (arc_index == ~0)
+  if (arc_index == ~(u8) 0)
     return;
 
   cm = &fm->feature_config_mains[arc_index];