CVSROOT: /cvs Module name: src Changes by: t...@cvs.openbsd.org 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