Skip to content

W
White Rabbit Switch - Software
Commit d8f6deab authored by Grzegorz Daniluk's avatar Grzegorz Daniluk
Browse files
Options

wrsw_rtud: fix missing loop brackets, otherwise we don't really check if the port is down

parent ee9933d2
Hide whitespace changes
Inline Side-by-side
Showing with 2 additions and 1 deletion
userspace/wrsw_rtud/rtud.c
View file @ d8f6deab
...@@ -228,7 +228,7 @@ static int rtu_daemon_learning_process(void) ...@@ -228,7 +228,7 @@ static int rtu_daemon_learning_process(void)
req.has_vid ? req.vid : 0, req.has_vid ? req.vid : 0,
req.has_prio ? req.prio : 0); req.has_prio ? req.prio : 0);
for (port_down = i = 0; i <= MAX_PORT; i++) for (port_down = i = 0; i <= MAX_PORT; i++) {
p = &hal_ports_local_copy[i]; p = &hal_ports_local_copy[i];
if (p->in_use && p->hw_index == req.port_id if (p->in_use && p->hw_index == req.port_id
&& !state_up(p->state)) { && !state_up(p->state)) {
...@@ -236,6 +236,7 @@ static int rtu_daemon_learning_process(void) ...@@ -236,6 +236,7 @@ static int rtu_daemon_learning_process(void)
pr_info("port down %d\n", i); pr_info("port down %d\n", i);
break; break;
} }
}
/* don't learn on ports that are down (FIFO tail?) */ /* don't learn on ports that are down (FIFO tail?) */
if (port_down) if (port_down)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment