diff options
| author |   Dave Barach <dave@barachs.net> | 2016-11-22 15:23:41 -0500 |
|---|---|---|
| committer |   Damjan Marion <dmarion.lists@gmail.com> | 2016-11-22 21:25:47 +0000 |
| commit | 7afe9e38269a30682a5e392b0e876e18d1465c31 (patch) | |
| tree | 8d627653a17e8a1c651975c77f568f4ff4d69b0e /vpp | |
| parent | d28bc62a726c51bcfb9b90a6dd381b4727421004 (diff) | |
Fix coverity warnings, VPP-486
Change-Id: I4ec49e116fdb418ebf9d84000f2a0cec1cf78b14
Signed-off-by: Dave Barach <dave@barachs.net>
Diffstat (limited to 'vpp')
| -rw-r--r-- | vpp/vpp-api/api.c | 25 |
1 files changed, 15 insertions, 10 deletions
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); } |