lib: Set proper write file when using backup config

When we read in a backup file, we should save the original
host.config so that we can put it back to the correct original
location after we read in the backup config.

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

staticd: Fix crash with NULL si

Move the call to the static_install_route to inside
of the loop, since we have changed the behavior of
how we send down routes to zebra a bit.

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

staticd: Allow daemon to have backup static routes

Modify staticd to allow it to have backup static routes
with higher admin distance.

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

lib, zebra: Allow protocols to use Distance as part of RR semantics

Allow protocols to specify to zebra that they would like zebra
to use the distance passed down as part of determine sameness for
Route Replace semantics.

This will be used by the static daemon to allow it to have
backup static routes with greater distances.

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

staticd: Code review comments fixes

1) Conform staticd to proper gnu gpl file format.
2) Fix a spelling mistake found.

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

doc: Add some documentation for staticd

Remove the ip route specific sections from zebra documenation and
create a specific one for the new staticd.

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

staticd: Fix recursive routes for certain types of nexthops

Existing NEXTHOP_TYPE_IPV4_IFINDEX and NEXTHOP_TYPE_IPV6_IFINDEX
routes allow recursion, but were broken when the route happened
to recursively resolve and the resolution nexthop changed.

This commit fixes this issue.  Please note that this issue was
in pre-move of static route handling to it's own daemon as well.
This was some easy low-hanging fruit, so to speak.

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

zebra: Remove zebra_static.c and .h

These are no longer needed so remove.

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

staticd: Enable backup config

When the user is not using the integrated config and
has upgraded to a version of FRR with staticd, that
probably means that static routes are stored in zebra.conf.
Take advantage of the backup backup config option and read
those in.

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

debianpkg, redhat, tools: Fixup startup/build for new daemon

Add code to allow FRR to properly build and handle the staticd
for some of the more common packaging.

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

staticd: Start the addition of a staticd

This is the start of separating out the static
handling code from zebra -> staticd.  This will
help simplify the zebra code and isolate static
route handling to it's own code base.

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

zebra: Allow a static daemon to connect

As part of moving the static route handling to it's own daemon
allow zebra to accept static route types from upper level
protocols.

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

Merge pull request #2696 from sworleys/Netlink-Fuzz

zebra: Add code for fuzzing netlink

Merge pull request #2700 from sworleys/Netlink-Prefix-Len-Check

zebra: Add check for prefix length from kernel messages

Merge pull request #2728 from donaldsharp/table_cleanup

lib: Add parameter names as a hint of what is expected.

Merge pull request #2735 from sworleys/Netlink-Next-Fault

zebra: Check for netlink message next fault

Merge pull request #2748 from chiragshah6/evpn_dev

zebra: display consistant mac count

zebra: display consistant mac count

show evpn mac vni all
show evpn mac vni x
does not display local svi and anycast mac into count.

Ticket:CM-20456
Testing Done:

Before:

TOR1# show evpn mac vni 1008
Number of MACs (local and remote) known for this VNI: 4
MAC               Type   Intf/Remote VTEP      VLAN
44:38:39:00:6b:4c local  vlan1008              1008
00:02:00:00:00:04 local  hostbond5             1008
00:02:00:00:00:02 local  hostbond4             1008
00:00:5e:00:01:01 local  vlan1008-v0           1008
00:02:00:00:00:0c remote 27.0.0.15
00:02:00:00:00:0a remote 27.0.0.15
dell-s6000-07#

After:

TOR1# show evpn mac vni 1008
Number of MACs (local and remote) known for this VNI: 6
MAC               Type   Intf/Remote VTEP      VLAN
44:38:39:00:6b:4c local  vlan1008              1008
00:02:00:00:00:04 local  hostbond5             1008
00:02:00:00:00:02 local  hostbond4             1008
00:00:5e:00:01:01 local  vlan1008-v0           1008
00:02:00:00:00:0c remote 27.0.0.15
00:02:00:00:00:0a remote 27.0.0.15

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

Merge pull request #2719 from pguibert6WIND/fix_tableno_vrf

zebra: fix missing table identifier passed for ip route vrf commands

Merge pull request #2742 from vangheem/master

fix json output when vrf not active

Merge pull request #2746 from chiragshah6/mdev1

bgpd: add keepalive thread name

Merge pull request #2745 from adharkar/frr-filtered_route

bgpd: Show routes filtered by prefix-list in filter-routes command

bgpd: add keepalive thread name

Testing Done:

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

bgpd: Show routes filtered by prefix-list in filter-routes command
Update:Addressed review comments

