]> git.puffer.fish Git - matthieu/frr.git/commit
ripd: split-horizon default differed between rip_interface_new and _reset
authorPaul Jakma <paul@jakma.org>
Wed, 25 May 2016 13:47:00 +0000 (14:47 +0100)
committerDonald Sharp <sharpd@cumulusnetworks.com>
Fri, 23 Sep 2016 16:12:16 +0000 (12:12 -0400)
commit1dec2166deabab95572bcf68d4e73a00460bc32e
treeaa6c17004924d6d310babfa2e6e7e7cea8127cd2
parent8d9e99a6917774bbc2ca0d90b39c26d866510964
ripd: split-horizon default differed between rip_interface_new and _reset

* rip_interface.c: Default for split_horizon_default differed between
  rip_interface_new and rip_interface_reset, causing at least some issues
  after interface events. See patchwork #604. Fix, and consolidate code.

  (rip_interface_{reset,clean}) rename these to 'interface', as that's more
  appropriate.  Spin the ri specific bodies of these functions out to
  rip_interface_{reset,clean} helpers.  Factor out the overlaps, so
  rip_interface_reset uses rip_interface_clean.

  (rip_interface_new) just use rip_interface_reset.

* ripd.h: Update for (rip_interface_{reset,clean})

Reported by xufeng zhang, with a suggested fix on which this commit expands.
See patchwork #604.  This commit addresses only the split-horizon
discrepency, issue #2.  The other issue they reported, #1, is not addressed,
though suggested fix seems inappropriate.

Cc: xufeng.zhang@windriver.com
ripd/rip_interface.c
ripd/ripd.c
ripd/ripd.h