Merge pull request #5900 from rubenk/bgpd-fix-typo

bgpd: fix typo in 'show bgp neighbors' output

Merge pull request #5886 from sworleys/Trust-Kernel-Direct-Routes

zebra: trust directly connected kernel/system routes

Merge pull request #5873 from mjstapp/zebra_lsp_dtor

zebra: add a free api for LSPs

bgpd: fix typo in 'show bgp neighbors' output

And fix a typo in a comment while we're at it

Signed-off-by: Ruben Kerkhof <ruben@rubenkerkhof.com>

Merge pull request #5890 from mjstapp/zapi_allow_label_num

zebra: allow multiple labels in LSP zapi message

Merge pull request #5897 from patrasar/2415676

pimd: fix crash seen while executing igmp related configuration

Merge pull request #5893 from donaldsharp/clang_mc_clang

ldpd: Remove double set of lif

pimd: fix crash seen while executing igmp related configuration

Issue 1:
1. Enable pim on an interface.
2. Configure query-interval or query max response time,
which results in pimd crash.

Root cause:
1. When pim is enabled on an interface, it creates a igmp socket
with querier_timer and other_querier time as NULL.
2. When query-interval/max_response_time is configured, it call the
function igmp_sock_query_reschedule() to reshedule the query. This
function check either of querier_timer or other_querier timer should
be running. Since in this case both are NULL, it results in crash.

Issue 2:
1. Enable pim on an interface.
2. Execute no ip igmp query-interval or query max response time,
which results in pimd crash.

Root cause:
1. When pim is enabled on an interface, it creates a pim interface
with querier_timer and other_querier time as NULL.
2. When no ip igmp query-interval/max_response_time is executed, it will
check either of querier_timer or other_querier timer should be running.
Since in this case both are NULL, it results in crash.

Fix:
When pim is enabled on an interface, it creates a igmp socket with
mtrace_only as true. So add a check if mtrace_only is true, then don't
reshedule the query.

Signed-off-by: Sarita Patra <saritap@vmware.com>

ldpd: Remove double set of lif

The lif variable was being set in the if statement and
immediately copied into from xf.  No need to do this
twice.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

zebra: allow multiple labels in LSP zapi message

The handlers for a couple of the main LSP-oriented zapi
messages explicitly limited themselves to a single out-label.
Allow multiple labels if the sender ... sends them.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

zebra: trust directly connected kernel/system routes

We made the decision to explicitly trust kernel and system routes
of every other type with 058c16b7e239f1c50a1d4b4376a6aff6b8dad959.

So, we should trust directly connected routes the same way, assuming
the interface exists.

Old Behavior:

K   2.2.2.1/32 [0/0] is directly connected, unknown inactive, 00:00:39

New Behavior:

K>* 2.2.2.1/32 [0/0] is directly connected, test1, 00:00:03

As a bonus, this fixes the issues we were seeing with not removing
directly connected routes of certain interface types when
those interfaces go down/are deleted.

Signed-off-by: Stephen Worley <sworley@cumulusnetworks.com>

Merge pull request #5855 from ton31337/fix/allow_using_add_sub_for_local-preference

bgpd: Allow using add/subtract for local-preference in route-maps

Merge pull request #5780 from chiragshah6/evpn_dev1

zebra:remote vtep delete handling

zebra: add a free api for LSPs

consolidate some LSP cleanup code into a common free function.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

Merge pull request #5874 from donaldsharp/hash_backet2

*: change hash_backet to hash_bucket

Merge pull request #5822 from rbauduin/patch-2

doc: Document attribute-unchanged directive

Merge pull request #5878 from mjstapp/fix_nhg_for_fpm

zebra: fix nexthop_group conversion in fpm code

*: change hash_backet to hash_bucket

It's been a year search and destroy.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>

Merge pull request #5862 from sworleys/PBR-No-Table-Range

pbrd: unconfigure table range

Merge pull request #5876 from ton31337/feature/doc_vrrpd_update

doc: Change interface name for vrrp example

zebra: fix nexthop_group conversion in fpm code

