Merge pull request #5623 from qlyoung/fix-zebra-rtadv-interval-overflow

zebra: disallow negative rtadv intvl, fix overflow

Merge pull request #5625 from qlyoung/fix-zapi-ipset-name-nullterm

zebra: ensure ipset name is null terminated

Merge pull request #5614 from qlyoung/fix-zebra-zapi-crashes

Fix zebra zapi crashes

Merge pull request #5580 from mjstapp/zebra_nhg_debug_category

zebra: add a debug category for nexthops

Merge pull request #5550 from manuhalo/fix_isis_def_level

isisd, yang: set default is-type to level-1-2

Merge pull request #5546 from mjstapp/fix_notify_nht

zebra: route changes via notify path should trigger nht

Merge pull request #5553 from slankdev/slankdev-fix-kernel-route-deletion-on-vrf

zebra: fix kernel-route's deletion on vrf

Merge pull request #5621 from qlyoung/fix-zclient-excess-nexthop-decode

lib: dont decode more nexthops than we can handle

Merge pull request #5626 from qlyoung/fix-zapi-pbr-foo

zebra: Fix zapi pbr foo, pseudowire ifname termination

Merge pull request #5622 from qlyoung/fix-iface-ifindex-compare

lib: fix ifindex comparison overflow

lib: dont decode more nexthops than we can handle

If someone provides us more nexthops than our configured multipath
setting, drop the rest of them

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

Merge pull request #5607 from ton31337/fix/deprecate_bgpTimerUp

bgpd: Time to deprecate bgpTimerUp

Merge pull request #5606 from ton31337/fix/docker_centos8_image_name

docker: Use proper container name when creating CentOS 8 container

Merge pull request #5632 from ton31337/fix/dont_use_pipes

packaging: Redirect stderr to /dev/null in %post action (frr.spec.in)

Merge pull request #5545 from ton31337/feature/show_bgp_json_regexp

bgpd: Print json output for show_ip_bgp_regexp_cmd

zebra: route changes via notify path trigger nht and mpls

Changes to a route via the dataplane notify path should
trigger nht and mpls lsp processing.

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

Merge pull request #5618 from qlyoung/fix-san-flags-additive

configure.ac: make msan & ubsan flags additive

Merge pull request #5619 from qlyoung/fix-zebra-netlink-undefined-bitshift

zebra: fix undefined bitshifts in netlink stuff

Merge pull request #5630 from slankdev/slankdev-bgpd-fix-large-rd

bgpd: fix large route-distinguisher's format

bgpd: fix large route-distinguisher's format

This commit is about #5629 's issue.
Before this commit, bgpd creates format string of
bgp-route-distinguisher as int32, but correctly format
is uint32. current bgpd's sh-run-cli generate int32 rd,
so if user sets the rd as 1:4294967295(0x1:0xffffffff),
sh-run cli generates 1: -1 as running-config. This
commit fix that issue.

Signed-off-by: Hiroki Shirokura <slank.dev@gmail.com>

Merge pull request #5467 from pogojotz/alpine-linux-build

alpine: fix build on current 'edge' version of Alpine

Merge pull request #5601 from donaldsharp/pim_rb

Pim rb

packaging: Redirect stderr to /dev/null in %post action (frr.spec.in)

Solves:
```
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
/usr/bin/sed: couldn't write 40 items to stdout: Broken pipe
```

This happens because `grep -q` returns immediately after first match
and closes the pipe while sed has more output to write.

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

zebra: fix undefined bitshifts in netlink stuff

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

zebra: null terminate interface name from wire

We read an ifname from the wire but don't make sure its null terminated,
fix it

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

zebra: fix iptable memleak, fix free funcs

- Fix iptable freeing code to free malloc'd list
- malloc iptable in zapi handler and use those functions to free it when
  done to fix a linked list memleak

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

zebra: ensure ipset name is null terminated

