summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDonald Sharp <sharpd@nvidia.com>2023-01-26 16:44:52 +0100
committerDonald Sharp <sharpd@nvidia.com>2023-01-26 17:40:33 +0100
commit9ca6d3db2b5c44fa38792a9a1000873ac264fcc7 (patch)
tree74d733d0538ee0ac946267d873bbd24c29f0a26a
parentzebra: i declaration shadows other i declared (diff)
downloadfrr-9ca6d3db2b5c44fa38792a9a1000873ac264fcc7.tar.xz
frr-9ca6d3db2b5c44fa38792a9a1000873ac264fcc7.zip
watchfrr: dmn variable shadows previous declaration
There is no need to declar the same temp variable 2 times. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
-rw-r--r--watchfrr/watchfrr.c11
1 files changed, 4 insertions, 7 deletions
diff --git a/watchfrr/watchfrr.c b/watchfrr/watchfrr.c
index 4a3575ae7..2b0fec1b2 100644
--- a/watchfrr/watchfrr.c
+++ b/watchfrr/watchfrr.c
@@ -941,13 +941,10 @@ static void phase_check(void)
if (!IS_UP(gs.special))
break;
zlog_info("Phased restart: %s is now up.", gs.special->name);
- {
- struct daemon *dmn;
- for (dmn = gs.daemons; dmn; dmn = dmn->next) {
- if (dmn != gs.special)
- run_job(&dmn->restart, "start",
- gs.start_command, 1, 0);
- }
+ for (dmn = gs.daemons; dmn; dmn = dmn->next) {
+ if (dmn != gs.special)
+ run_job(&dmn->restart, "start",
+ gs.start_command, 1, 0);
}
gs.phase = PHASE_NONE;
THREAD_OFF(gs.t_phase_hanging);