summaryrefslogtreecommitdiffstats
path: root/snapcraft
diff options
context:
space:
mode:
authorDonald Sharp <sharpd@nvidia.com>2022-03-07 17:28:28 +0100
committerDonald Sharp <sharpd@nvidia.com>2022-03-07 17:28:28 +0100
commite3015d915b1fa7016647dbb195611e510a9942f0 (patch)
tree0f2ec8f8a9c72c1825de564019213433f77587bf /snapcraft
parentMerge pull request #10732 from anlancs/zebra-minor-move (diff)
downloadfrr-e3015d915b1fa7016647dbb195611e510a9942f0.tar.xz
frr-e3015d915b1fa7016647dbb195611e510a9942f0.zip
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 'snapcraft')
0 files changed, 0 insertions, 0 deletions