From 7afe9e38269a30682a5e392b0e876e18d1465c31 Mon Sep 17 00:00:00 2001 From: Dave Barach Date: Tue, 22 Nov 2016 15:23:41 -0500 Subject: Fix coverity warnings, VPP-486 Change-Id: I4ec49e116fdb418ebf9d84000f2a0cec1cf78b14 Signed-off-by: Dave Barach --- vlib/vlib/threads_cli.c | 8 ++++---- vnet/vnet/l2tp/decap.c | 4 ++-- vpp/vpp-api/api.c | 25 +++++++++++++++---------- 3 files changed, 21 insertions(+), 16 deletions(-) diff --git a/vlib/vlib/threads_cli.c b/vlib/vlib/threads_cli.c index aef67576751..6651a4dd671 100644 --- a/vlib/vlib/threads_cli.c +++ b/vlib/vlib/threads_cli.c @@ -444,9 +444,9 @@ test_frame_queue_nelts (vlib_main_t * vm, unformat_input_t * input, while (unformat_check_input (line_input) != UNFORMAT_END_OF_INPUT) { - if (unformat (line_input, "nelts %u"), &nelts) + if (unformat (line_input, "nelts %u", &nelts)) ; - else if (unformat (line_input, "index %u"), &index) + else if (unformat (line_input, "index %u", &index)) ; else return clib_error_return (0, "parse error: '%U'", @@ -511,9 +511,9 @@ test_frame_queue_threshold (vlib_main_t * vm, unformat_input_t * input, while (unformat_check_input (line_input) != UNFORMAT_END_OF_INPUT) { - if (unformat (line_input, "threshold %u"), &threshold) + if (unformat (line_input, "threshold %u", &threshold)) ; - else if (unformat (line_input, "index %u"), &index) + else if (unformat (line_input, "index %u", &index)) ; else return clib_error_return (0, "parse error: '%U'", diff --git a/vnet/vnet/l2tp/decap.c b/vnet/vnet/l2tp/decap.c index ee307b642d8..e8986935e93 100644 --- a/vnet/vnet/l2tp/decap.c +++ b/vnet/vnet/l2tp/decap.c @@ -218,8 +218,8 @@ done: else { /* Go to next node on the ip6 configuration chain */ - ASSERT (session); - vnet_feature_next (session->sw_if_index, &next_index, b); + if (PREDICT_TRUE (session != 0)) + vnet_feature_next (session->sw_if_index, &next_index, b); } } diff --git a/vpp/vpp-api/api.c b/vpp/vpp-api/api.c index 330b9159e19..91726e163b6 100644 --- a/vpp/vpp-api/api.c +++ b/vpp/vpp-api/api.c @@ -9166,14 +9166,14 @@ vl_api_ipsec_spd_dump_t_handler (vl_api_ipsec_spd_dump_t * mp) spd_index = p[0]; spd = pool_elt_at_index (im->spds, spd_index); - pool_foreach (policy, spd->policies, ( - { - if (mp->sa_id == ~(0) - || ntohl (mp->sa_id) == - policy->sa_id) - send_ipsec_spd_details (policy, q, - mp->context);} - )); + /* *INDENT-OFF* */ + pool_foreach (policy, spd->policies, + ({ + if (mp->sa_id == ~(0) || ntohl (mp->sa_id) == policy->sa_id) + send_ipsec_spd_details (policy, q, + mp->context);} + )); + /* *INDENT-ON* */ #else clib_warning ("unimplemented"); #endif @@ -9184,9 +9184,12 @@ vl_api_feature_enable_disable_t_handler (vl_api_feature_enable_disable_t * mp) { vl_api_feature_enable_disable_reply_t *rmp; int rv = 0; + u8 *arc_name, *feature_name; - u8 *arc_name = format (0, "%s%c", mp->arc_name, 0); - u8 *feature_name = format (0, "%s%c", mp->feature_name, 0); + VALIDATE_SW_IF_INDEX (mp); + + arc_name = format (0, "%s%c", mp->arc_name, 0); + feature_name = format (0, "%s%c", mp->feature_name, 0); vnet_feature_registration_t *reg; reg = @@ -9216,6 +9219,8 @@ vl_api_feature_enable_disable_t_handler (vl_api_feature_enable_disable_t * mp) vec_free (feature_name); vec_free (arc_name); + BAD_SW_IF_INDEX_LABEL; + REPLY_MACRO (VL_API_FEATURE_ENABLE_DISABLE_REPLY); } -- cgit 1.2.3-korg