Revert "bgpd: add nb support for suppress-map in aggregate cmd"

This reverts commit 90e21f3535ff2d887cbb52f0e4e6fe4907095d70.

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

Revert "bgpd: neighbor peergrp config to transactional cli"

This reverts commit f4b8ec07219cce13c8627efe311247ee06f3a8ab.

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

Revert "*: shorten name of prefix-limit options"

This reverts commit afe03f6094656c9f96150223c1ed5ee8e9a4ba93.

Revert "bgpd: convert max prefix clis to transactional"

This reverts commit 04261decb9901c931f1800f27c459f3a0483776c.

Revert "bgpd: convert neighbor peer config to transactional"

This reverts commit 8611c7f3b6f33cb5ef8687bc126354d06eadbfcf.

Revert "bgpd: shorten names for default originate nb callbacks"

This reverts commit cc4440c3177f312294ec0cec836f41e453b367fb.

Revert "bgpd: default originate transactional callbacks"

This reverts commit 7c54def2926e85d760a7ab42f879a6545630878f.

Revert "bgpd: Actually return the group peer"

This reverts commit e5209541f2f425c25ef016be2cade323b327ea17.

Revert "bgpd: Check if peer is not NULL before calling peer_as_change()"

This reverts commit 7a68370285ed4d64e1ed9dcab60b17bfb26a0ba4.

Revert "bgpd: Respect `-e X` value for multipath"

This reverts commit 76bd07b6881add1fbeafa5dc11c48d9ad6a00f14.

Revert "bgpd: fix compiler warnings in nbr cmd"

This reverts commit 555c8ab7ad7a9f60c8d6cc9dd48705f0ee364797.

Revert "bgpd: convert to yang modeled nbr cmds"

This reverts commit 232c75cd032a0ff8a6540f03a10cb1ae3143af06.

Revert "bgpd: Prevent null deref"

This reverts commit e80bc08944be7cbe4085c03653818139ccd616d1.

Revert "bgpd: convert neighbor plist to transactional cli"

This reverts commit d9986d26c0a48f9cc2afeaa15fe304b094c041e4.

Revert "yang: nbr plist nb conversion for more afi-safis"

This reverts commit f4eac84c5ac136bc7dd37f2c950d375e94616bc4.

Revert "bgpd: convert nbr rmap transactional cli"

This reverts commit c6685575336566680aeb537e8ed5abb164be0268.

Revert "bgpd: fix unsetting of multipath as-set"

This reverts commit 174c5be33c433b9cc586d3aeee178407d282d97f.

Revert "bgpd: fix removal of rd vpn export"

This reverts commit c576dc6a4f5c62aeeb5e77e41c0f8efbf7f5797f.

Revert "bgpd: Temp fix to allow numbered peers to be part of a peer group"

This reverts commit 2a059a5448c37699eabd8802b44427978ec645ac.

Revert "bgpd: Remove hidden `neighbor X route-map Y <in|out>` command"

This reverts commit 4c3e9f072a415aa02f367aaff1cc90dd6fda5d06.

Revert "bgpd: fix 'neighbor X prefix-list Y in' SAFI VPN"

This reverts commit 9fa7a8bf3769914620225826e4ca8ef3f8182106.

Revert "bgpd: don't rely on northbound control plane protocol vrf pointer"

This reverts commit 3fa607bebacd4881053414d641e8376b4e4f6e83.

Revert "bgpd: convert send-community to transactional cli"

This reverts commit e48a9a90e07ccdeae8f6bcdff9c433f5d6ea0b39.

Revert "bgpd: remove duplicated checks from CLI"

This reverts commit 45d1d7c9a2b7b41125e565af566e10d470349122.

Revert "bgpd: don't use operational state in "router bgp" command"

This reverts commit 4062b455a6eacfe3f5107daa68351b18cc0ca7e7.

Revert "bgpd: convert l2vpn evpn nbr route-map to nb"

This reverts commit 8be861e60b7f6069c8a1c2aab75bbbbed3b57829.

Revert "bgpd: prefix-list and route-map nb fixes"

This reverts commit 68a57089122b22414b04ebc28bd0db81051fdcbd.

Revert "bgpd: vrf route leaking, fix the bgp instance delete and re-add"

