Yes. I believe it wasn't there until the second version of the branch, so didn't see it at first. But the second patch is still needed. Well I would like to see for regression results on it from Martin, as I only tested bgpd and it does have some zebra implications.

Lou




On August 2, 2016 8:50:11 AM Paul Jakma <[email protected]> wrote:

I'd already applied this based on the other email, made up my own commit
subject for you:

http://git.savannah.gnu.org/gitweb/?p=quagga.git;a=commitdiff;h=811586ad4cfedd5b5d6d6c1f8a2d674b84ba637e

Presumably that's OK?

regards,

Paul

On Mon, 1 Aug 2016, Lou Berger wrote:

   bgpd, doc, lib, zebra: nexthop-tracking in zebra
---
bgpd/bgp_route.c | 5 -----
1 file changed, 5 deletions(-)

diff --git a/bgpd/bgp_route.c b/bgpd/bgp_route.c
index a88a022..4cb6c14 100644
--- a/bgpd/bgp_route.c
+++ b/bgpd/bgp_route.c
@@ -3778,14 +3778,9 @@ bgp_static_withdraw (struct bgp *bgp, struct prefix *p, afi_t afi,
{
  struct bgp_node *rn;
  struct bgp_info *ri;
-  struct bgp_info *new;

  /* Make new BGP info. */
  rn = bgp_node_get (bgp->rib[afi][safi], p);
-  new = info_make(ZEBRA_ROUTE_BGP, BGP_ROUTE_STATIC, bgp->peer_self,
-                 bgp_attr_default_intern(BGP_ORIGIN_IGP), rn);
-
-  SET_FLAG (new->flags, BGP_INFO_VALID);

  /* Check selected route and self inserted route. */
  for (ri = rn->info; ri; ri = ri->next)


--
Paul Jakma | [email protected] | @pjakma | Key ID: 0xD86BF79464A2FF6A
Fortune:
Every word is like an unnecessary stain on silence and nothingness.
                -- Beckett




_______________________________________________
Quagga-dev mailing list
[email protected]
https://lists.quagga.net/mailman/listinfo/quagga-dev

Reply via email to