<div dir="ltr">Hi David <div><br></div><div>Thanks, yeah should have looked into it. Should i send out a v3 with this change ?</div><div><br></div><div> - Balaji</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 30, 2014 at 4:58 PM, David Lamparter <span dir="ltr">&lt;<a href="mailto:david@opensourcerouting.org" target="_blank">david@opensourcerouting.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, Oct 27, 2014 at 05:05:53PM -0200, Everton Marques wrote:<br>
&gt; On Thu, Oct 23, 2014 at 2:25 PM, Balaji.G &lt;<a href="mailto:balajig81@gmail.com">balajig81@gmail.com</a>&gt; wrote:<br>
&gt; &gt; MBGP routes are used only for PIM RPF checks and hence should<br>
&gt; &gt; not be installed in the kernel&#39;s FIB. Ignore route node set to Multicast<br>
&gt; &gt; SAFI.<br>
<br>
</span>Almost forgot the &quot;Applied, thanks!&quot; on this.<br>
<br>
I moved down one bit:<br>
<span class=""><br>
&gt; &gt;<br>
&gt; &gt; Signed-off-by: Balaji.G &lt;<a href="mailto:balajig81@gmail.com">balajig81@gmail.com</a>&gt;<br>
&gt; &gt; ---<br>
&gt; &gt;  zebra/zebra_rib.c | 4 ++++<br>
&gt; &gt;  1 file changed, 4 insertions(+)<br>
&gt; &gt;<br>
&gt; &gt; diff --git a/zebra/zebra_rib.c b/zebra/zebra_rib.c<br>
&gt; &gt; index dc7e1ca..3f393d9 100644<br>
&gt; &gt; --- a/zebra/zebra_rib.c<br>
&gt; &gt; +++ b/zebra/zebra_rib.c<br>
&gt; &gt; @@ -1249,6 +1249,7 @@ rib_process (struct route_node *rn)<br>
&gt; &gt;    struct nexthop *nexthop = NULL, *tnexthop;<br>
&gt; &gt;    int recursing;<br>
&gt; &gt;    char buf[INET6_ADDRSTRLEN];<br>
<br>
</span>This:<br>
<span class=""><br>
&gt; &gt; +  rib_table_info_t *info = rn-&gt;table-&gt;info;<br>
&gt; &gt;<br>
&gt; &gt;    assert (rn);<br>
<br>
</span>is now below the assert, like:<br>
  rib_table_info_t *info;<br>
  assert (rn);<br>
  info = rn-&gt;table-&gt;info;<br>
<br>
because otherwise the assert is useless and will never trigger since<br>
it&#39;ll already crash on rn-&gt;table deref if rn == NULL.<br>
<br>
<br>
-David<br>
<span class=""><br>
&gt; &gt;<br>
&gt; &gt; @@ -1286,6 +1287,9 @@ rib_process (struct route_node *rn)<br>
&gt; &gt;        if (! nexthop_active_update (rn, rib, 0))<br>
&gt; &gt;          continue;<br>
&gt; &gt;<br>
&gt; &gt; +      if (info-&gt;safi == SAFI_MULTICAST)<br>
&gt; &gt; +       continue;<br>
&gt; &gt; +<br>
&gt; &gt;        /* Infinit distance. */<br>
&gt; &gt;        if (rib-&gt;distance == DISTANCE_INFINITY)<br>
&gt; &gt;          continue;<br>
&gt; &gt; --<br>
&gt; &gt; 1.9.1<br>
&gt; &gt;<br>
&gt; &gt;<br>
&gt; This fix is really needed.<br>
&gt; Only comment is the description could be more generic because it fixes MRIB<br>
&gt; routes installed from other sources than MBGP as well.<br>
&gt;<br>
&gt; Everton<br>
<br>
</span>&gt; _______________________________________________<br>
&gt; Quagga-dev mailing list<br>
&gt; <a href="mailto:Quagga-dev@lists.quagga.net">Quagga-dev@lists.quagga.net</a><br>
&gt; <a href="https://lists.quagga.net/mailman/listinfo/quagga-dev" target="_blank">https://lists.quagga.net/mailman/listinfo/quagga-dev</a><br>
<br>
</blockquote></div><br></div></div>