We copy a fixed length buffer from the wire but don't ensure it is null
terminated. Then print it as a c-string. Lul.

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

zebra: disallow negative rtadv intvl, fix overflow

- Disallow RA interval < 0
- Fix integer overflow issue converting interval to seconds from
  milliseconds
- Add missing "m" to "ms"

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

lib: fix ifindex comparison overflow

Very small (negative!) ifindexes, when subtracted, can overflow.

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

bgpd: Time to deprecate bgpTimerUp

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

docker: Use proper container name when creating CentOS 8 container

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

Merge pull request #5611 from qlyoung/fix-bgp-no-listen-docstring

bgpd: fix too much no listen doc string

configure.ac: make msan & ubsan flags additive

These were setting = SAN_FLAGS, we want to add to the variable. Note
MSAN can't be used with any other sanitizer (except UBSan) but the
compiler will complain about that if you use it wrong so we don't need
to enforce it here.

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

Merge pull request #5604 from qlyoung/add-ubsan-option

configure.ac: add --enable-undefined-sanitizer

Merge pull request #5615 from ton31337/feature/supported_rfc_list_add_8212

doc: Add rfc8212 to supported RFCs list

zebra: check pbr rule msg for correct afi

further down we hash the src & dst ip, which asserts that the afi is one
of the well known ones, given the field names i assume the correct afis
here are af_inet[6]

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

pimd: Do not warn for common occurrence in igmp code

Do not warn when we receive a multicast address that
matches 224.0.0.0/24.

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

pimd: Convert the upstream_list and hash to a rb tree

Convert the upstream_list and hash to a rb tree, Significant
time was being spent in the listnode_add_sort.  This reduces
this time greatly.

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

pimd: Convert the channel_oil_list|hash to a rb_tree

The channel_oil_list and hash are taking significant
cpu at scale when adding to the sorted list.  Replace
with a RB_TREE.

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

doc: Add rfc8212 to supported RFCs list

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

zebra: use correct attr size for netlink enc

a bool is not 4 bytes

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

Merge pull request #5613 from srimohans/ietf

doc: Updating FRR supported RFC list

doc: Updating FRR supported RFC list

1. Added subsections for BGP, MPLS, ISIS, OSPF etc.
2. Updated RFCs for BGP, MPLS, OSPF.

Signed-off-by: Sri Mohana Singamsetty <msingamsetty@vmware.com>

bgpd: fix too much no listen doc string

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

configure.ac: add --enable-undefined-sanitizer

For UBSan

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

Merge pull request #5594 from ton31337/feature/rhel8_docker

docker: Create build for CentOS 8

Merge pull request #5593 from ton31337/feature/rhel8_rpm

packaging: Add CentOS 8 support

Merge pull request #5602 from donaldsharp/ripng_redistribute

Rip redistribute fixes

ripd: Fix redistribute/no redistribute rinse repeate commands

ripd was using zclient_redistribute for installation but not
for removal.  As such the lib/zclient.c was not properly tracking
add/removal.  I think it would be best to just let rip to track
this instead of zclient.

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

ripngd: Fix redistribute/no redistribute rinse repeat commands

ripngd was using zclient_redistribute for installation but not
for removal.  As such the lib/zclient.c was not properly tracking
add/removal.  I think it would be best to just let ripng to track
this instead of zclient.

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

Merge pull request #5589 from ton31337/fix/send_fsm_error_notification_where_needed

bgpd: Send notification to the peer on FSM error

Merge pull request #5595 from ton31337/fix/bgp_listen_definition

bgpd: Change description for `[no] bgp listen limit` commands

Merge pull request #5598 from zoltan/bgp_cfg_typo

bgpd: fix console typo

Merge pull request #5597 from chiragshah6/evpn_dev2

bgpd: skip ra for blackhole nexthop type

bgpd: fix console typo

The console printout had a typo for the number of configured peers label.

