lib: Cleanup recent commit warn->werror issues in lib/routemap.c

The get_route_map_delete_event function should return a value
even if we never get to that part of the function.  Make sure
we know why we are here so it can be fixed appropriately in
the future.

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

Merge pull request #4417 from sworleys/Move-Multicast-Mode

zebra: Move multicast mode to being a property of the router

Merge pull request #4436 from donaldsharp/tools_frr_flush

tools: On shutdown no need to flush from tools/frr.in script

No log commands (#3581)

No log commands

Merge pull request #4345 from NaveenThanikachalam/route_map_dep

lib: Changes made to dependencies of a route-map do not take effect dynamically.

Merge pull request #4078 from lkrishnamoor/rmap_vni_filter

bgpd: Filtering received EVPN routes based on VNI does not work

Merge pull request #4274 from rgirada/fix_clear_mroute

pimd: Re-deisgn the "clear ip mroute" command.

Merge pull request #4445 from pguibert6WIND/display_show_nht_if_not_found

zebra: the route nexthop interfaces per vrf were not displayed ok

zebra: the route nexthop interfaces per vrf were not displayed ok

the interface search done was not looking in the appropriate zns. The
display was then wrong. Update the show command with the correct zns.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

Merge pull request #4442 from opensourcerouting/libtool-doc

doc: add libtool note

doc: add libtool note

(and document that ASAN/MSAN/TSAN are mutually exclusive while at it)

Signed-off-by: David Lamparter <equinox@diac24.net>

tools: On shutdown no need to flush from tools/frr.in script

Zebra already flushes routes on proper shutdown if you are not
using the -K option.  If you are using the -K option then you
do not want the tools/frr script to flush routes.

If zebra crashes and we restart then load up will either delete
the routes or leave them depending on the -K option.

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

Merge pull request #4435 from donaldsharp/zclient_buffer_sizing

lib, zebra: Ensure route encoding has enough space

Merge pull request #4430 from lkrishnamoor/hostname_crash

lib: crash when FRR hostname length > 80 chars

lib: crash when FRR hostname length > 80 chars

Although the RFC states hostname length should be < 255 chars,
FRR allows infinite length technically. However, when you try
to set a hostname > 80 chars, you would immediately notice a crash.

RCA: Crash due to buffer overflow. Large buffer sprintf'd into smaller
buffer. Usage of sprintf function instead of snprintf which is safer.

Signed-off-by: Lakshman Krishnamoorthy <lkrishnamoor@vmware.com>

bgpd: Filtering received EVPN routes based on VNI does not work

Issue1: When "neighbor X.X.X.X route-map RM-VNI-FILTER in" is configured under evpn address-family,
all the received routes are dropped regardless of whether the route has a matching vni or not.

Issue2: Routes with 2 labels are not filtered correctly

Issue3: Interpreting the label based on tunnel type, vxlan was not done correctly.
Vxlan label has 24 bits, whereas, MPLS label is 20 bits long

Fix1: The handler bgp_update() that services the received route ignored the route's label while deciding whether to filter it or not. As part of the fix, the handler now uses the label info to make the decision about whether to filter the route or not.

Fix2: route_match_vni() now tries to match both the labels within the route, not just the one.

Signed-off-by: Lakshman Krishnamoorthy <lkrishnamoor@vmware.com>

lib: Changes made to dependencies of a r-map do not take effect.

Say, more than one sequence of a route-map uses the same named entity
in its match clause. After that entity is removed from any one of the
route-map sequences, any further changes made to that entity doesn't
dynamically take effect.
A reference counter, that allows the named entity to keep a count of
the route-maps dependent on it,  has been introduced to address this issue.

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

lib, zebra: Ensure route encoding has enough space

When you have compiled FRR with a large multipath number
then encoding large ecmp routes between zebra and the
routing daemons.  There exists a theoritical size
of multipath that will cause the encoding to be larger
than the ZEBRA_MAX_PACKET_SIZ.  In the cases where
we have allocated streams that will encode routes
then let's ensure that whatever size we have will
auto-fit what we say we can send.

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

Merge pull request #4434 from donaldsharp/fix_warnings

Fix warnings

doc: Add doc for new configure option

Add documentation for the '--command-log-always' daemon cli
and how to use it.

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

lib: Add '--command-log-always` to all daemons startup

Add 'no log commands' cli and at the same time add a
--command-log-always to the daemon startup cli.

If --command-log-always is specified then all commands are
auto-logged and the 'no log commands' form of the command
is now ignored.

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

lib: Fix gcc 9 warning -> error issue

gcc is complaing about this with --enable-dev and --enable-werror:

In function 'nb_log_callback',
    inlined from 'nb_transaction_apply_finish' at lib/northbound.c:1106:4:
lib/northbound.c:777:2: error: '%s' directive argument is null [-Werror=format-overflow=]
  777 |  zlog_debug(
      |  ^~~~~~~~~~~
  778 |   "northbound callback: event [%s] op [%s] xpath [%s] value [%s]",
      |   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  779 |   nb_event_name(event), nb_operation_name(operation), xpath,
      |   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  780 |   value);
      |   ~~~~~~
  CC       lib/ringbuf.lo

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

lib: vtypath_default could be overwritten

The vtypath_default variable had a possibility of being overwritten
due to size constraints.  This fixes this issue.

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

Merge pull request #4315 from lkrishnamoor/route_map_3rd_state

lib: Introducing a 3rd state for route-map match cmd: RMAP_NOOP

Merge pull request #4369 from patrasar/lmqc_lmqt

pimd: new cli to configure last-member-query-count & last-member-quer…

Merge pull request #4400 from LabNConsulting/working/master/centos7-doc

fix centos7 build - set SPHINXBUILD to match centos7 yum installed package

Merge pull request #4426 from mjstapp/fix_memstats_cli

lib, vtysh: make 'debug memstats' work via vtysh

Merge pull request #4427 from mjstapp/fix_topo_memleak

doc: clean up memleak section of topotests doc

lib, vtysh: make debug memstats work in vtysh

Allow vtysh to send the 'debug memstats-on-exit' command
to the running daemons.

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

lib: Introducing a 3rd state for route-map match cmd: RMAP_NOOP

Introducing a 3rd state for route_map_apply library function: RMAP_NOOP

Traditionally route map MATCH rule apis  were designed to return
a binary response, consisting of either RMAP_MATCH or RMAP_NOMATCH.
(Route-map SET rule apis return RMAP_OKAY or RMAP_ERROR).
Depending on this response, the following statemachine decided the
course of action:

Action: Apply route-map match and return the result (RMAP_MATCH/RMAP_NOMATCH)
State1: Receveived RMAP_MATCH
THEN: If Routemap type is PERMIT, execute other rules if applicable,
otherwise we PERMIT!
Else: If Routemap type is DENY, we DENYMATCH right away

State2: Received RMAP_NOMATCH, continue on to next route-map, otherwise,
return DENYMATCH by default if nothing matched.

With reference to PR 4078 (https://github.com/FRRouting/frr/pull/4078),
we require a 3rd state because of the following situation:

The issue - what if, the rule api needs to abort or ignore a rule?:
"match evpn vni xx" route-map filter can be applied to incoming routes
regardless of whether the tunnel type is vxlan or mpls.
This rule should be N/A for mpls based evpn route, but applicable to only
vxlan based evpn route.

Today, the filter produces either a match or nomatch response regardless of
whether it is mpls/vxlan, resulting in either permitting or denying the
route.. So an mpls evpn route may get filtered out incorrectly.
Eg: "route-map RM1 permit 10 ; match evpn vni 20" or
"route-map RM2 deny 20 ; match vni 20"

With the introduction of the 3rd state, we can abort this rule check safely.
How? The rules api can now return RMAP_NOOP (or another enum) to indicate
that it encountered an invalid check, and needs to abort just that rule,
but continue with other rules.

Question: Do we repurpose an existing enum RMAP_OKAY or RMAP_ERROR
as the 3rd state (or create a new enum like RMAP_NOOP)?
RMAP_OKAY and RMAP_ERROR are used to return the result of set cmd.

We chose to go with RMAP_NOOP (but open to ideas),
as a way to bypass the rmap filter

As a result we have a 3rd state:
State3: Received RMAP_NOOP
Then, proceed to other route-map, otherwise return RMAP_PERMITMATCH by default.

Signed-off-by:Lakshman Krishnamoorthy <lkrishnamoor@vmware.com>

Merge pull request #4413 from donaldsharp/bgp_distance_comes_closer

Bgp distance comes closer

Merge pull request #4410 from donaldsharp/invalid

doc: Fix doc showing command that does not exist

pimd: new cli to configure last-member-query-count & last-member-query-interval

Introduce new cli commands ip igmp last-member-query-count <1-7>
ip igmp last-member-query-interval <1-255> deciseconds.

Display the config in show running config and show ip igmp interface

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

Merge pull request #4408 from donaldsharp/bgp_status_fsm

bgpd: Update an fsm debug message

Merge pull request #4393 from donaldsharp/debug_all

Debug all

Merge pull request #4267 from qlyoung/fix-misc-compile-warnings

Fix misc compile warnings, remove strcpy & strcat

Merge pull request #4402 from chiragshah6/evpn_dev1

bgpd: fix debug to have proper nhop display

Merge pull request #4415 from mjstapp/dplane_dist_setters

zebra: add setters for distance, instance

Merge pull request #4351 from patrasar/pim_show_ip_mroute_summary

pimd: new cli command show ip mroute summary

zebra: Move multicast mode to being a property of the router

The multicast mode enum was a global static in zebra_rib.c
it does not belong there, it belongs in zebra_router, moving.

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

lib: fix helpstring truncation

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

bgpd: use XCALLOC to allocate string buf

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

lib: use static storage for vty_cwd

Why are we allocating this

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

bgpd: fix pointer bug introduced in strlcat change

Inconsistent use of a string pointer led to improperly terminated
strings (terminated too soon)

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

bgpd: suppress dead store warning

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

ospfd: initialize maybe-uninitialized bool

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

lib, zebra: remove uses of strncpy

This removes the last removable uses of strncpy in FRR.

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

zebra: strcat -> strlcat

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

staticd: strcpy -> strlcpy

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

vtysh: strcpy -> strlcpy, strcat -> strlcat

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

pbrd: strcpy -> strlcpy

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

pimd: strcpy -> strlcpy

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

ripngd: strcat -> strlcat

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

lib: remove some strcpy, strcat

Replace with strlcpy, strlcat

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

bgpd: remove strcpy, strcat

Replace with strlcpy, strlcat

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

bgpd: fix rfapi false compiler warning

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

bgpd: fix false compiler warning

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

lib: fix false compiler warning

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

lib: fix uninitialized variable in backtrace output

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

zebra: fix maybe-uninitialized pointer

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

zebra: fix unused variable on OmniOS

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

zebra: add setters for distance, instance

Add a few more 'setters' for dplane context objects.

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

doc: clean up memleak section of topotests doc

Cleaned up some repetition (cut-and-paste?) in the topotests
developer doc.

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

pimd: new cli command show ip mroute summary

Introduced a new command "show ip mroute summary"
to display total number of (*, G) and (S, G) mroutes
created and number of mroutes installed in the kernel.

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

Merge pull request #4414 from opensourcerouting/feature/fix-isis-warnings

isisd: Fix some minor warnings

Merge pull request #4326 from sworleys/Move-NH-Active-Functions

zebra: Move nexthop_active_XXX functions to zebra_nhg.c

Merge pull request #4404 from qlyoung/const-linklist

lib: const a couple linklist apis

Merge pull request #4228 from mjstapp/dplane_notif

zebra: async notifications from the dataplane

zebra: BGP always sends distance no need to double check

BGP always sends down the correct distance to use.  We do
not need rib_add_multipath to double check the code.

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

bgpd: Always send down correct admin distance

In all cases that we are sending routes down to zebra send the
correct admin distance.

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

isisd: Remove unnecessary use of strcpy

`strcpy` is a dangerous function and should not be used. In this
particular place, there is no need for copying strings at all, so let's
just stick to referencing static strings.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

isisd: Remove warning about shadowing a variable

There is no need to redefine `struct isis_lsp *lsp` inside of the
if condition. Let's just remove it.

Signed-off-by: Christian Franke <chris@opensourcerouting.org>

Merge pull request #4401 from manuhalo/fix_isisd_remove_conf

isisd: del routes when area is unconfigured

isisd: del routes when area is unconfigured

attempt to fix #4399

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

pimd: Re-deisgn the "clear ip mroute" command.

Made changes to clean up the all upstreams and ifchannels
in FRR apart from cleanup datapath mroutes when this command
issued.

Signed-off-by: Rajesh Girada <rgirada@vmware.com>

Merge pull request #4407 from opensourcerouting/snap-fix-master

Snap fix master

snapcraft: Add libdb5.3 dependency

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

snapcraft: Disable libyang CACHE for snap package (Temp workaround)

There seems to be a bug in latest libyang running when running in
a snap container which causes a failure of the cache logic. Disable
CACHE for now as it's not yet needed. Will be re-enabled in a later
release

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

snapcraft: Add FabricD to Snap package

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

snapcraft: Update bgpd to use newer rpki lib

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

snapcraft: Add libyang to snap package

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

snapcraft: Add missing staticd.conf.default

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

snapcraft: Don't stage libc6

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

snapcraft: Update RTRLIB to 0.6.3

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

doc: Fix doc showing command that does not exist

The `bgp bestpath prefix-validate disallow-invalid` command referenced in
the rpki documentation does not actually exist, remove the erroneous docs

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

bgpd: Update an fsm debug message

When debugging I was having a hard time correlating some data and noticed that
a particular debug was not being very useful.

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

zebra: Move nexthop_active_XXX functions to zebra_nhg.c

Since these functions are not really rib processing problems
let's move them to zebra_nhg.c which is meant for processing of
nexthop groups.

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

lib: Separate out the debug_init api

Separate out the debug_init api to have 2 functions:

1) Function to register a callback
2) Function to initiate the cli.

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

