zebra: Convert to `struct zebra_ile` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_neigh` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_l3nvi` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_vxlan_sg` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_mac` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_vtep` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

zebra: Convert to `struct zebra_evpn` as per our internal standard

We do not use typedef's to talk about structures as per our standard.
Fixing.

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

Merge pull request #9106 from opensourcerouting/clippy-elf-bsd-cond

lib/clippy: fix forgotten conditional

Merge pull request #9340 from stefreak/patch-1

tools: fix frr pathspace folder permissions

Merge pull request #9516 from qlyoung/fix-alpine-docker-images

docker: build libyang2 along with FRR

Merge pull request #9443 from opensourcerouting/topo-multiview-fix

tests: Make bgp_multiview_topo1 predictable

Merge pull request #9469 from ton31337/fix/extcommunity_bandwidth_floating_to_hex

bgpd: Use IEEE-754 Floating Point for storing extcommunity bandwidth

Merge pull request #9439 from ton31337/feature/set_ext_community_to_none

bgpd: Route-map `set extcommunity none`

Merge pull request #9534 from donaldsharp/ospf6_crashes

ospf6d: Prevent crash of show ipv6 ospf data adv-router 0.0.0.0 links…

ospf6d: Prevent crash of show ipv6 ospf data adv-router 0.0.0.0 linkstate-id 0.0.0.0

With this sequence of events:
eva# conf
eva(config)# router ospf6
eva(config-ospf6)# end
eva# show ipv6 ospf data adv-router 0.0.0.0 linkstate-id 0.0.0.0
OSPF6: Received signal 11 at 1630442431 (si_addr 0x0, PC 0x559dcfa3a656); aborting...
OSPF6: zlog_signal+0x18c                  7fd2cc8229f7     7fff606775d0 /lib/libfrr.so.0 (mapped at 0x7fd2cc770000)
OSPF6: core_handler+0xe3                  7fd2cc8616ad     7fff606776f0 /lib/libfrr.so.0 (mapped at 0x7fd2cc770000)
OSPF6: funlockfile+0x50                   7fd2cc74f140     7fff60677840 /lib/x86_64-linux-gnu/libpthread.so.0 (mapped at 0x7fd2cc73b000)
OSPF6:     ---- signal ----
OSPF6: ospf6_lsdb_type_show_wrapper+0x5d     559dcfa3a656     7fff60677dd0 /usr/lib/frr/ospf6d (mapped at 0x559dcf9a5000)
OSPF6: show_ipv6_ospf6_database_adv_router_linkstate_id+0x1f9     559dcfa3c24a     7fff60677e50 /usr/lib/frr/ospf6d (mapped at 0x559dcf9a5000)

OSPF6 crashes.  Fix.

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

Merge pull request #9531 from LabNConsulting/chopps/bgp-fast-reconnect

tests: add global BGP json config for [re]connect timer

tests: add global BGP json config for [re]connect timer

- Fix the syntax.

Signed-off-by: Christian Hopps <chopps@labn.net>

Merge pull request #9318 from Prerana-GB/ibgp_knob

bgp: BGP knob for faster convergence of bgp sessions

Merge pull request #9530 from ton31337/fix/doc_connect_timers_bgp

doc: Add missing `timers` for connect timer in BGP

doc: Add missing `timers` for connect timer in BGP

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

Merge pull request #9397 from tkms1122/master

bgpd: minimum-holdtime knob to prevent session establishment with BGP peer with low holdtime.

Merge pull request #9526 from LabNConsulting/chopps/bgp-test-defaults

tests: log bgp nbr changes, and allow config of connect timer

Merge pull request #9525 from idryzhov/ospfd-crash

ospfd: correctly cleanup spf data

Merge pull request #9522 from mobash-rasool/ospfv3-bug-fixes

ospfd: add dead-interval 40 if configured in show running

tests: log bgp nbr changes, and allow config of connect timer

- Allow tests to also change the connect timer as they can for the
keep-alive and holddown timers.

Signed-off-by: Christian Hopps <chopps@labn.net>

ospfd: correctly cleanup spf data

ospf_spf_cleanup frees the data so we need to reset the stale pointers.

Fixes #9523.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

Merge pull request #9518 from ton31337/fix/redundand_interface_name_for_peer

bgpd: Drop redundand `WORD` for no_neighbor_interface_cmd

tests: Test if IEEE floating-point encoding for bw works with older format

Just check if backward-compatibility works fine between uint32 / IEEE encodings.

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

ospfd: add dead-interval 40 if configured in show running

Problem Statement:
==================
When hello-interval is configured as 5, automatically dead interval becomes
4 times of hello i.e 20 seconds. But user wants the dead interval as
40 seconds and hello as 5 seconds. Therefore user configures it.
Now "ip ospf dead-interval 40" is not shown in "show running-config"
Therefore when user restarts the daemon, the dead interval goes back to
20 seconds and the neighbors are down.

Fix:
==================
If user configures dead-interval as 40, show it in show running config.

Fixes: #9401
Signed-off-by: Mobashshera Rasool <mrasool@vmware.com>

Merge pull request #9502 from idryzhov/cmd-vtysh-cleanup

lib, vtysh: make node-processing code more generic

alpine: fix path for daemons file install

The install statement was putting the daemons file at "/etc/frr" instead
of making a new directory "/etc/frr" and putting it inside there.

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

docker: add expected log functions to alpine docker-start

Some basic functions are required by frrcommon.sh

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

bgpd: Drop redundand `WORD` for no_neighbor_interface_cmd

Just to be consistent with neighbor_interface_cmd.

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

docker: build libyang2 along with FRR

Alpine images have been broken for some time because libyang2 is not
available in Alpine. This patch updates our Dockerfile to build a
libyang2 APK and install it into the image to satisfy FRR's libyang2
dependency.

Unfortunately, libyang2 erroneously includes an internal header from
glibc, making it dependent on glibc to build. FRR's official Docker
images are based on Alpine, which only offers musl libc. Until libyang2
fixes this problem, the libyang2 source that is installed in this image
is a patched version that is compatible with musl libc and not an
official version.

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

Merge pull request #8964 from LabNConsulting/chopps/config-faster

tests: configure/reset routers in parallel

bgpd: Add `neighbor PEER link-bw-encoding-ieee`

This is to avoid breaking changes between existing deployments of
extended community for bandwidth encoding. By default FRR uses uint32
to encode bandwidth, which is not as the draft requires (IEEE floating-point).

This switch enables the required encoding per-peer.

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

doc: Add `neighbor PEER link-bw-encoding-ieee`

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

Merge pull request #9484 from anlancs/fix-doc-bgp-desc

doc: add a missing field for bgp summary's display

Merge pull request #9511 from LabNConsulting/chopps/ospf-asbr-test-fix

tests: fix ospf ASBR summary test

doc: add a missing field for bgp summary's display

Signed-off-by: anlan_cs <anlan_cs@tom.com>

bgpd: Use IEEE-754 Floating Point for storing extcommunity bandwidth

https://datatracker.ietf.org/doc/html/draft-ietf-idr-link-bandwidth-07 says:

The bandwidth of the link is expressed as 4
   octets in IEEE floating point format, units being bytes (not bits!)
   per second.  It is carried in the Local Administrator subfield of the
   Value Field.

Before:

```
  Extended Community (16), length: 8, Flags [OT]:
    unknown extd community typecode (0x0004), Flags [none]
      0x0000:  0004 fdeb 0001 e848
    0x0000:  0004 fdeb 0001 e848
  Updated routes:
    172.16.16.1/32
