Merge pull request #13517 from Keelan10/ospf_ti_lfa-memory-leak

ospfd: Fix  ospf_ti_lfa memory leak

ospf_ti_lfa: Free p_space memory allocation

Free p_space

Signed-off-by: Keelan Cannoo <keelan.cannoo@icloud.com>

Merge pull request #13430 from opensourcerouting/feature/rip_allow-ecmp_limit

ripd: Implement allow-ecmp X command

Merge pull request #13490 from Jafaral/fix-ospf-asbr-test

tests: improve log messages, clear neighbors to avoid bad state

tests: reset neighbor state machine to avoid bad/stale state

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

Merge pull request #13507 from opensourcerouting/feature/bgp_route-map_match_source-protocol

bgpd: Implement match source-protocol for route-maps

tests: Check if `match source-protocol` route-map cmd works for BGP

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Implement `match source-protocol` for route-maps

The main idea is to filter routes by matching source (originating) protocol
for outgoing direction. For instance, filter outgoing routes to an arbitrary
router that are static only. Or filter out only routes learned from RIP.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

tests: log the router id in ospf tests, improve log in a couple of places

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

Merge pull request #13489 from LabNConsulting/ospf-opaque-stale-fix

ospfd: OSPF opaque LSA stale processing fix and topotests.

Merge pull request #13495 from LabNConsulting/chopps/munet-0.13.3

tests: update to munet 0.13.3, restoring GNU screen functionality

doc: Use `match source-protocol` for route-maps in BGP as well

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #13485 from donaldsharp/ospf_loopback_cost

ospfd: Respect loopback's cost that is set and set loopback costs to 0

Merge pull request #13479 from ryndia/fix_leak

[WIP] ospfd: opsf_abr.c memory leak fix, free unused range

ospfd: Respect loopback's cost that is set and set loopback costs to 0

When setting an loopback's cost, set the value to 0, unless the operator
has assigned a value for the loopback's cost.

RFC states:

If the state of the interface is Loopback, add a Type 3
link (stub network) as long as this is not an interface
to an unnumbered point-to-point network. The Link ID
should be set to the IP interface address, the Link Data
set to the mask 0xffffffff (indicating a host route),
and the cost set to 0.

FRR is going to allow this to be overridden if the operator specifically
sets a value too.

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

Merge pull request #13486 from opensourcerouting/fix/doc_aigp_neighbor

doc: Add `neighbor aigp` command for BGP

Merge pull request #13493 from anlancs/fix/bgpd-cleanup-11

bgpd: Cosmetic change for mac check

tests: update to munet 0.13.3, restoring GNU screen functionality

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

bgpd: Cosmetic change for mac check

Remove useless `return` code, and correct a spelling.

Signed-off-by: anlan_cs <vic.lan@pica8.com>

ospfd: opsf_abr.c memory leak fix, free range

The new create range attribute is send to add to ranges, but not use if the prefix already exist in the ranges.

Signed-off-by: ryndia <dindyalsarvesh@gmail.com>

Merge pull request #13488 from LabNConsulting/chopps/wf-python

doc: python, required by clippy, is no longer a non-essential tool

    ospfd: OSPF opaque LSA stale processing fix and topotests.

        1. Fix OSPF opaque LSA processing to preserve the stale opaque
            LSAs in the Link State Database for 60 seconds consistent with
            what is done for other LSA types.
         2. Add a topotest that tests for cases where ospfd is restarted
            and a stale OSPF opaque LSA exists in the OSPF routing domain
            both when the LSA is purged and when the LSA is reoriginagted
            with a more recent instance.

Signed-off-by: Acee <aceelindem@gmail.com>

doc: python, required by clippy, is no longer a non-essential tool

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

doc: Add `neighbor aigp` command for BGP

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #13429 from opensourcerouting/feature/mark_pr_as_need_rebase

github: Add rebase label on PR if it's > 50 commits behind

Merge pull request #13438 from gpnaveen/vrf_on_loopback

tests: removing un used loopback interface from bgp admin dist.

Merge pull request #13460 from Jafaral/ospf-fix2