This reverts commit ef7c53e244d5d0cd7c0ba2536fc931abf28df656.

Revert "bgpd: fix bgp_get_vty return values"

This reverts commit d09328e5991c9f657758921264492825e7081175.

Revert "bgpd: fix crash when as/type mismatches in config"

This reverts commit 0d6d0208a55c786513f472ad690bae1788e173fa.

Revert "bgpd: simplify bgp_global_local_as_modify code"

This reverts commit debb7b1931cdf5b3e06ca244bf09bb009b5a8677.

Revert "bgpd: forbid modification of bgp instance type"

This reverts commit d3e3677096e5cf30116ca63879caf44e25b080ad.

Revert "bgpd: don't return error for GR no-op commands"

This reverts commit 895b0d0ad6e39ad54e82e5c3d9060aa390c8bdd8.

Merge pull request #9264 from ak503/nhrp_fix

nhrp: fix display of nhs command.

Merge pull request #9164 from pguibert6WIND/flowspec_vrflite_shortcut

bgpd: flowspec redirect vrf uses vrf table instead of allocated table id

Merge pull request #9191 from idryzhov/rip-auth

ripd: fix authentication key length

Merge pull request #9231 from idryzhov/zebra-rmap-set-src

zebra: remove checks for src address existence when using "set src"

Merge pull request #9239 from idryzhov/ospf-passive-fix

ospfd: fix "no ip ospf passive" command

Merge pull request #9243 from ton31337/fix/revert_db0e636dc45f9bd2c76528a8368332c56f2c8f1e

bgpd: Revert a couple of BGP dampening related commits

Merge pull request #9249 from ton31337/fix/dont_capability

bgpd: Use strict AS4 capability when processing parsing/generating pkts

Merge pull request #9259 from opensourcerouting/moar-json

*: can't get enough JSON

nhrp: fix display of nhs command.

Before:
ip nhrp nhs dynamicnbma 192.168.1.1

After:
ip nhrp nhs dynamic nbma 192.168.1.1

Signed-off-by: Dmitrii Turlupov <dturlupov@factor-ts.ru>

Merge pull request #9056 from askorichenko/test-dont-capability

bgpd: Clear capabilities field when resetting a bgp neighbor

Merge pull request #9258 from mjstapp/fix_rule_strlcpy

zebra: use strlcpy in dplane_rule_init

Merge pull request #9232 from idryzhov/interface-node-cleanup

*: cleanup interface node installation

lib: add "json" option to "show ip[v6] prefix-list"

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

zebra: add "json" option to "show interface"

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

lib: add "json" option to "show route-map"

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

lib: add "json" option to "show ip[v6] access-list"

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

bgpd: add "json" option to "show bgp as-path-access-list"

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

zebra: use strlcpy in dplane_rule_init

Use strlcpy for safety in dplane rule init api.

Signed-off-by: Mark Stapp <mjs.ietf@gmail.com>

Merge pull request #9255 from ton31337/fix/bgp_max_packet_size_no_capabilities

bgpd: Set extended msg size only if we advertised and received capability

zebra: remove checks for src address existence when using "set src"

1. This check is absolutely useless. Nothing keeps user from deleting
   the address right after this check.
