On 4/7/2016 2:12 PM, Christian Franke wrote:
On 04/07/2016 12:26 PM, Igor Ryzhov wrote:
I studied zebra_rib.c history and found one strange change in commit 0abf6796
(http://git.savannah.gnu.org/cgit/quagga.git/commit/?id=0abf6796c3d8ae8f5ea8624668424bc1554de25e).
Before this patch there were two different functions for installing and
uninstalling kernel routes - rib_install_kernel and rib_uninstall_kernel.
At the start of rib_install_kernel there was code block:
[...]
So now in one of scenarios UNSET_FLAG is used instead of SET_FLAG.
I don't know zebra's source code well so I can't say that this change is a
mistake, but I think it may be, so, please, check it.
Maybe it also connected with my problem somehow?
I think the new code is correct and old code was wrong. In either case
it should not concern your route since your route is unicast and that
branch is only taken for non-unicast routes.
I agree. The new code is correct.
--Jafar
-Christian
_______________________________________________
Quagga-dev mailing list
[email protected]
https://lists.quagga.net/mailman/listinfo/quagga-dev
_______________________________________________
Quagga-dev mailing list
[email protected]
https://lists.quagga.net/mailman/listinfo/quagga-dev