Recent commit that embedded the nhg_hash_entry's group
missed a couple of fpm modules.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

Merge pull request #5859 from donaldsharp/clang_latest

Clean up clang latest SA warnings

tests: Add a test case for `set local-preference +/-LOCAL_PREF`

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

doc: Use +/- for `set local-preference` to add/subtract the value

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

bgpd: Allow using add/subtract for local-preference in route-maps

We already have a generic support for add/sub in route-maps. It's already
handled in route_value_compile().

Just convert to string (allow passing (-) minus sign) - works like expected.

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

doc: Change interface name for vrrp example

Looks like copy-paste error.

Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>

Merge pull request #5875 from donaldsharp/support_rfcs

doc: Add notice of rfc-4893 in supported rfc's.

Merge pull request #5857 from mjstapp/embed_nhg_in_nhe

zebra,lib: Embed lib nexthop-group in zebra hash entry

doc: Add notice of rfc-4893 in supported rfc's.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

zebra: Embed lib nexthop-group in zebra hash entry

Embed nexthop-group, which is just a pointer, in the zebra
nexthop-hash-entry object, rather than mallocing one.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

Merge pull request #5872 from Jafaral/backet

Revert "*: change hash_backet to hash_bucket"

Revert "*: change hash_backet to hash_bucket"

This reverts commit 3895c42a2eb534a70ef25412547e798c1c1ef5a1.

LabN CI update needs to be coordinated before merging this

Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>

ldpd: Cleanup indentation in merge_nbrps

We had a very deeply nested function, reduce the indentation
for easier reading.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: Cleanup indentation in bgp_route_refresh_receive

Some code in bgp_route_refresh_receive was spread across several
lines because of an end of line commit.  Move comment to a place
to allow better formating.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

Merge pull request #5870 from donaldsharp/hash_backet

*: change hash_backet to hash_bucket

bgpd: agg_node_lookup not unlocking

SA has found a case where we did a table lookup of a rn( and
associated lock of that node ) where we did not unlock it.

Unlock the node before moving on to the next one.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

nhrpd: Cleanup set but unused variables

There existed some variables set but never used.  Clean this up.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

ospfd: Cleanup set but unused variables

There existed some variables set but never used.  Clean this up.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

zebra: Cleanup set but unused variables

There existed some variables set but never used.  Clean this up.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

ldpd: Cleanup set but unused variables

There existed some variables set but never used.  Clean this up.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

lib: Cleanup set but unused variables

There existed some variables set but never used.  Clean this up.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

bgpd: Cleanup set but unused variables

There existed some variables set but never used.  Clean this up.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

Merge pull request #5864 from kuldeepkash/route_map

tests: Optimizing route-map test cases

*: change hash_backet to hash_bucket

It's been a year search and destroy.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

tests: Modified scripts to optimize execution time for route-map test suite

1. Modified scripts to optimize execution time for route-map test suite
2. Removed unnecessary print statements from /lib/bgp.py

Signed-off-by: Kuldeep Kashyap <kashyapk@vmware.com>

Merge pull request #5866 from qlyoung/fix-ipaddr-isset

lib: fix ipaddr_isset

lib: fix ipaddr_isset

Meaning inverted by mistake

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>

doc: document attribute-unchanged directive

This directive was undocumented, but is the way to configure the
next-hop to be left unchanged for ipv6 setups (in ipv4, a route-map with
`set next-hop unchanged` works fine, but this is unavailable for ipv6) .

Signed-off-by: Raphael Bauduin <rb@raphinou.com>

pbrd: unconfigure table range

Implement the [no] version of `pbr table range`. We had the command
but were doing nothing with it.

This just calls the set_table_range API again using the defaults.

Signed-off-by: Stephen Worley <sworley@cumulusnetworks.com>

Merge pull request #5783 from ton31337/fix/bad_formatting_bgpd_gr

bgpd: Format properly `show bgp neighbors` for graceful restart stuff

Merge pull request #5270 from NaveenThanikachalam/rmap

lib: Optimizing route-maps

Merge pull request #5771 from pguibert6WIND/show_route_table_protocol