Changed "show bgp ipv4 neighbor filtered-routes"
to show routes filtered by prefx lists, distribute lists and filter lists

Closes: #2653
Signed-off-by: Ameya Dharkar adharkar@vmware.com

Merge pull request #2743 from donaldsharp/bgp_null_stuff

Put back some removed Code

lib,zebra: fix json output when vrf1 when not active

When I did a show ip route with `json` on a vrf when it didn't exist,
frr would output invalid json.

Signed-off-by: Nathan Van Gheem <nathan@cumulusnetworks.com>

bgpd: Notice when we unlock if we should NULL pointer

The bi->net pointer that is being unlocked had a commit
that removed the `bi->net = NULL;` recently.  This code
was preventing a use after free crash being experienced
in other code paths.  While commit 37e679629f9 was fixing
a different code path crash.

Make the parent->net pointer aware it may be locked/freed
from multiple places and to not NULL the pointer to it
unless we have actually freed the data.

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

lib: Modify route unlock code to return appropriate pointer

Modify the unlock code for a route_node to return NULL on
pointer freed or to return the node itself again.

We'll need to go through the code and fix this pattern,
but this is a problem for another day.  Get this fix in
place and we can make it a low hanging problem to fix.

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

Merge pull request #2708 from ton31337/feature/default-originate_apply_route-maps

bgpd: Set attributes according route-maps if applied for default-originate

Merge pull request #2737 from vishaldhingra/master

bgpd: changes for crash seen in BGP on "no rt vpn" bug Id 2667

zebra: fix missing table identifier passed for ip route vrf commands

The parameter was missing in that vty command. Then it is being added.
Also some documentation is refreshed.

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

Merge pull request #1 from vishaldhingra/bug_fix

bgpd: changes for crash seen in BGP on "no rt vpn" bug Id 2667

bgpd: Set attributes according route-maps if applied for default-originate

Merge pull request #2729 from qlyoung/doc-misc-bgp-fixes

doc: misc bgp.rst fixes

bgpd: Show routes filtered by prefix-list in filter-routes command

Changed "show bgp ipv4 neighbor <peer> filtered-routes"
to show routes filtered by prefx lists, distribute lists and filter lists

Closes: #2653
Signed-off-by: Ameya Dharkar adharkar@vmware.com

zebra: Check for netlink message next fault

NLMSG_NEXT decrements the buffer length (status) by
the header msg length (nlmsg_len) everytime its called.
If nlmsg_len isn't accurate and set to be larger than
what it should represent, it will cause status to
decrement passed 0. This makes NLMSG_NEXT return a
pointer that references an inaccessible address.
When that is passed to NLMSG_OK, it segfaults.

Add a check to verify that there is still something to read
before we try to.

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

Merge branch 'master' into Netlink-Prefix-Len-Check

zebra: Make prefix length check return error

Prefix length validation checks should be returning an error
rather than 0. Switch to that and make them error messages.

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

Style, don't initialize netlink_read flag

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

zebra: Make fuzzer code use default netlink socket

Change the fuzzing code so that it fakes data from
the listening socket rather than using its own pseudo one.

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

Merge pull request #2727 from donaldsharp/pim_jp_lollipops

pimd: RPT Prunes do not need to be figured for a *,G Prune

Merge pull request #2607 from pguibert6WIND/complement_fs_patch3

Complement fs patch3

Merge pull request #2681 from qlyoung/doc-cleanup-rpki

doc: cleanup rpki.rst

Merge pull request #2662 from donaldsharp/tests

Tests

doc: misc bgp.rst fixes

* Clean up command stubs
* Document `disable-connected-check`
* Add markup
* Clarify section on autonomous systems
* Rewrap line to 80 characters
* Fix typos
* Fix grammar

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

lib: Add parameter names as a hint of what is expected.

Add some parameter names to functions in table.h to give a
clue as to what we expect people to pass in.

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

tests: Let compiler know this was an intentionall fallthrough

Let the compiler know that this code intentionally fell
through to the next case statement in a switch.

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

tests: Prevent weird type promotion objection in bgp tests

The va_start function cannot take a object that can be type promoted
Looks like a new warning coming in from a new compiler.

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

tests: Fix const change

Some of the parameters for lib/srcdest.. have changed to
consts, make the test cases respect that.

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

pimd: RPT Prunes do not need to be figured for a *,G Prune

Packet sending in PIM is a two step process.
1) Gather data size of next G to be packed into a packet.
2) Write data

