From 5237c77cdd8041f173ceefe4919cd7e9e1130805 Mon Sep 17 00:00:00 2001 From: Neale Ranns Date: Wed, 21 Mar 2018 16:34:28 -0700 Subject: IGMP: coverity fixes and remove checks for scapy IGMPv3 Change-Id: Ic2eddc803f9ba8215e37388a686004830211cf6f Signed-off-by: Neale Ranns --- src/plugins/igmp/igmp.c | 12 ++++-------- src/plugins/igmp/igmp.h | 2 +- 2 files changed, 5 insertions(+), 9 deletions(-) (limited to 'src/plugins') diff --git a/src/plugins/igmp/igmp.c b/src/plugins/igmp/igmp.c index 97baae57f1d..e98e976cc96 100644 --- a/src/plugins/igmp/igmp.c +++ b/src/plugins/igmp/igmp.c @@ -206,7 +206,7 @@ ip4_lookup (ip4_address_t * a, igmp_membership_report_v3_t * igmp, u16 n, for (i = 0; i < n; i++) { - if ((!ip4_address_compare (a, &group_ptr (igmp, l)->dst_address)) && + if ((!ip4_address_compare (a, &(group_ptr (igmp, l)->dst_address))) && (type == group_ptr (igmp, l)->type)) { rv = 1; @@ -549,12 +549,11 @@ igmp_timer_process (vlib_main_t * vm, vlib_node_runtime_t * rt, igmp_main_t *im = &igmp_main; uword *event_data = 0, event_type; f64 time_start; - u8 enabled = 0; igmp_timer_t *timer = NULL; while (1) { - if (enabled) + if (NULL != timer) vlib_process_wait_for_event_or_clock (vm, timer->exp_time - time_start); else @@ -571,14 +570,11 @@ igmp_timer_process (vlib_main_t * vm, vlib_node_runtime_t * rt, DBG ("time: %f", vlib_time_now (vm)); /* timer expired */ - timer->func (vm, rt, im, timer); + if (NULL != timer) + timer->func (vm, rt, im, timer); next_timer: timer = igmp_get_next_timer (im); - if (timer == NULL) - enabled = 0; - else - enabled = 1; } return 0; } diff --git a/src/plugins/igmp/igmp.h b/src/plugins/igmp/igmp.h index c98cbd1dc0b..0dcf11f64f7 100644 --- a/src/plugins/igmp/igmp.h +++ b/src/plugins/igmp/igmp.h @@ -36,7 +36,7 @@ #define DBG(...) #endif /* IGMP_DBG */ -#define group_ptr(p, l) ((igmp_membership_group_v3_t *)((void*)p + l)) +#define group_ptr(p, l) ((igmp_membership_group_v3_t *)((char*)p + l)) enum { -- cgit 1.2.3-korg