infrastation at yandex
Apr 16, 2012, 3:28 AM
Post #2 of 2
07.04.2012, 16:58, "Joakim Tjernlund" <joakim.tjernlund [at] transmode>:
Re: zebra: fix ifindex test condition (BZ#487)
[In reply to]
> 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
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.
Quagga-dev mailing list
Quagga-dev [at] lists