vty: add command to get route with table, vrf and protocol

lib: use const in nexthop_group copy api

Add some const to the nhg copy api.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

pimd: fix SA reports

Fix a couple of SA warnings introduced recently; replace some
'strcpy' calls.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

Merge pull request #5838 from volta-networks/fix_isis_ifdown

isisd: do not disable circuit on ifdown

Merge pull request #5844 from qlyoung/fix-zebra-gr-unnecessary-malloc

Fix zebra gr unnecessary malloc

Merge pull request #5847 from mjstapp/fix_pim_sas

pimd: fix SA reports

pimd: fix SA reports

Fix a couple of SA warnings introduced recently; replace some
'strcpy' calls.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

Merge pull request #5842 from qlyoung/fix-test-then-xfree-again

lib: do not test before XFREE

Merge pull request #5845 from TomasKorbar/master

Make frr able to be built by gcc 10

Merge pull request #5768 from donaldsharp/bgp_nexthop

Bgp nexthop

Merge pull request #5831 from donaldsharp/small_test_speedups

Small test speedups

pimd: Make frr able to be built by gcc 10

- Add extern modifier to some declarations in header file and move
qpim_all_pim_routers_addr definition to pimd/pimd.c

`GCC now defaults to -fno-common. As a result, global variable accesses
are more efficient on various targets. In C, global variables with
multiple tentative definitions now result in linker errors.`
Taken from https://gcc.gnu.org/gcc-10/changes.html

Signed-off-by: Tomas Korbar <tkorbar@redhat.com>

Merge pull request #5840 from opensourcerouting/route-map-recurse

lib: don't allow to recurse inside route-map node

zebra: remove null check before XFREE in GR code

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>

zebra: reduce scope of XCALLOC for gr processing

Somewhat gnarly code flow here that might be leaking memory - can't tell
if it's a test artifact or not, but in any case this reduces the
situations in which we need to alloc a block.

And we don't need to check XCALLOC for success...
And we don't need to null check before XFREE...
Or set XFREE'd pointers to NULL...

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>

lib: do not test before XFREE

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>

lib: don't allow to recurse inside route-map node

vtysh should handle going back up one level to try the command, there is
no need to be able to recurse inside route-map.

This also fixes a problem with northbound hitting the XPath queue limit
of 8 nodes.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

tests: Reduce rip[ng]-topo1 run time

Reduce rip topo1 run time from ~100 seconds down to ~45 seconds.

1) Reduce very very long sleeps
2) Modify rip timers to be more aggressive in sending data.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

Merge pull request #5826 from mjstapp/fix_cli_pfx_doc

doc: correct ip prefix token mapping table

tests: bgp_multiview_topo1 was taking too long

The bgp_multiview_topo1 test had two issues

a) An abbundance of sleeps that are not necessary
b) The convergence time was too short.

The upshoot of this is that the test goes from 72 -> 33 seconds
run time on my machine.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

Merge pull request #5788 from karamalla0406/large_comm

bgpd: support 'show ip bgp large-community" on default vrf

Merge pull request #5834 from opensourcerouting/route-map-fix-crash

lib: fix route-map coverity scan issue

Merge pull request #5836 from donaldsharp/bgp_collision

bgpd: Add a better breadcrumb for when bgp is missconfiged

Merge pull request #5837 from qlyoung/fix-zapi-pbr-unsupported-ip-family-log-message-newline

zebra: remove \n in zapi pbr family log msg

isisd: do not disable circuit on ifdown

when zebra detects that an interface is gone, notify the circuit but do
not disable it - the interface is still configured until it isn't.

Without this fix, removing the interface in the kernel and then removing
the circuit from the configuration would cause an assertion in isis_csm.c:78

Signed-off-by: Emanuele Di Pascale <emanuele@voltanet.io>

zebra: remove \n in zapi pbr family log msg

Signed-off-by: Quentin Young <qlyoung@cumulusnetworks.com>

bgpd: Add a better breadcrumb for when bgp is missconfiged

