infrastation at yandex
Apr 16, 2012, 3:28 AM
Post #2 of 2
(111 views)
Permalink
|
|
Re: zebra: fix ifindex test condition (BZ#487)
[In reply to]
|
|
07.04.2012, 16:58, "Joakim Tjernlund" <joakim.tjernlund [at] transmode>:
> http://git.savannah.gnu.org/cgit/quagga.git/commit/?id=4f1735fd6ac5d0881bafa9bd421e00645b6c60fd
>
> Was browsing Quagga repo to see the latest fixes and found the above commit.
> Sadly that commit does not fix much, it only deals with ifindex for connected routes.
> Either have a look at my patch I sent long ago(Don't delete too many routes) or
> slap a
> if (ifindex && ifindex != nexthop->ifindex)
> continue; /* ifindex doesn't match */
> before testing anything else
Hello, Jocke.
I made this commit back then judging by the feedback from #487, the case probably even reproduced locally. Of course, there is always a place for a mistake, I will review the change with your arguments considered.
--
Denis Ovsienko
_______________________________________________
Quagga-dev mailing list
Quagga-dev [at] lists
http://lists.quagga.net/mailman/listinfo/quagga-dev
|
