On Sun, Sep 30, 2018 at 12:33:44PM +0200, Denis Fondras wrote:
> On Sun, Sep 30, 2018 at 12:08:36PM +0200, Claudio Jeker wrote:
> > This is wrong. It should be initialized to:
> > vstate = p->validation_state;
> >
>
>
> Index: rde.c
> ===================================================================
> RCS file: /cvs/src/usr.sbin/bgpd/rde.c,v
> retrieving revision 1.430
> diff -u -p -r1.430 rde.c
> --- rde.c 29 Sep 2018 08:11:11 -0000 1.430
> +++ rde.c 30 Sep 2018 10:27:27 -0000
> @@ -3153,7 +3153,7 @@ rde_softreconfig_in(struct rib_entry *re
> if (action == ACTION_ALLOW) {
> /* update Local-RIB */
> path_update(&rib->rib, peer, &state, &prefix,
> - pt->prefixlen, vstate);
> + pt->prefixlen, p->validation_state);
> } else if (action == ACTION_DENY) {
> /* remove from Local-RIB */
> prefix_remove(&rib->rib, peer, &prefix,
>
>
Even better. OK claudio@
--
:wq Claudio
