CVSROOT: /cvs
Module name: src
Changes by: [email protected] 2023/05/15 03:31:14
Modified files:
usr.sbin/bgpctl: Tag: OPENBSD_7_3 bgpctl.8 parser.c
usr.sbin/bgpd : Tag: OPENBSD_7_3 rde_filter.c rtr.c
usr.sbin/bgplgd: Tag: OPENBSD_7_3 qs.c
Log message:
Rename 'invalid' to 'disqualified' in the 'show rib' table.
'invalid' conflicts with the 'in' keyword and the parser is not smart
enough to handle this. As a secondary benefit the term 'invalid' is
less overloaded. There are various reasons why prefixes are not eligible
in the route decision process calling them all 'invalid' is a bit harsh.
job@, tb@ and benno@ agree
from claudio
Properly handle references for nexthops and labels in filterset_copy()
Up until recently filterset_copy() was only used in the parent process
where none of those references exist but by using filterset_copy() in
the RDE to populate per-peer output filters the code needs to handle
such references. Without this rules like 'match to any set nexthop X'
will cause internal corruption on config reloads and session resets.
Bug report from Marko Cupac (marko.cupac (at) mimar.rs)
OK tb@
from claudio
Pass ASPA objects in reverse since this is what aspa_add_set() expects.
OK tb@
from claudio
Adjust bgplgd after renaming of the invalid option in bgpctl.
This does not change the query string argument. We may do this
at a later stage.
OK tb@
from claudio
This is errata/7.3/002_bgpd.patch.sig