```

0001 e848 - means 125000 (1Mbps), which is encoded incorrect.

After:

```
  Extended Community (16), length: 8, Flags [OT]:
    unknown extd community typecode (0x0004), Flags [none]
      0x0000:  0004 fdeb 47f4 2400
    0x0000:  0004 fdeb 47f4 2400
  Updated routes:
    172.16.16.1/32
```

47f4 2400 - means the same, but in floating point format.

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

Merge pull request #9503 from opensourcerouting/ospf6d-redistribute-metrics

ospf6d: extend the "redistribute" command with more options

lib/clippy: fix forgotten conditional

This fixes some spurious warnings on *BSD, where `elffile_add_dynreloc`
isn't used since `elf_getdata_rawchunk` is not available.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>

Merge pull request #9509 from LabNConsulting/chopps/dirlocals

tests: Update dir-locals

Merge pull request #9499 from gsol10/bogus_lsp

isisd: Fix sending of LSP with null seqno

Merge pull request #9488 from pguibert6WIND/fix_nhrp_neigh_state

Fix nhrp neigh state

Merge pull request #9504 from donaldsharp/cbit_is_needed

lib: Send c-bit when not using bfdd as daemon of choice

Merge pull request #9473 from ton31337/fix/BGP_STR_unified

bgpd: Use BGP_STR for CLI definitions everywhere for BGP CLI

tests: fix ospf ASBR summary test

Previously the check did nothing as the input_dict parameter was wrong.

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: Update dir-locals

- format with black in emacs
- wrap text at 88 (black)

Signed-off-by: Christian Hopps <chopps@labn.net>

lib: Send c-bit when not using bfdd as daemon of choice

Commit: 4b983eef2cb5c7306a8303f002d0e053ebeabdca

Modified the zapi send receive of the c-bit to only
be under the HAVE_BFDD.  If you are using ptm-bfd
then the decoder function still expects this to be
sent down.  This commit puts this behavior back

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

Merge pull request #9480 from SaiGomathiN/pim-coverity

pimd: Fixing coverity issues

Merge pull request #9496 from idryzhov/vrf-cmd-init-unused-arg

lib: remove unused argument from vrf_cmd_init

tests: configure routers in parallel

Signed-off-by: Christian Hopps <chopps@labn.net>

tests: improve bgp test determinism by resetting configs

Signed-off-by: Christian Hopps <chopps@labn.net>

vtysh: make node walkup code more generic

No need to have special processing for every single node. Just always
use "exit" the necessary number of times - it works for all nodes.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

Merge pull request #9500 from tomaxuser/master

doc: Basic spellchecking of bgp.rst

ospf6d: extend the "redistribute" command with more options

Add the "metric" and "metric-type" options to the "redistribute"
command.

This is a small commit since the logic of setting the metric
value and type of external routes was already present due to the
implementation of the "default-information originate" command months
ago. This commit merely extends the "redistribute" command to
leverage that functionality.

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

lib: make node_parent more generic

Instead of adding a separate case clause for every node, just find the
node structure in the global list and get its parent node from there.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

Merge pull request #9331 from idryzhov/explicit-exit

*: explicitly print "exit" at the end of every node config

Merge pull request #9492 from Jafaral/pim-cov

pimd: fix coverity/dead-code issue

Merge pull request #9498 from idryzhov/fix-bgp-update-source

bgpd: fix update-source for ipv6

doc: Basic spellchecking of bgp.rst

Signed-off-by: Tomáš Szaniszlo <tomaxuser@gmail.com>

bgpd: fix update-source for ipv6

There's no IPv6 LL address on loopback/vrf interfaces. So if the user
configures update-source, the session is never going to be established.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

isisd: Fix sending of LSP with null seqno

Check sequence number when building LSP missing in received CSNP

Signed-off-by: Guillaume Solignac <gsoligna@protonmail.com>

lib: remove unused argument from vrf_cmd_init

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

lib, zebra: complete the ndm flags on zclient api

Insist on the fact that zclient neighbor state flags are
mapped over netlink state flags. List all the defines
currently known on kernel, and create a netlink API to
convert netlink values to zclient values. The function is
simplified as it is a 1-1 match.

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

nhrp, zebra, lib: pass exact received neighbor state value to nhrp

As NHRP expects some notification of neighboring entries on GRE
interface, when a new interface notification is encountered, the
exact neighbor state flag is found. Previously, the flag passed
to the upper layer was forced to NDM_STATE which is REACHABLE,
as can be seen on below trace:

2021/08/25 10:58:39 NHRP: [QQ0NK-1H449] Netlink: new-neigh 102.1.1.1 dev gre1 lladdr 10.125.0.2 nud 0x2 cache used 1 type 5

When passing the real value, NHRP received an other value like STALE.

2021/08/25 11:28:44 NHRP: [QQ0NK-1H449] Netlink: new-neigh 102.1.1.1 dev gre1 lladdr 10.125.0.2 nud 0x4 cache used 0 type 5

This flag is important for NHRP, as it permits to monitor the link
layer of NHRP entries.

Fixes: d603c0774eba ("nhrp, zebra, lib: enforce usage of zapi_neigh_ip structure")
Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>

Merge pull request #9466 from idryzhov/vrf-netns

lib, zebra: move vrf netns commands from lib to zebra

Merge pull request #9467 from gpnaveen/bgp_v4overv6

tests: adding 11 bgp v4 over v6 test cases.

Merge pull request #9490 from pguibert6WIND/fix_ipv6_seg6

bgpd: fix uninitialised segs6 buffer

pimd: fix coverity/dead-code issue

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

bgpd: fix uninitialised segs6 buffer

Below dump may be observed when receiving bgp ipv6 updates.

2021/08/25 16:52:32 BGP: [V15FP-4CPVK] Tx route add VRF 0 4004::1/128 metric 0 tag 0 count 1 nhg 0
2021/08/25 16:52:32 BGP: [JQXM8-V0CKB]   nhop [1]: 2003::4 if 0 VRF 0 wt 0  P8�o�

Fix it.

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

Merge pull request #9487 from ton31337/fix/no_need_for_break_after_frr_help_exit

*: Drop `break` after using frr_help_exit() in switch/case

*: Drop `break` after using frr_help_exit() in switch/case

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

Merge pull request #9474 from idryzhov/lib-bgp-dependency

lib: remove the dependency on bgpd code

Merge pull request #9435 from SaiGomathiN/sai-igmp

pimd: IGMP groups are not getting timeout

Merge pull request #9414 from mobash-rasool/pim-upst-4

pimd: IGMPv2 query timer gets delayed in a scenario

Merge pull request #9449 from opensourcerouting/ospf6d-nssa-fixes

ospf6d: NSSA fixes

Merge pull request #9457 from anlancs/ospf6-void

ospf6d: suppress coverity warning of return value

pimd: Fixing coverity issues

Problem
======
In pim_msg_send_frame api, the while loop was executed only once.

Fix
===
while is changed to if, as in the code flow
the while part is getting executed only once.

Signed-off-by: Sai Gomathi <nsaigomathi@vmware.com>

pimd: Fixing coverity issues

Signed-off-by: Sai Gomathi <nsaigomathi@vmware.com>

Merge pull request #9458 from anlancs/fix-ospf6-null

ospf6d: fix lock leak of using ALL_LSDB in GR helper

ospf6d: further validate NSSA ASBR routes used for route computation

Add missing sanity check specified in the NSSA RFC.

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

tests: enable more debugs in the ospf6_topo2 topotest

Enable more debugs to facilitate troubleshooting whenever a problem
happens.

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

ospf6d: do not allow an area to be stub and NSSA at the same time

That should not be allowed under any circumstance.

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

ospf6d: fix logging of border router routes

The prefix of routes to border routers consists of two pieces of
information embedded in a single struct (prefix.u.lp):

  struct prefix {
          uint8_t family;
          uint16_t prefixlen;
          union {
   [snip]
                  struct {
                          struct in_addr id;
                          struct in_addr adv_router;
                  } lp;
          } u __attribute__((aligned(8)));
  };

As such, using prefix2str() (or the %pFX format specifier) isn't
correct when logging such routes.

This commit adds a few special cases here and there to handle
OSPF6_DEST_TYPE_ROUTER routes differently. It'd probably be a good
idea to add a helper function to handle all cases in a single place,
but that can be left for a second moment.

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

Merge pull request #7876 from gpnaveen/ospf_gr_helper_using_scapy

tests: ospf gr helper topotests with scapy.

lib: remove the dependency on bgpd code

The library code should not depend on a specific daemon's code.

Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>

bgpd: Use BGP_STR for CLI definitions everywhere for BGP CLI

Just have some cleanup.

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

Merge pull request #9310 from mobash-rasool/ospfv2-bug-fixes-02

ospfd: NSSA area should not exchange Type-4 LSAs in DD

Merge pull request #8729 from gpnaveen/ospf_route_maps

tests: ospf routemap basic test cases 2.

doc: Updating the BGP document with BGP fast convergence CLI

Signed-off-by: Prerana G.B <prerana@vmware.com>

topotests: Topotest changes for new bgp fast convergence knob

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

bgpd: BGP knob to teardown session immediately when peer is unreachable

When BGP is notified by RIB that peer address is unreachable then BGP session must be brought
down immediately and not wait for the hold-timer expiry. Today single-hop EBGP already behaves
this way but need to change for iBGP and multi-hop EBGP sessions.

Signed-off-by: Prerana G.B <prerana@vmware.com>, Pushpasis Sarkar <spushpasis@vmware.com>

Merge pull request #9460 from LabNConsulting/chopps/fix-ospf-config

tests: fix ospf json config confusion

Merge pull request #9462 from idryzhov/snapcraft-proto

snapcraft: add missing dependency

pimd: IGMP groups are not getting timeout

Problem :
=======
When all the groups from Ixia are stopped,
groups still keep refreshing and not getting timeout

RCA:
====
IGMP Report is coming in include mode without any source address, this problem will come.

Fix :
===
If the requested filter mode is INCLUDE *and* the requested
source list is empty, then the entry corresponding to the
requested interface and multicast address is deleted if present.
If no such entry is present, the request is ignored.

When an interface receives the IGMP report without any source, then the group is deleted.

Signed-off-by: Sai Gomathi <nsaigomathi@vmware.com>

tests:  ospf gr helper topotests with scapy.

Testing ospf gr helper using scapy tool.

Signed-off-by: nguggarigoud <nguggarigoud@vmware.com>

Merge pull request #9464 from idryzhov/ospfv3-debug-doc

doc: merge duplicated ospf6d debugging sections