diff options
| author |   Dave Barach <dave@barachs.net> | 2017-09-13 09:18:08 -0400 |
|---|---|---|
| committer | Dave Barach <dave@barachs.net> | 2017-09-13 10:04:20 -0400 |
| commit | 0eacfa32687730ae6d7dbc67201165fac26682d6 (patch) | |
| tree | 6a78686bfd0c816401ceca46dc6fd87e77074a5f /src/vpp/stats/stats.c | |
| parent | 68ec940a469be5e1696fa6bb8dd4cea54d092796 (diff) | |
VPP-972: fix coverity warnings in the stats scraper
Change-Id: I8843e57105c6c29bcdaf1e6e3bf8e9caf6bfd2ec
Signed-off-by: Dave Barach <dave@barachs.net>
Diffstat (limited to 'src/vpp/stats/stats.c')
| -rw-r--r-- | src/vpp/stats/stats.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/vpp/stats/stats.c b/src/vpp/stats/stats.c index c78a8aef3cd..ff52c4dbfc6 100644 --- a/src/vpp/stats/stats.c +++ b/src/vpp/stats/stats.c @@ -1495,7 +1495,7 @@ static void } rp = pool_elt_at_index (sm->stats_registrations, p[0]); - if (!rp->stats_registrations & INTERFACE_COMBINED_COUNTERS) // Client but doesn't want this. + if (!(rp->stats_registrations & INTERFACE_COMBINED_COUNTERS)) // Client but doesn't want this. { clib_warning ("pid %d: already disabled for interface COMBINED stats...", @@ -1576,7 +1576,7 @@ vl_api_want_ip4_fib_stats_t_handler (vl_api_want_ip4_fib_stats_t * mp) } rp = pool_elt_at_index (sm->stats_registrations, p[0]); - if (!rp->stats_registrations & IP4_FIB_COUNTERS) // Client but doesn't want this. + if (!(rp->stats_registrations & IP4_FIB_COUNTERS)) // Client but doesn't want this. { clib_warning ("pid %d: already disabled for interface ip4 fib...", mp->pid); @@ -1656,7 +1656,7 @@ vl_api_want_ip6_fib_stats_t_handler (vl_api_want_ip6_fib_stats_t * mp) } rp = pool_elt_at_index (sm->stats_registrations, p[0]); - if (!rp->stats_registrations & IP6_FIB_COUNTERS) // Client but doesn't want this. + if (!(rp->stats_registrations & IP6_FIB_COUNTERS)) // Client but doesn't want this. { clib_warning ("pid %d: already disabled for interface ip6 fib...", mp->pid); |