diff options
| author | Donald Sharp <sharpd@nvidia.com> | 2022-03-07 11:28:28 -0500 |
|---|---|---|
| committer | Donald Sharp <sharpd@nvidia.com> | 2022-03-07 11:28:28 -0500 |
| commit | e3015d915b1fa7016647dbb195611e510a9942f0 (patch) | |
| tree | 0f2ec8f8a9c72c1825de564019213433f77587bf /lib/printf/printf-pos.c | |
| parent | 583ba572b7d4c486d1aba86de864e142cea433a2 (diff) | |
bgpd: Fix continue/break change from old commit
Commit: ea47320b1d0eeaa56f945fa356da7e4ca7f2b0b2
Modified the bgp_clear_stale_route function to have
better indentation, but in the process changed some
`continue;` statements to `break;` which modified
the looping and caused stale paths to not always be
removed upon an update.
To reproduce: A ---- B, setup with addpath and GR
One side has a prefix with nhop1 and nhop2, kill one
side and then resend the same prefix with nhop3,
paths nhop1 and 2 become stale and never removed.
Code inspection clearly shows that that `continue`
statements became `break` statements causing the
loop over all paths to stop prematurely.
The fix is to change the break back to continue
statements so the loop can continue instead of
stopping.
Signed-off-by: Donald Sharp <sharpd@nvidia.com>
Diffstat (limited to 'lib/printf/printf-pos.c')
0 files changed, 0 insertions, 0 deletions