lib: Make the debug callbacks a list of callbacks

Allow multiple callers to initialize themselves to receive
callbacks for debug on/off operations.

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

bgpd: fix debug to have proper nhop display

Display nexthop based on route type.

Ticket:CM-25129
Testing Done:

evpn route:
*  [2]:[0]:[0]:[48]:[aa:aa:aa:aa:01:1a]:[32]:[45.0.2.111]
    36.0.0.25              0 64000 5560 i

old:
BGP: Tx route add VRF 46 45.0.2.111/32 metric 0 tag 0 flags 0x1409 nhnum 1
BGP:   nhop [1]: 2400:1d:: if 50 VRF 46

New:
BGP: import evpn prefix [2]:[aa:aa:aa:aa:01:1a]:[45.0.2.111]/224 as
ip prefix 45.0.2.111/32 in vrf vrf1

BGP: bgp_zebra_announce: p=45.0.2.111/32, bgp_is_valid_label: 2
BGP: Tx route add VRF 46 45.0.2.111/32 metric 0 tag 0 flags 0x1409 nhnum 1
BGP:   nhop [1]: 36.0.0.25 if 50 VRF 46

Signed-off-by: Chirag Shah <chirag@cumulusnetworks.com>

lib: const a couple linklist apis

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

Merge pull request #4391 from ton31337/feature/delete_prefix_list_by_sequence_number

plist: Delete prefix-list by sequence number

zebra: generate updates from notifications

If an async notification changes a route that's current,
generate an update to keep the kernel in sync.

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

Merge pull request #4384 from dslicenc/bgp-no-delete-default

bgpd: disable deleting default instance if vrf instances exist

Merge pull request #4371 from manuhalo/fix_isisd_large_mtu

isisd: support circuits with mtu > 8192

Merge pull request #4239 from sarav511/rp

pimd: PIM Bootstrap Message Processing

Merge pull request #4322 from sworleys/Nexthop-Cmp

lib: Add nexthop_cmp

doc/developer: set SPHINXBUILD to match centos7 yum installed package

Signed-off-by: Lou Berger <lberger@labn.net>

zebra: mpls lsp async notifications

Add LSP notification event type; add a handler for LSP notifs;
dispatch to that handler.

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

zebra: add dplane context lsp setters

Add some setters for dataplane context LSP data to support
LSP async notifications.

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