tests: remove sleep ospf test

Merge pull request #13484 from sri-mohan1/srib-ldpd

ldpd: changes for code maintainability

Merge pull request #13470 from crosser/fix-test-frrscript

test_frrscript: fix dep for out of tree build

ldpd: changes for code maintainability

these changes are for improving the code maintainability and readability

Signed-off-by: sri-mohan1 <sri.mohan@samsung.com>

Merge pull request #13445 from donaldsharp/lua_scripting_mem_leak

zebra: Reduce creation and fix memory leak of frrscripting pointers

Merge pull request #13437 from raja-rajasekar/raja-rajasekar/show_cmd_fix

bgpd: Fixing the show bgp <vrf> <afi> <safi> detail command

github: Add rebase label on PR if it's > 50 commits behind

The idea is to catch some PRs and rerun them in CI to avoid merging PRs that
are based on the stale base branch and causes issues afterwards.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #13471 from opensourcerouting/fix/aigp_non_transitive

bgpd: AIGP should be non-transitive

tests: remove sleep ospf test

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

tests: Fix out of tree build for lua scripting

test_frrscript is run from the `tests` directory and expects the sample
lua script `script1.lua` to be present in the `lib` directory. When the
package is built out of tree (which always happens when a debian
package is built), and scripting is enabled, test fails because the lua
file is not present in the `tests/lib/` subdir of the _build_ directory.

Fix this by adding `script1.lua` as an extra dependency for
`test_frrscript`, and a recipe that copies the file from the source tree
to the build tree (note: it needs to be marked ".PHONY" because
otherwise `make` thinks that it already exists, in the source tree).

After this commit, the following command starts to work:

    dpkg-buildpackage --build-profiles=pkg.frr.lua -b -uc

Signed-off-by: Eugene Crosser <crosser@average.org>

tests: Adjust AIGP metric numbers for iBGP setup

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

bgpd: Make sure AIGP attribute is non-transitive

The AIGP attribute is an optional, non-transitive BGP path attribute.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

doc: add documentation for show [ip] bgp [vrf] [afi] [safi] detail [json]

Signed-off-by: Rajasekar Raja <rajasekarr@nvidia.com>

Merge pull request #13468 from opensourcerouting/ripd-memtype-fix

ripd: fix another memtype mismatch

Merge pull request #13413 from chiragshah6/fdev2

zebra: re-install NHG on interface up

Merge pull request #13464 from sri-mohan1/srib-ldpd

ldpd: changes for code maintainability

ripd: fix another memtype mismatch

Fixes #13447.

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

Merge pull request #13465 from donaldsharp/ospf_metric_ifindex

tests: ospf_metric_propagation is looking for a specific ifindex and vrfid

Merge pull request #13453 from donaldsharp/dplane_memory_leak

Dplane memory leak

tests: ospf_metric_propagation should not look for a specific vrfId

There is no guarantee that the vrfId is going to be the same across
tests, as that the vrfId is chosen based upon the ifindex of the
vrf device.  As such we should not be looking for the vrfId, but
the correct vrf name.

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

tests: ospf_metric_propagation is looking for a specific ifindex

The test ospf_metric_propagation is looking for a specific ifindex
this ifindex is not guaranteed to be any particular value by the underlying
OS.  So let's remove this test for it.  As a side note I am seeing
tests fail in upstream CI because of this.

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

Merge pull request #13446 from louis-6wind/fix-ext-comm

bgpd: fix show running of nt extcommunity

ldpd: changes for code maintainability

these changes are for improving the code maintainability and readability

Signed-off-by: sri-mohan1 <sri.mohan@samsung.com>

Merge pull request #13440 from zice312963205/bgpfix

bgpd: fix the issue of connected tag error when BGP subscribes to NHT…

Merge pull request #13458 from sri-mohan1/srib-ldpd

ldpd: changes for code maintainability

ldpd: changes for code maintainability

these changes are for improving the code maintainability and readability

Signed-off-by: sri-mohan1 <sri.mohan@samsung.com>