Signed-off-by: Zoltan Arnold Nagy <zoltan.arnold.nagy@gmail.com>

bgpd: Send notification to the peer on FSM error

We should send a NOTIFICATION message with the Error Code Finite State
Machine Error if we receive NOTIFICATION in OpenSent state
as defined in https://tools.ietf.org/html/rfc4271#section-8.2.2

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

bgpd: skip ra for blackhole nexthop type

bgp nexthop cache update triggers RA for global ipv6
nexthop update.
In case of blackhole route type the outgoing interface
information is NULL which leads to bgpd crash.

Skip sending RA for blackhole nexthop type.

Ticket:CM-27299
Reviewed By:
Testing Done:

Configure bgp neighbor over global ipv6 address.
Configure static blackhole route with prefix includes
connected ipv6 global address.
Upon link flap, zebra sends nexthop update to bgp.
Bgp nexthop cache skips sending RA for blackhole nexthop type.

router bgp 65002
 bgp router-id 91.189.93.190
 ...
 neighbor 2001:67c:1360::b peer-group internal

static route:
ipv6 route 2001:67c:1360::/48 Null0 254

iface rowlink.4010
        address 91.189.93.190/32
        address 2001:67c:1360::a/128

Trigger ifdown rowlink.4010; ifup rowlink.4010

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

Merge pull request #5596 from donaldsharp/md5_doc

doc: Add bgp neighbor password command

doc: Add bgp neighbor password command

The bgp neighbor password command was not documented additionally
the fact that you may need to instruct the kernel to have more
memory available for tcp sockets when using this feature on a large
number of peers.

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

bgpd: Change description for `[no] bgp listen limit` commands

It's quite confusing when you see this:
```
exit1-debian-9(config-router)# bgp listen
  listen  Configure BGP defaults
```

And:
```
exit1-debian-9(config-router)# no bgp listen
  listen  unset maximum number of BGP Dynamic Neighbors that can be created
```

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

docker: Create build for CentOS 8

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

packaging: Fix bogus date in frr.spec.in

```
RPM build errors:
    bogus date in %changelog: Sun May 28 2018 Rafael Zalamena <rzalamena@opensourcerouting.org> - 7.3_dev_MyRPMVersion
```

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

doc: Use proper numbering for `packaging-redhat.rst`

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

doc: Add a section in `Building FRR` for CentOS 8

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

packaging: Adopt frr.spec.in to CentOS 8

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

Merge pull request #5350 from sworleys/Fix-NS-Defualt-Info

zebra: Fix default ns->info should point to default zebra_ns

zebra: add a debug category for nexthops

Add a category for nexthops and nh-groups, since we've got
quite a bit of code in there now.

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

zebra: fix kernel-route's deletion on vrf

zebra can catch the kernel's route deletion by netlink.
but current FRR can't delete kernel-route on vrf(l3mdev)
when kernel operator delete the route on out-side of FRR.
It looks problem about kernel-route deletion.

This problem is caused around _nexthop_cmp_no_labels(nh1,nh2)
that checks the each nexthop's member 'vrf_id'.
And _nexthop_cmp_no_labels's caller doesn't set the vrf_id
of nexthop structure. This commit fix that case.

Signed-off-by: Hiroki Shirokura <slank.dev@gmail.com>

Merge pull request #5579 from donaldsharp/pim_ip_mroute_last

vtysh: Ensure interface `ip mroute ...` commands are last

Merge pull request #5584 from qlyoung/pim-fix-iph-trust

pimd: readd iph length checks

Merge pull request #5583 from donaldsharp/multicast_nh

bgpd: Allow ipv4 multicast to use v4 nexthops

Merge pull request #5581 from Jafaral/pimroute-rf

pimd: refactor ip mroute cmd

Merge pull request #5585 from qlyoung/fix-pimd-mtrace-missing-length-check

pimd: fix missing igmp mtrace length check

