From bf5335cf3421811cd419d9b40be69d2ad9ed8bde Mon Sep 17 00:00:00 2001 From: Donatas Abraitis Date: Mon, 4 Oct 2021 19:26:20 +0300 Subject: pimd: Do not explicitly set the thread pointer to NULL FRR should only ever use the appropriate THREAD_ON/THREAD_OFF semantics. This is espacially true for the functions we end up calling the thread for. Signed-off-by: Donatas Abraitis --- pimd/pim_igmp.c | 2 -- 1 file changed, 2 deletions(-) (limited to 'pimd/pim_igmp.c') diff --git a/pimd/pim_igmp.c b/pimd/pim_igmp.c index 50de7124d..795c96c83 100644 --- a/pimd/pim_igmp.c +++ b/pimd/pim_igmp.c @@ -671,7 +671,6 @@ void pim_igmp_general_query_on(struct igmp_sock *igmp) ifaddr_str, query_interval, startup_mode ? "startup" : "non-startup", igmp->fd); } - igmp->t_igmp_query_timer = NULL; thread_add_timer(router->master, pim_igmp_general_query, igmp, query_interval, &igmp->t_igmp_query_timer); } @@ -1052,7 +1051,6 @@ static void igmp_read_on(struct igmp_sock *igmp) zlog_debug("Scheduling READ event on IGMP socket fd=%d", igmp->fd); } - igmp->t_igmp_read = NULL; thread_add_read(router->master, pim_igmp_read, igmp, igmp->fd, &igmp->t_igmp_read); } -- cgit v1.2.3