2. This check prevents zebra from correctly reading the user config with
   "set src" because of a race with interface startup (see #4249).
3. NO OPERATIONAL DATA USAGE ON VALIDATION STAGE.

Fixes #7319.

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

Merge pull request #9238 from leonshaw/fix/netns-delete

lib, zebra: Preserve user-configured VRF on netns deletion

Merge pull request #8982 from dlqs/lua-func-stack

Merge pull request #9211 from donaldsharp/revert_logic_changes

lib: prevent crash in make check in some situations

bgpd: Set extended msg size only if we advertised and received capability

If we don't advertise any capabilities (dont-capability-negotiate), we
shouldn't set msg size to 65k only if received this capability from another
peer.

Before:

```
~/frr# vtysh -c 'show ip bgp update-group' | grep 'Max packet size'
    Max packet size: 65535
```

After:

```
~/frr# vtysh -c 'show ip bgp update-group' | grep 'Max packet size'
    Max packet size: 4096
```

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

bgpd: flowspec redirect vrf uses vrf table instead of allocated table id

Until now, when bgp flowspec entry action was to redirect to a vrf, a
default route was installed in a specific table. that route was a vrf
route leak one. The process can be simplified, as vrf-lite already
has a table identifier. Actually, because policy routing is used to
redirect traffic to a defined table (with ip rule command), use
the table identifier of the VRF.

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

tests: Check if BGP connection established if using no capabilities adv

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

bgpd: Use strict AS4 capability when processing parsing/generating pkts

PeerA sets `dont-capability-negotiate` for PeerB. It does not send any
capabilities to PeerB. This leads to situation when PeerA received AS4 cap,
while it doesn't send AS4 to PeerB and tries parsing AS_PATH using 32bits.

[GTTPK-RX2GP][EC 33554436] Malformed AS path from 192.168.0.2, length is 4
[SM0KX-WXMGK] bgp_attr_malformed: attributes: , origin ?
[P7TRR-4J6XT][EC 33554487] 192.168.0.2: Attribute AS_PATH, parse error - treating as withdrawal
[WX70K-6XXVF][EC 33554454] 192.168.0.2 rcvd UPDATE with errors in attr(s)!! Withdrawing route.
[Z2KFR-ZGJC8] 192.168.0.2 rcvd UPDATE w/ attr: , origin ?
[P9VY3-5B6NX] 192.168.0.2 rcvd UPDATE wlen 0 attrlen 80 alen 73
[QMZ79-K2DH7][EC 33554454] 192.168.0.2 [Error] Update packet error (wrong prefix length 64 for afi 1)
[P9SYB-54XRZ][EC 33554454] 192.168.0.2 [Error] Error parsing NLRI
[V1CHF-JSGRR] %NOTIFICATION: sent to neighbor 192.168.0.2 3/10 (UPDATE Message Error/Invalid Network Field) 0 bytes
[HTHRX-GQYGJ][EC 33554454] bgp_process_packet: BGP UPDATE receipt failed for peer: 192.168.0.2

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

Merge pull request #9244 from LabNConsulting/chopps/fix-bgp-list-test

tests: add early interface config for topojson and fix test

Merge pull request #9246 from mjstapp/fix_doc_bgp_colons

doc: fix bgp user doc colons

tests: add early interface config for topojson and fix test

- A more general fix for the bgp listener test which requires interfaces be
configured in the kernel when the bgpd daemons are launched.

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

Merge pull request #9228 from LabNConsulting/chopps/scapy-sendpkt

tests: add scapy_sendpkt.py util, replace arping use with it

tests: add scapy_sendpkt.py util, replace arping use with it

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

doc: fix bgp user doc

Too many colons in the bgp doc source file.

Signed-off-by: Mark Stapp <mjs.ietf@gmail.com>

Merge pull request #9196 from donaldsharp/test_queued

tests: Increase timeout for loaded systems

Revert "bgpd: fix memory leak in dampening"

This reverts commit 322e1085fe52540261864bd73a3ecdb37f7203d3.

Revert "bgpd: fix missing delete from the list in dampening"

This reverts commit 755a977b8aef231bae28994159eef2ffcfd5b599.

Revert "bgpd: use double-linked list instead of single-linked list in dampening"

Tested with full feed, this stucks and bgpd even stops responding.

```
[T58XM-TP956][EC 268435457] bgpd state -> unresponsive : no response yet to ping sent 90 seconds ago
```

This reverts commit db0e636dc45f9bd2c76528a8368332c56f2c8f1e.

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

Merge pull request #9214 from donaldsharp/assert_cleanup

zebra: bugfix of error quit of zebra, due to no nexthop ACTIVE

Merge pull request #9217 from idryzhov/bgp-damp-leaks

bgpd: fix various memory leaks in dampening

ospfd: fix "no ip ospf passive" command

This command is currently always treated as an "unset" command, assuming
that active is the default type of the interface. In reality, the default
type of the interface can be changed using "passive-interface default"
command. Both "no" and regular commands can be "set" commands, depending
on the default value. They are treated as an "unset" when there's already
a config of the opposite type.

All this logic is in ospf_passive_interface_update.

Fixes #9240.

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

Merge pull request #9028 from mobash-rasool/ospfv3-asbr-summarisation

Ospfv3 ASBR summarisation feature

Merge pull request #9237 from donaldsharp/code_block

doc: Fix code-block display for example shell commands

lib, zebra: Preserve user-configured VRF on netns deletion

Don't clear VRF's user-configured flag when netns is deleted.

Signed-off-by: Xiao Liang <shaw.leon@gmail.com>

doc: Fix code-block display for example shell commands

the code-block was jumbled and not displaying properly.

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

bgpd: use double-linked list instead of single-linked list in dampening

This code always used double-linked list before this rework in 8.0 that
introduced all these crashes and memory leaks. Using single-linked list
is actually a performance regression, because there are frequent removes
here and single-linked list obviously handles removes much worse.

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

bgpd: fix missing damp info free when cleaning bgp path

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

bgpd: fix missing delete from the list in dampening

When bgp_damp_info_free is called from bgp_route.c, we were never
deleting the BDI from the list.

Move the deletion inside bgp_damp_info_free to cover all cases.

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

bgpd: cleanup bgp_damp_info_free

bgp_damp_config, afi and safi are never used.

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

bgpd: fix memory leak in dampening

bdi->path is never NULL, therefore the structure was never freed.

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

bgpd: fix incorrect usage of slist in dampening

Current code is a complete misuse of SLIST structure. Instead of just
adding a SLIST_ENTRY to struct bgp_damp_info, it allocates a separate
structure to be a node in the list.

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

Merge pull request #9216 from donaldsharp/simple_snmp_fix

tests: Set addresses before we use snmpd

Merge pull request #9225 from idryzhov/bgp-damp-list-readd

bgpd: fix missing list add in dampening

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

ospf6d: Type-7 LSA originated with wrong sequence number

*: cleanup interface node installation

The only difference in daemons' interface node definition is the config
write function. No need to define the node in every daemon, just pass
the callback as an argument to a library function and define the node
there.

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

Merge pull request #9222 from ton31337/fix/bgp_dampening_clear

bgpd: Drop double-pointer for bgp_damp_info_free()

bgpd: fix missing list add in dampening

One more crash in dampening code...

When bgp_damp_withdraw is called, if there's already a BDI structure,
bgp_damp_info_claim is called to re-assign the bdi->config in case it
was changed. The problem is that bgp_damp_info_claim actually removes
the BDI from the reuse list of the old config and never adds it to the
reuse list of the new config. We must do this to prevent the crash
because all the code assumes that BDI is always in some list.

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

Merge pull request #9212 from idryzhov/pim-crash

pimd: fix missing list remove when deleting mesh group

Merge pull request #9215 from idryzhov/bgp-damp-crash

bgpd: fix double free in dampening code

bgpd: Drop double-pointer for bgp_damp_info_free()

This causes a crash using `clear ip bgp dampening <prefix>`.

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

Merge pull request #9207 from donaldsharp/snmp_pytest_mark

Snmp pytest mark

Merge pull request #9209 from donaldsharp/bgp_ipv6_unicast

bgpd: Mark the node as the correct type for bgp ipv6 unicast

docs: update parens

Signed-off-by: Donald Lee <dlqs@gmx.com>

lib: Add parens around macro args

Signed-off-by: Donald Lee <dlqs@gmx.com>

lib: Fix condition for snprintf

Signed-off-by: Donald Lee <dlqs@gmx.com>

bgpd: fix double free in dampening code

bgp_damp_info_unclaim already calls bgp_reuselist_del. We must not call
it again here.

Fixes #9046.

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

tests: Set addresses before we use snmpd

The test_simple_snmp.py test starts bgp, zebra and snmpd at the
same time.  Then zebra configuration is read in and interface
addresses are applied.  If snmp start slower than zebra
the snmp process can properly get it's ip address to bind to
if it is faster than zebra, it will fail.  Ensure that the
test has addresses before we start daemons.

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

zebra: bugfix of error quit of zebra, due to no nexthop ACTIVE

There exists some rare situations where fpm will attempt
to send a route update with no valid nexthops.  In that
case an assert would be hit.  This is not good for
trying to keep your routing daemons up and running
when we can safely just recover the situation.

Fixes #7588
Signed-off-by: batmancn <batmanustc@gmail.com>
<fixed commit message, and used zlog_err>
Signed-off-by: Donald Sharp <sharpd@nvidia.com>