After 1 we need to ensure that the next G to pack will actually
fit in a packet.  If it does not send what we've currently written
and start a new packet to send.

Because this was a 2 step process it is important to be consistent
in what you think you have packed -vs- what you think you should.

PIM has a bug where we were considering S,G RPT Prunes for a *,G
even when the *,G was being pruned.  This lead to a situation where
we were figuring a write size of more data then what we actually wrote
into a packet.  This would leave a 8 byte whole of 0's in the packet
due to the way we moved pointers around.

Fix the code so that we do not attempt to consider S,G rpt prunes
for a *,G prune when figuring out how much we should write in step 1.

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

Merge pull request #2711 from pacovn/isis_test_fixes

tests/isisd: fix warnings

Merge pull request #2712 from pacovn/Security_Uninitialized_memory_access

pimd: uninitialized memory access fix

bgpd: changes for crash seen in BGP on "no rt vpn" bug Id 2667

There is a default BGP VPN and BGP VRF instance in L3VPN configuration.
The routes are imported and exported between BGP VPN and BGP VRF.
Suppose there is one route in BGP VRF and exported to BGP VPN.
In BGP VPN there is bgp_info struct with bgp_info_extra struct has parent pointer pointing to the bgp_info of BGP VRF.
We take the lock for bgp_node and bgp_info of BGP VRF in the context of BGP VPN.
bgp_info has a back pointer to bgp_node via net.

Now when we have done "no rd vpn" in BGP VRF then in bgp_info_extra_free we have to free the parent resources.
In this context only unlocking is required. It should not set the BGP VRF (bgp_info->net) to NULL.

Signed-off-by: vishaldhingra vdhingra@vmware.com

Merge pull request #2704 from sworleys/Netlink-Nexthop-Hang

zebra: Add check for nexthop loop to prevent hanging

Merge pull request #2706 from chiragshah6/mdev

bgpd: Add evpn prefix to debug bgp updates prefix

Merge pull request #2714 from qlyoung/fix-unused-variable-zserv

zebra: remove unused var in zserv.c

pimd: uninitialized memory access fix

Signed-off-by: F. Aragon <paco@voltanet.io>

doc: move rpki reference to correct spot

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

zebra: remove unused var in zserv.c

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

Merge pull request #2699 from sworleys/Netlink-MTU-Check

zebra: Add check for mtu on netlink RTM_NEWLINK

Refactor code for new fuzzing netlink flag

Changed the configure flag used by netlink fuzzing
and refactored code accordingly.

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

tests/isisd: fix warnings

Commit ae9c9aba changed isis_vertex_id_init() and isis_find_vertex()
parameters, so compiler reported warnings in the test (the actual data
passing through is the same because of the union used in the latest changes).
This commit fixes the warnings in the test.

Signed-off-by: F. Aragon <paco@voltanet.io>

Merge pull request #2665 from chiragshah6/evpn_dev

bgpd: support evpn nd ext community

Merge pull request #2707 from opensourcerouting/fix-srcdest-netlink-read

zebra: do not ignore ipv6 srcdest routes

Merge pull request #2640 from pguibert6WIND/doc_ldp

doc: add Ldp user documentation

Remove whitespace change in netlink fuzz code

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

Change log statements to use unsigned format

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

bgpd: Add evpn prefix to debug bgp updates prefix

Adding EVPN prefix of Type 2, 3 and 5 routes to bgp updates
prefix filters.

Ticket:CM-14476
Testing Done:

Configure multiple evpn options under 'debug bgp updates prefix'.
Below is the running-config output.

MAC-IP route with just MAC:
debug bgp updates prefix l2vpn evpn type macip mac
00:02:00:0a:0a:0a
MAC-IP route with MAC and IP:
debug bgp updates prefix l2vpn evpn type macip mac
00:02:00:00:00:0c ip 45.0.1.9
MAC-IP route with just MAC and IPv6:
debug bgp updates prefix l2vpn evpn type
macip mac 00:02:00:00:00:0a ip 2001:fee1:0:1::8
Type-3:
debug bgp updates prefix l2vpn evpn type multicast ip 27.0.1.19
Type-5:
debug bgp updates prefix l2vpn evpn type prefix
ip 2060:1:1:1::/64

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

Merge pull request #2703 from opensourcerouting/ldpd-missing-command

ldpd: add missing command under the ipv6 address-family node

zebra: do not ignore ipv6 srcdest routes

Commit a2ca67d1d2 consolidated IPv4 and IPv6 handling. It also applied
our ignorance for IPv4 srcdest routes onto IPv6.

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

