From: Donatas Abraitis <donatas.abraitis@gmail.com>
Date: Mon, 4 Oct 2021 14:49:06 +0000 (+0300)
Subject: bgpd: Don't set NULL for struct thread at bgp_dump_interval_func()
X-Git-Tag: base_8.2~358^2
X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=a33842f08dbbb7bfdd5eab2439a8d202e70f39ff;p=mirror%2Ffrr.git

bgpd: Don't set NULL for struct thread at bgp_dump_interval_func()

NULL is already handled by thread_fetch().

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
---

diff --git a/bgpd/bgp_dump.c b/bgpd/bgp_dump.c
index 299ee305be..2f0b87aa3a 100644
--- a/bgpd/bgp_dump.c
+++ b/bgpd/bgp_dump.c
@@ -165,13 +165,11 @@ static int bgp_dump_interval_add(struct bgp_dump *bgp_dump, int interval)
 			interval = interval
 				   - secs_into_day % interval; /* always > 0 */
 		}
-		bgp_dump->t_interval = NULL;
 		thread_add_timer(bm->master, bgp_dump_interval_func, bgp_dump,
 				 interval, &bgp_dump->t_interval);
 	} else {
 		/* One-off dump: execute immediately, don't affect any scheduled
 		 * dumps */
-		bgp_dump->t_interval = NULL;
 		thread_add_event(bm->master, bgp_dump_interval_func, bgp_dump,
 				 0, &bgp_dump->t_interval);
 	}
@@ -453,7 +451,6 @@ static int bgp_dump_interval_func(struct thread *t)
 {
 	struct bgp_dump *bgp_dump;
 	bgp_dump = THREAD_ARG(t);
-	bgp_dump->t_interval = NULL;
 
 	/* Reschedule dump even if file couldn't be opened this time... */
 	if (bgp_dump_open_file(bgp_dump) != NULL) {