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 --- vpp/vpp-api/api.c | 25 +++++++++++++++---------- 1 file changed, 15 insertions(+), 10 deletions(-) (limited to 'vpp') 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