pimd: readd iph length checks

Kernel might not hand us a bad packet, but better safe than sorry here.
Validate the IP header length field. Also adds an additional check that
the packet length is sufficient for an IGMP packet, and a check that we
actually have enough for an ip header at all.

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

Merge pull request #5268 from qlyoung/vrrp-vrf

VRRP VRF support

pimd: fix missing igmp mtrace length check

We check that the IGMP message is sufficently sized for an mtrace query,
but not a response, leading to uninitialized stack read.

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

bgpd: Allow ipv4 multicast to use v4 nexthops

When passing a v4 multicast route to a peer send
the v4 nexthop as a preferred methodology.

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

pimd: refactor ip mroute cmd

combine:
  ip mroute INTERFACE A.B.C.D
  ip mroute INTERFACE A.B.C.D A.B.C.D

into:
  ip mroute INTERFACE A.B.C.D [A.B.C.D]

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

Merge pull request #5567 from opensourcerouting/bring-them-back

lib: fix build of the northbound plugins

Merge pull request #5563 from mjstapp/fix_nhg_depends_find

zebra: improve efficiency of depends_find()

vtysh: Ensure interface `ip mroute ...` commands are last

Ensure when displaying interface based ip mroute commands that they
are last.

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

Merge pull request #5574 from ton31337/fix/no_bgp_listen_range_peer-group

bgpd: Make sure we can use `no bgp listen range ...`

bgpd: Make sure we can use `no bgp listen range ...`

Fixes:
```
exit1-debian-9(config-router)# no bgp listen range 192.168.10.0/24 peer-group TEST
% Peer-group does not exist
exit1-debian-9(config-router)#
```
Closes https://github.com/FRRouting/frr/issues/5570

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

lib: fix build of the northbound plugins

Commit 1b3e9a21dd4 removed the global visibility of the yang_modules
variable, breaking the build of all northbound plugins. Revert a
small part of that commit to fix this issue.

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

Merge pull request #5566 from ton31337/fix/ebgp-requires-policy_with_ibgp

 bgpd: Do not apply eBGP policy for iBGP peers

Merge pull request #5543 from Jafaral/routerid

zebra: add 'show router-id'

tests: Test if eBGP policy is not applied to iBGP peers

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

bgpd: Do not apply eBGP policy for iBGP peers

Treat iBGP peers as they have a policy applied.

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

Merge pull request #5549 from donaldsharp/automated

doc: Update workflow to include new automated testing Requirement

Merge pull request #5561 from qlyoung/doc-update-overview

Doc update overview

Merge pull request #5564 from taspelund/end-of-rib_notification

bgpd: Remove misleading 'NOTIFICATION' string from End-of-RIB log

vrrpd: support namespace vrf too

-.-

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

bgpd: Remove misleading 'NOTIFICATION' string from End-of-RIB log

'NOTIFICATION' string in this message incorrectly implies a BGP
Notification message was the cause of this log. Removing it to
reduce confusion and replacing with function name.

Signed-off-by: Trey Aspelund <taspelund@cumulusnetworks.com>

lib: document vrf_socket(), vrf_bind()

These two don't really do what you might expect, document them

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

zebra: improve efficiency of depends_find()

Do less malloc and free in depends_find(), when looking for
a singleton nexthop in the nhg hash.

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

doc: fix duplicate -S option in bgpd manpage

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

doc: update overview.rst for the new millennium

Provide a more succinct summary of what FRR is, what it does, where it
sits in the network stack. Also remove some outdated comments about
features.

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

Merge pull request #5559 from mjstapp/fix_test_typelist

tests: fix endian bug in test_typelist

Merge pull request #5556 from donaldsharp/ospf_use_after

ospfd: Prevent use after free on shutdown

tests: fix endian bug in test_typelist

Fix a byte-swapping bug that appeared on big-endian arch but
wasn't visible on little-endian runs.

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