From: Joakim Tjernlund <joakim.tjernlund@transmode.se>
Date: Wed, 14 Apr 2010 14:01:25 +0000 (+0200)
Subject: ospfd: Only refresh external default route once.
X-Git-Tag: frr-2.0-rc1~2156
X-Git-Url: https://git.puffer.fish/?a=commitdiff_plain;h=46154fe3675df5417a712285b66fd12bd1b4fc6d;p=mirror%2Ffrr.git

ospfd: Only refresh external default route once.

* ospf_zebra.c: (ospf_distribute_list_update_timer) forces a
  refresh of default route each time it finds a default prefix.
  This is suboptimal, just record that it needs to be done and
  do it once.

Signed-off-by: Joakim Tjernlund <Joakim.Tjernlund@transmode.se>
---

diff --git a/ospfd/ospf_zebra.c b/ospfd/ospf_zebra.c
index 462f67c819..50ca85e167 100644
--- a/ospfd/ospf_zebra.c
+++ b/ospfd/ospf_zebra.c
@@ -929,7 +929,7 @@ ospf_distribute_list_update_timer (struct thread *thread)
   struct external_info *ei;
   struct route_table *rt;
   struct ospf_lsa *lsa;
-  int type;
+  int type, default_refresh = 0;
   struct ospf *ospf;
 
   ospf = ospf_lookup ();
@@ -950,13 +950,15 @@ ospf_distribute_list_update_timer (struct thread *thread)
 	if ((ei = rn->info) != NULL)
 	  {
 	    if (is_prefix_default (&ei->p))
-	      ospf_external_lsa_refresh_default (ospf);
+	      default_refresh = 1;
 	    else if ((lsa = ospf_external_info_find_lsa (ospf, &ei->p)))
 	      ospf_external_lsa_refresh (ospf, lsa, ei, LSA_REFRESH_IF_CHANGED);
 	    else
 	      ospf_external_lsa_originate (ospf, ei);
 	  }
     }
+  if (default_refresh)
+    ospf_external_lsa_refresh_default (ospf);
   return 0;
 }