Merge pull request #13455 from sri-mohan1/srib-ldpd

ldpd: changes for code maintainability

ldpd: changes for code maintainability

these changes are for improving the code maintainability and readability

Signed-off-by: sri-mohan1 <sri.mohan@samsung.com>

Merge pull request #13451 from sri-mohan1/srib-ldpd

ldpd: changes for code maintainability

Merge pull request #13424 from LabNConsulting/chopps/log-config-file-cmds

bgpd: fix the issue of connected tag error when BGP subscribes to NHT from Zebra

Imagine the following scenario:
1.Create a multihop ebgp peer and config the ttl as 254 for both side.
2.Call bgp_start and start an active connection.
Bgp will send a nht register with non-connected flag.
3.The function bgp_accept be called by remote connection.
Bgp will create a accept peer as a passive connection with default ttl(1). And then will send a nht register again with connected flag. This register result will cover the first one.
4.The active connection come to establish first. In funciton "peer_xfer_conn", check for "PEER_FLAG_CONFIG_NODE" flag of "from_peer->doppelganger" will not be pass, so we can not repair the nht register error forever.
Then the bgp nexthop will be like this:
2000::60 invalid, #paths 0, peer 2000::60
Must be Connected
Last update: Thu May 4 09:35:14 2023

The route from this peer can not be treat with a vaild nexthop forever.
This change will fix this error.

Signed-off-by: Jack.zhang <hanyu.zly@alibaba-inc.com>

zebra: dplane_gre_set could return while leaking ctx

Prevent this function from leaking the ctx memory.
Also properly record that something has gone wrong.

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

zebra: Dplane ctx allocation cannot fail

Having tests for memory allocation success makes no sense
given what happens when frr fails to allocate memory.

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

zebra:re-install dependent nhgs on interface up

Upon interface up associated singleton NHG's
dependent NHGs needs to be reinstalled as
kernel would have deleted if there is no route
referencing it.

Ticket:#3416477
Issue:3416477
Testing Done:
flap interfaces which are part of route NHG,
upon interfaces up event, NHGs are resynced
into dplane.

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

zebra: re-install nhg on interface up

Intermittently zebra and kernel are out of sync
when interface flaps and the add's/dels are in
same processing queue and zebra assumes no change in nexthop.
Hence we need to bring in a reinstall to kernel
of the nexthops and routes to sync their states.

Upon interface flap kernel would have deleted NHGs
associated to a interface (the one flapped),
zebra retains NHGs for 3 mins even though upper
layer protocol removes the nexthops (associated NHG).
As part of interface address add ,
re-add singleton NHGs associated to interface.

Ticket: #3173663
Issue: 3173663

Signed-off-by: Ashwini Reddy <ashred@nvidia.com>
Signed-off-by: Chirag Shah <chirag@nvidia.com>

ripd: Do not overrun with more ECMP paths than Zebra supports

Let's say FRR is compiled with ECMP max 16, we enter `allow-ecmp 10`, but
Zebra supports only 4.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

ldpd: changes for code maintainability

these changes are for improving the code maintainability and readability

Signed-off-by: sri-mohan1 <sri.mohan@samsung.com>

zebra: Reduce creation and fix memory leak of frrscripting pointers

There are two issues being addressed:

a) The ZEBRA_ON_RIB_PROCESS_HOOK_CALL script point
was creating a fs pointer per dplane ctx in
rib_process_dplane_results().

b) The fs pointer was not being deleted and directly
leaked.

For (a) Move the creation of the fs to outside
the do while loop.

For (b) At function end ensure that the pointer
is actually deleted.

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

bgpd: fix show running of nt extcommunity

Setting

> bgp extcommunity-list standard EXT seq 5 permit nt 192.168.1.3:0

results in:

> bgp extcommunity-list standard EXT seq 5 permit NT:192.168.1.3

Fix the display

Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>

Merge pull request #13436 from donaldsharp/ospf_json_mem_leak

ospfd: Fix memory leaks w/ `show ip ospf int X json` commands

Merge pull request #13428 from opensourcerouting/ripd-bfd-fixes

