Merge pull request #3362 from pacovn/Coverity_1475469_null_check

bgpd: null check (Coverity 1475469)

Merge pull request #3364 from opensourcerouting/bugfix/isis-ipv6-only

IS-IS: Bugfix ipv6 only

Merge pull request #3374 from opensourcerouting/bugfix/vty-shadow

lib/vty: Fix warning about shadowed variable

Merge pull request #3371 from donaldsharp/vtysh_ospf_instance

vtysh: Don't attempt to reconnect the non-instanced ospf process

Merge pull request #3325 from adeg/bugfix/bgpd-vtysh-fixes

bgp_vty: Fix vpn_leak_postchange_all() every time "router bgp ASNUM" context is entered

Merge pull request #3176 from chiragshah6/evpn_dev

zebra: duplicate address detection and dampening

lib/vty: Fix warning about shadowed variable

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

vtysh: Don't attempt to reconnect the non-instanced ospf process

When running ospf instances we should not attempt to reconnect
the default ospf instance on running a command.

This commit should be targeted enough because in the case
of normal operation we connect to everything we should
and only set the VTYSH_WAS_ACTIVE flag for those we
truly have lost connection too.

Before:

donna.cumulusnetworks.com# config t
donna.cumulusnetworks.com(config)# router ospf 100
Warning: connecting to ospfd...failed!
donna.cumulusnetworks.com(config-router)#

After:
donna.cumulusnetworks.com# conf t
donna.cumulusnetworks.com(config)# router ospf 100
donna.cumulusnetworks.com(config-router)# end
donna.cumulusnetworks.com#

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

Merge pull request #3367 from karamalla0406/frr3333

bgpd: BGP daemon crashed when a L2VNI is unconfigured

Merge pull request #3363 from pacovn/static_analysis__ISO_C_return_compliance

bgpd isisd lib: fix return on void functions

Merge pull request #3368 from pacovn/static_analysis__ISO_C_empty_initializer

isisd lib ospfd pbrd python: fix empty init

isisd lib ospfd pbrd python: fix empty init

ISO C forbids empty initializer braces. Empty initializers have been
replaced with {0}

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

bgpd: BGP daemon crashed when a L2VNI is unconfigured

When a VNI is unconfigured it deletes all of its import and export
route-targets.  There is a export route-target link list and import
route-target linked list.  There are redudant loops in  the
route-target deletion code.  In the first iteration it deleted the
route-target and freed the RT node, but not list node.
In the 2nd iteration it tries to free the RT node again, resulting in
 the double free of RT node.

Signed-off-by: "Kishore Aramalla karamallavmware.com"

Merge pull request #3339 from opensourcerouting/bugfix/isis-flooding-issues

isis: Fix flooding issues

isisd: Fix bug in tlvs_protocols_supported_to_adj

We should set nlpids[0] if we only set on protocol,
not nlpids[1].

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

isisd: Consider non-mt links usable when either v4 or v6 works

When we run in non-mt mode, we should consider links which have either
working IPv4 or IPv6 active and look at the neighbors nlpids to judge
wether a link is usable.

Fixes: #3336

Merge pull request #3359 from qlyoung/true-atomics

Restrict atomics to 32-bits only

bgpd isisd lib: fix return on void functions

ISO C forbids ‘return’ with expression, in function returning void.

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

Merge pull request #2929 from kssoman/rtr_fix

bgpd, zebra: Creating Loopback Interface Flaps BGPd, it should update

bgpd: null check (Coverity 1475469)

Null check of 'rn' returned by bgp_node_lookup() because it could be
deferenced afterwards into bgp_nexthop_get_node_info()

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

tools: add non-32 bit atomic warning to checkpatch

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

*: only use 32-bit atomics

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

Merge pull request #3358 from opensourcerouting/libtool-cfg-warn

build: refuse non-working linking options

Merge pull request #3346 from donaldsharp/pim_possible_issues

Pim possible issues

Merge pull request #3294 from pguibert6WIND/distribute_list_ipv6

lib: distribute-list ipv6 can be (un)configured

Merge pull request #3311 from dslicenc/static-int-up

staticd: install static routes in a vrf when next-hop interface comes up

Merge pull request #2725 from pguibert6WIND/workflow_lts

doc: change workflow to support long term maintenance branches

Merge pull request #3343 from qlyoung/doc-add-vtysh-dev

doc: add dev doc for vtysh

build: refuse non-working linking options

We only support:
* --enable-shared --disable-static --disable-static-bin
* --enable-shared --enable-static --disable-static-bin
* --enable-shared --enable-static --enable-static-bin

(The second option is not particularly useful.)

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