Currently During bgp open collision resolution if both
the router-id's are the same, we correctly follow
the RFC and close the connection.  The problem is of course
that there is no notification of the error in configuration
to the end user other than a subtle open debug message.

Explicitly call out the miss-configuration as an error message
as that this miss-config took several hours of debugging to notice.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

doc: correct ip prefix token mapping table

Fix the table that shows the mapping between ip network cli
tokens and the data struct that the cli handler will see.
Optional network/subnet types appear as a pointer to an
all-zeroes struct, not a NULL pointer.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

Merge pull request #5802 from ton31337/feature/aggregate-address_origin_override

bgpd: Allow overriding ORIGIN for aggregate-address

lib: fix route-map coverity scan issue

Use better TAILQ free idiom to avoid coverity scan warnings. This fixes
the coverity scan issue 1491240 .

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

Merge pull request #5830 from mjstapp/fix_rtadv_stubs

zebra: fix missing rtadv stub functions

Merge pull request #5809 from donaldsharp/vrf_name

Print out vrf name as well as id

Merge pull request #5795 from donaldsharp/lua_dev_doc

doc: Add some basic how-to for lua and clean up some found issues

zebra: fix missing rtadv stub functions

Add in a few missing stub route-advert functions; these are
needed to build frr with v6 route adverts disabled.

Signed-off-by: Mark Stapp <mjs@voltanet.io>

Merge pull request #5823 from donaldsharp/irdp_what

Irdp build and log fixes

Merge pull request #5651 from AnuradhaKaruppiah/evpn-pim-fixes

EVPN-PIM: complete anycast (MLAG) VTEP support

Merge pull request #5813 from mjstapp/zapi_labels_use_nh

*: encode zapi labels message using nexthops

zebra: Fix checksum calculation to not include old checksum

As part of checksum calculation for a received packet we were
comparing the checksum returned from in_cksum.  Typically
when we calculate the checksum the value stored in the checksum
must be all 0's.  Store the received checksum and then set
the checksum to 0 and then compare.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

zebra: Fix zlog_debug -> flog_err for error situations in IRDP

In several places we would send debug messages for failure situations
that really should be errors.

Signed-off-by: Donald Sharpd <sharpd@cumulusnetworks.com>

zebra: SO_BROADCAST needs a uint32_t instead of a uint8_t

Using SO_BROADCAST, in the linux kernel, requires a uint32_t to be passed
in for all SOL_SOCKET calls.  Modify code to use it.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

lib: Optimizing route-maps - Part-4

The commit includes the documentation for the newly introduced
commands to enable/disable the optimization.

Signed-off-by: NaveenThanikachalam <nthanikachal@vmware.com>

lib: Optimizing route-maps - Part-3

* This commit implements the code style suggestions from Polychaeta.
* This commit also introduces a CLI to toggle the optimization and, a hidden
  CLI to display the contents of the constructed prefix tree.

Signed-off-by: NaveenThanikachalam <nthanikachal@vmware.com>

lib: Optimizing route-maps - Part-2

This commit introduces the logic that computes the best-match route-map index
for a given prefix.

Signed-off-by: NaveenThanikachalam <nthanikachal@vmware.com>

lib: Optimizing route-maps - Part-1

* This commit introduces the building blocks.
    A per-route-map prefix tree is introduced.
    This tree will consist of the prefixes defined within the prefix-lists
    that are added to the match clause of that route-map.

Signed-off-by: NaveenThanikachalam <nthanikachal@vmware.com>

Merge pull request #5801 from donaldsharp/bgp_peer_sort

Bgp peer sort

Merge pull request #5790 from chiragshah6/mdev

bgpd: format fields to bgp evpn vni json cmds

Merge pull request #5807 from ton31337/fix/do_not_show_failed_if_shutdown

bgpd: Show the real reason why the peer is failed

Merge pull request #5815 from donaldsharp/feb_sa_stuff

bgpd: Fix up a couple of new SA issues found

Merge pull request #5817 from donaldsharp/more_bgp_rcvd_sent

bgpd: Add a couple more spaces for output on MsgRcvd and MsgSent