zebra: add information about which port is monitored

Each ipset with port value monitors either src port or dst port.
The information is added to show pbr iptable commmand.

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

bgpd: flowspec 'no local-install any' wrong order in show runni

When configuring an interface, the no local-install any command appears,
and leads to confusions. because the effect of that command differs if
it is executed after local-install <interfaces> or before executing
local-install <interfaces>, the proposal fix here is to suppress that
command from the vty available commands.

PR=59595
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
Acked-by: Alain Ritoux <alain.ritoux@6wind.com>

bgpd: display more than one FS entre per IP

because the IP destination criterium may match several entries, the show
command may return more than one entry.

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

bgpd: add the pbr identifier in the log messages

for clarity sake, the identifier is inserted in the debug line.

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

bgpd: flush remaining entry if bgp_pbr_match is equal

When adding an entry, a check is done in order to flush previously
configured entries. The whole parameters are checked so as to not remove
some entries that have ipset entries equal, but not iptable settings.

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

bgpd: rework jhash algorithm for bgp_pbr_match entries

Instead of relying on jhash_1word for some parameters that are not 32
bit size, the jash(pointer, len) function is used.

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

bgpd: add debug trace when bgp fs pbr remove too

Removal action from flowspec to policy-routing is also traced back.

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

zebra: show pbr iptable per iptable

Add ability to pass a ip table parameter.

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

bgpd: flowspec pbr entries listed on the bgp information entry

Because one flowspec entry can create 1-N bgp pbr entries, the list is
now updated and visible. Also, because the bgp_extra structure is used,
this list is flushed when the bgp_extra structure is deleted.

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

zebra: Add check for nexthop loop to prevent hanging

Bad nexthop messages from netlink were causing zebra
to hang here. Added a check to verify the length
of the nexthop so it doesn't keep trying to read.

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

ldpd: add missing command under the ipv6 address-family node

Bug found by Patrick Velder.

Signed-off-by: Renato Westphal <renato@opensourcerouting.org>

Merge pull request #2698 from sworleys/Netlink-Filter-AFI

zebra: Add address family filters

Merge pull request #2697 from mjstapp/routemap_const

libs, daemons: use const in route-map apply

Merge pull request #2701 from FRRouting/revert-2670-feature/use_med_if_applied_for_default_route

Revert "Use metric for default-originate with route-map if applied"

Revert "Use metric for default-originate with route-map if applied"

Merge pull request #2670 from ton31337/feature/use_med_if_applied_for_default_route

Use metric for default-originate with route-map if applied

Merge pull request #2646 from AnuradhaKaruppiah/evpn-fixes

zebra: install EVPN gateway MAC as static/sticky

Merge pull request #2641 from donaldsharp/pim_igmp_dr

pimd: Do not create upstream state when not DR for igmp request

Remove not needed filters and fix typos

Some more address family filters we can safely ignore
as well as typos in logger. Added AF_MPLS as filterable.

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

zebra: Add check for prefix length from kernel messages

Zebra needed a check that varifies the prefix length
of an address is a valid length when receiving route
changes and interface address changes.

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

Merge pull request #2668 from dslicenc/cm21722-v6-vrf-display

zebra: remove default vrf output for kernel vrf ipv6 blackhole default

Merge pull request #2689 from dslicenc/vrf-exit-reload

tools: handle exit-vrf as end of context block

zebra: Add check for mtu on netlink RTM_NEWLINK

Zebra needed a check for mtu from the message it
received from the kernel before adding the new link.

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

zebra: Add address family filters

The zebra netlink socket was attempting to read netlink
messages with invalid address families in a couple areas.
Added filters and warn messages.

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

Style for Add code for fuzzing netlink

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

zebra: clean up const use in bsd build

Missed a bsd/routing socket change when enforcing use of const
in some kernel-facing api calls.

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

libs, daemons: use const in route-map apply

Use 'const prefix *' in route-map apply apis; led to some
corresponding changes in several daemons.

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

Merge pull request #2652 from LabNConsulting/working/master/confdate_cleanup

Simplify deprecation check

zebra: Add code for fuzzing netlink

This code allows you to fuzz the netlink listening socket
in zebra by --enable-fuzzing and passing the -w [FILE]
option when running zebra.

File collection is stored in /var/run/frr/netlink_*
where each number is just a counter to keep the
files distinct.

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

Merge pull request #2672 from pcarana/pcarana-add-rpki-rh

RedHat spec: Add the rpki module when needed