bgpd: Creating Loopback Interface Flaps BGPd (#2865)

* The function bgp_router_id_zebra_bump() will check for active bgp
  peers before chenging the router ID.
  If there are established peers, router ID is not modified
  which prevents the flapping of established peer connection

* Added field in bgp structure to store the count of established peers

Signed-off-by: kssoman <somanks@vmware.com>

zebra: dup addr detect mac consolidation

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

zebra: dup addr detect neigh consolidation

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

bgp, zebra: address review comments

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

zebra: dup addr detect add fields to show evpn cmd

TOR# show evpn
L2 VNIs: 5
L3 VNIs: 0
Advertise gateway mac-ip: No
Duplicate address detection: Enable
  Detection max-moves 5, time 180

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

zebra: dup addr detect fix warnings

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

zebra: dup addr detect clear command

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

zebra: dup addr detect operation commands

Display duplicate detected list of MACs and VNIs.
- Per VNI
- Across all VNIs

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

lib, zebra: dup addr detect display detection fields

Display following Per MAC and Neigh's output:
If duplicate address detection is under process,
display detection start time and detection count.
If duplicate address detection detected an address
as duplicate, display detection time and duplicate
status.

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

zebra: dup addr detect freeze action

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

zebra: dup addr detect warn-only

Duplicate address detection warning only action
upon an address detected as duplicate.

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

zebra: dup addr detect zapi changes

EVPN Duplicate Address Detection (DAD)
zapi information

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

bgpd: dup addr detect config cli

Duplicate address detection configuration clis
under bgp l2vpn evpn config mode.
- Enabled/Disable (global knob) for feature.
- Configure cli for duplicate detection action
freeze and freze until time (auto-recovery).

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

bgpd: dup addr detect data struct for cfg

Enable/disable duplicate address detection
there are 3 actions
warning-only: Default action which generates
only frr warning (syslog) to user for any
duplicate detecton
freeze: Permanently freezes address, manual
intervene required.
freeze with time: An address will recover once
the time has expired (auto-recovery).

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

Merge pull request #3341 from donaldsharp/unneeded

zebra: Remove uncompiled file

Merge pull request #3352 from pacovn/Coverity_1475489_resource_leak

bgpd: fix resource leak (Coverity 1475489)

Merge pull request #3353 from qlyoung/fix-bgpd-community-list-deletion-nit

bgpd: fix small error in community-list patch

Merge pull request #3338 from ton31337/fix/optional_args_for_exclude

bgpd: Optionally remove AS number when using `no set as-path exclude`

Merge pull request #3344 from ton31337/fix/optional_args_for_community-lists

bgpd: Remove community-list by name without typing full rule

bgpd: fix resource leak (Coverity 1475489)

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

Merge pull request #3348 from donaldsharp/bsd_route_install

zebra: Fix bsd privs elevation

zebra: Fix privs elevation

This Commit: f183e380fae61b7c1f89fed6e32ed5a9d1ede8a8 broke
priviledge escalation on *bsd.  This puts it back.

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

pimd: When we fail to initiate ifchannel backout work done

When we receive a igmp report and attempt to initiate
a pim ifchannel for it and that fails to work then
let's back out the work done setting stuff up to this
point.

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

pimd: Add some debugs for when local membership fails

When we fail to add a local membership add some additional debugs
so that we can have a bit more information on when something goes
bad.

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

bgpd: Remove community-list by name without typing full rule

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

bgpd: Optionally remove AS number when using `no set as-path exclude`

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

zebra: Remove uncompiled file

The zebra_l2_null.c is not compiled and not needed, remove.

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

isisd: Fix behavior on reception of self-originated LSP

We should only update and reflood our own LSPs when the received LSP
is newer than the local copy.

In all other cases, we should simply acknowledge it or resend our own
LSP.

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

fabricd: Fix flooding bug

Due to `lsp` getting shadowed, we would send each T0 its own LSP
whenever we actually wanted to flood a different LSP.

Fix this and set -Wshadow=local in my build environment. m(

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

isisd: Always consider purges as newer

When receiving an LSP with same sequence number but different
checksum as in the local database, we would always treat it as
newer than the local LSP.

That behavior is incorrect if the local LSP is indeed a purged
LSP waiting for age-out and the received one is not.

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

Merge pull request #3326 from qlyoung/fix-lla-reinstallation

zebra: force neighbor entry reinstallation

Merge pull request #3316 from qlyoung/remove-pr-admonishment

.github: remove style reminder from PR template

Merge pull request #3323 from qlyoung/doc-zebra-protocol

doc: update zebra protocol documentation

Merge pull request #3329 from rubenk/autoconf-cleanup

configure.ac: consistently quote all m4 macros

Merge pull request #3331 from mjstapp/fix_lib_id_warning

libs: rename two id_alloc macros to resolve bsd conflict

Merge pull request #3332 from qlyoung/fix-code-block-err

doc: use correct specifier for code block

Merge pull request #3164 from pguibert6WIND/bgpstandalone_importexport

bgpd: allow vrf validity and bgp vrf import/export, when zebra is off

libs: rename two id_alloc macros to resolve bsd conflict

Two of the macros in lib/id_alloc had conflicts on some platforms;
rename them to be unique.

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

configure.ac: consistently quote all m4 macros

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

Merge pull request #3322 from qlyoung/remove-vestigial-secondary

Remove vestigial secondary

Merge pull request #3324 from qlyoung/fix-zebra-router-memleak

zebra: fix zebra router memleaks

bgpd: add missing endline

Missing endline was resulting in garbled output in vtysh in some cases, for example, when there were no peers configured and the user has issued "bgp disable-ebgp-connected-route-check" command.

Signed-off-by: Anton Degtyarev <anton@cumulusnetworks.com>

bgpd: Fix bgpd doing vpn_leak_postchange_all() every time "router bgp ASNUM" command is entered in vtysh

In rare cases when the default BGP instance is instantiated after VRF bgp instances (see comment to bgp_mplsvpn.c:vpn_leak_postchange_all() for an example), the "router bgp" command needs to call vpn_leak_postchange_all() to start the route leaking process. The issue was it was never checked if the "router bgp" command was used to create the default BGP instance or just to enter into "router bgp" command context. This resulted in vpn_leak_postchange_all() executed every time (and vpn routes re-announced to all peers) when the user was entering "router bgp" command context.

Signed-off-by: Anton Degtyarev <anton@cumulusnetworks.com>

Merge pull request #3320 from mjstapp/fix_dp_ecmp

zebra: Fix netlink installation of recursive ecmp routes

doc: change workflow to support long term maintenance branches

Add in the workflow the possibility to do long term support.

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

Merge pull request #3295 from pguibert6WIND/ospf_area_unconfigure

ospfd: permit reconfiguring network after area suppressed

zebra: Fix netlink installation of recursive ecmp routes

Recursive multipath nexthops were broken by the initial async
dataplane - we were trying to install an extra, invalid
nexthop.

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

bgpd: allow vrf validity and bgp vrf import/export, when zebra is off

if zebra is not started, then vrf identifiers are not available. This
prevents import/exportation to be available. This commit permits having
import/export available, even when zebra is not started.

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

Merge pull request #3310 from adeg/bugfix/bgpd-mplsvpn-route-import-check

bgpd: fix bgp path info for mplsvpn leaked routes

Merge pull request #3051 from mitch-skiba/addpath_change_V1

Addpath - Reuse IDs

Merge pull request #3318 from qlyoung/agentx-remove-warning-msg

lib: remove agentx already enabled warning

Merge pull request #3317 from qlyoung/gitignore-libtool-orig

frr: ignore libtool.orig

Merge pull request #3287 from donaldsharp/v6_access_list

zebra: Add `match ipv6 address WORD` as a legal option

Merge pull request #3286 from donaldsharp/late_registration

bgpd: Late registration of Extended Nexthop should allow RA's to happen

Merge pull request #3313 from qlyoung/doc-strip-whitespace

doc: strip trailing whitespace

Merge pull request #3314 from qlyoung/doc-ipv6-isis-router-cmd

doc: add missing docs for ipv6 isis router cmd

Merge pull request #3315 from qlyoung/doc-dev-fixes

Doc dev fixes

Merge pull request #3307 from donaldsharp/bondo

zebra: Let zebra know about bond and blond slave intf types

staticd: remove stale header info in static_fixup_intf_nh

Signed-off-by: Don Slice <dslice@cumulusnetworks.com>

Merge pull request #3312 from pguibert6WIND/ospf6_missing_vty

ospf6d: add missing vty commands to ospf6 area command

Merge pull request #3303 from opensourcerouting/bugfix/isis-fragment-retransmission

isisd: Fix issues with purged fragments

zebra: Let zebra know about bond and blond slave intf types

The interface type can be a bond or a bond slave, add some
code to note this and to display it as part of a show interface
command.

Signed-off-by: Dinesh Dutt <didutt@gmail.com>
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>

ospf6d: add missing vty commands to ospf6 area command

it was not possible to configure per area identifier under decimal
format some ospf6 area options. this is the case of filter list, or
export-list.

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

Merge pull request #3289 from donaldsharp/onlink_schmonlink

zebra: Carry onlink if set from resolving nexthop

Merge pull request #3276 from donaldsharp/zclient_update

*: Replace zclient_new with zclient_new_notify

staticd: install static routes in a vrf when next-hop interface comes up

Problem reported with cross-vrf static routes that the routes weren't
installed when the target interface is bounced.  Determined that we did
not initiate re-install of the statics in that particular case, so added
it. Test case previously failing now passes.

Signed-off-by: Don Slice <dslice@cumulusnetworks.com>

Merge pull request #3285 from srimohans/watchfrr

watchfrr: replace /usr/sbin/service frr with /usr/lib/frr/frr script

Merge pull request #3281 from opensourcerouting/update-libyang-instructions

doc: update libyang build instructions

Merge pull request #3299 from opensourcerouting/fix-bfd-ipv6-unnumbered

bfdd: fix BGP unnumbered peer setup

Merge pull request #3305 from opensourcerouting/bugfix/isis-lsp-aggregate-time

isisd: Adjust duration until lsp is regenerated

Merge pull request #3304 from opensourcerouting/bugfix/openfabric-flooding

fabricd: never flood back through the incoming interface