ripd: fix problemas

Merge pull request #13435 from mjstapp/fix_pim_cpp_notice

pimd: replace CPP_NOTICE lines with TODO comments

Merge pull request #13423 from LabNConsulting/chopps/coverity-string-fix

mgmtd: simplify commit id to just be a timeval string

bgpd: Fixing the show bgp <vrf> <afi> <safi> detail command

Adding support to show cmds like  show bgp vrf all detail, show bgp
<afi> <safi> detail & show bgp <vrf> <afi> <safi> detail

Issue:3168406
Ticket:#3168406

Signed-off-by: Rajasekar Raja <rajasekarr@nvidia.com>

tests: removing un used loopback interface from bgp admin dist.

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

lib: log commands read from config file

When the user specifies `--command-log-always` in CLI arguments then also log
commands executed from loading the config file.

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

mgmtd: simplify commit id to just be a timeval string

Also fixes coverity warning

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

Merge pull request #13357 from Jafaral/fix_ospf_prop

tests: clear the ospf neighbor to force a state change

ospfd: Fix memory leaks w/ `show ip ospf int X json` commands

FRR has a memory leak in the case when int X does not
exist and a memory leak when int X does exist.  Fix
these

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

pimd: replace CPP_NOTICE lines with TODO comments

Replace the noisy CPP_NOTICE lines with TODO comments.

Signed-off-by: Mark Stapp <mjs@labn.net>

tests: clear the ospf neighbor to force a state change

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

ripd: Implement `allow-ecmp X` command

Allow setting an arbitrary number of paths to be installed instead of ALL.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

doc: Add RIP `allow-ecmp` command

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

tests: Check RIP allow-ecmp an arbitrary number of paths

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

tests: Check if RIP `allow-ecmp` command can inject specified number of routes

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

ripd: use correct memory type when freeing BFD profile

Fixes #13415.

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

ripd: fix null-pointer dereference

Fixes #13416.

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

Merge pull request #13425 from FRRouting/revert-12646-mpls_alloc_per_nh

Revert "MPLS allocation mode per next hop"

Revert "MPLS allocation mode per next hop"

Broken tests, let's revert now.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

Merge pull request #13422 from donaldsharp/pim_thread_off_fix

pimd: Fix old commit that got in

pimd: Fix old commit that got in

An old fix that used THREAD_OFF was pushed in, should
have used EVENT_OFF instead.

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

Merge pull request #13020 from SaiGomathiN/2462808-3

pimd: PIM not sending register packets after changing from non DR to DR

Merge pull request #12646 from pguibert6WIND/mpls_alloc_per_nh

MPLS allocation mode per next hop

Merge pull request #13376 from louis-6wind/fix-flex-algo-mem-leak

isisd, lib: fix flex-algo memory leak

Merge pull request #13366 from zmw12306/rte_tag

ripd: add non-zero check for RIPv1 reserved field

Merge pull request #13403 from anlancs/fix/zebra-missing-vrf-flag

zebra: Fix missing VRF flag

Merge pull request #13404 from louis-6wind/fix-flex-algo-race-condition

isisd: fix a flex algo race condition and a minor fix

Merge pull request #13235 from Orange-OpenSource/link-state

lib: Fix memory leaks in Link State library

Merge pull request #13270 from pguibert6WIND/better_srv6_output_seg6local

zebra: display seg6local only when specified

Merge pull request #13354 from zmw12306/parse_request_subtlv

babeld: add parse_request_subtlv into parse_packet

Merge pull request #13377 from donaldsharp/ospf_dead_interval

tests: Increase the dead interval to be longer for neighbor testing

Merge pull request #13389 from anlancs/fix/pimd-wrong-mrt-sockopt

pimd: Fix wrong setsockopt() call

Merge pull request #13393 from opensourcerouting/fix/topotest_bgp_bfd_down_cease_notification

tests: Do not try establishing a connection from r1 to r2

Merge pull request #13395 from LabNConsulting/chopps/mgmtd-debug-flags

mgmtd: fully implement debug flags for mgmtd and clients