tests: fix zebra_rib range

Range is wrong. We want values 1 and 2 but we only test 1.

> >>> for i in range(1, 2):
> ...     print(i)
> ...
> 1

Fixes: abd2a1ff3f ("tests: Test some basic kernel <-> zebra interactions")
Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>

Merge pull request #16902 from donaldsharp/forwarding_on_off

tests: Add v4/v6 forwarding off/on

tests: Add v4/v6 forwarding off/on

There are no tests that ensured that turning off then on
v4 and v6 forwarding actually worked.  This does so.
This was found via looking at the code coverage.

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

Merge pull request #16859 from mjstapp/bgp_cancel_once

bgpd: cancel events once in peer_free()

Merge pull request #16873 from louis-6wind/fix-yang-rmap

yang: use relative path for route-map

Merge pull request #16882 from mjstapp/fix_if_table_unlock

zebra: unlock if_table route_nodes

Merge pull request #16887 from zhou-run/202409211508

isisd: Fix infinity flag not being set successfully

Merge pull request #16886 from LabNConsulting/aceelindem/ospf-nbr-filter-startup-config-fix

ospfd: Interface 'ip ospf neighbor-filter' startup config not applied.

Merge pull request #16891 from okda-networks/bgp_yang

yang: added as-notation leaf to global-bgp-config

Merge pull request #16892 from cscarpitta/tests/add-more-srv6-testcases

tests: Add more SRv6 testcases

tests: Add testcase for SRv6 End.DT6

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

tests: Add testcase for SRv6 End.DT4

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

tests: Add testcase for SRv6 End.DX6

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

tests: Fix unused variable warning

Fix warning unused variable `router_list`.

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

yang: added as-notation leaf to global-bgp-config

Signed-off-by: ali aqrabawi <aaqrabaw@okdanetworks.com>

Merge pull request #16889 from cscarpitta/fix/remove-duplicate-import-toopotest

tests: Remove duplicate `check_ping` import

Merge pull request #16888 from cscarpitta/fix/remove-log-monitor-cli

tests: Remove `log monitor XX ` from topotests

Merge pull request #16883 from donaldsharp/bgp_dump_stuff

tests: Add a very basic `dump bgp PATH..` topotest

tests: Remove duplicate `check_ping` import

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

tests: Remove `log monitor XX` from `srv6_sid_manager` test

`log monitor XX` command is deprecated and does nothing. Let's remove it.

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

tests: Remove `log monitor XX` from `srv6_sid_manager` test

`log monitor XX` command is deprecated and does nothing. Let's remove it.

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

tests: Remove `log monitor XX` from `srv6_encap_src_addr` test

`log monitor XX` command is deprecated and does nothing. Let's remove it.

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

isisd: Fix infinity flag not being set successfully

When creating SPF neighbors and calling spf_adj_list_parse_tlv(), the F_ISIS_SPF_ADJ_METRIC_INFINITY flag is only set to a local variable flags and not set to the SPF neighbor sadj->flags. Additionally, the pseudo node metric pseudo_metric is also not reflected in the F_ISIS_SPF_ADJ_METRIC_INFINITY flag, the metric should be checked using sadj->metric.

Signed-off-by: zhou-run <zhou.run@h3c.com>

ospfd: Interface 'ip ospf neighbor-filter' startup config not applied.

When 'ip ospf neighbor-filter <filter-name>' is configured in the startup
configuration, it is in the running configuration but not applied on the
interface.

Signed-off-by: Acee Lindem <acee@lindem.com>

Merge pull request #16881 from donaldsharp/zebra_nhg_install_fix

zebra: Pass in ZEBRA_ROUTE_MAX instead of true

tests: Add a very basic `dump bgp PATH..` topotest

None of the bgp dump code was even tested.  Add a bit
of basic stuff that it at least generates a dump file.
This can be extended at a future time.

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

zebra: unlock if_table route_nodes

Must unlock if we break during iteration over any lib/table
tree.

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

zebra: Pass in ZEBRA_ROUTE_MAX instead of true

zebra_nhg_install_kernel takes a route type.  We don't
know it at that particular spot but we should not be passing
in `true`.  Let's use ZEBRA_ROUTE_MAX to indicate we do not
know, so that the correct thing is done.

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

Merge pull request #16876 from opensourcerouting/fix/ZEBRA_SEG6_LOCAL_ACTION_END_DX6_should_put_full_inaddr6

zebra: Send a correct size of ctx->nh6 for SRv6 SEG6_LOCAL_ACTION_END_DX6

Merge pull request #16874 from enkechen-panw/redist-fix

bgpd: fix misplaced arguments in bgp_redistribute_add()

Merge pull request #16875 from opensourcerouting/fix/deprecate_json

isisd: Remove deprecated JSON fields

zebra: Send a correct size of ctx->nh6 for SRv6 SEG6_LOCAL_ACTION_END_DX6

Fixes: f6e58d26f638d0bcdc34dfc5890669036a0129df ("zebra, sharpd: add srv6 End.DX6 support")
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>

isisd: Remove deprecated JSON fields

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

Merge pull request #16738 from odivlad/fix-stdout-fork

lib: Attach stdout to child only if --log=stdout and stdout FD is a tty

Merge pull request #16782 from pguibert6WIND/fix_nexthop_set_failed_message

bgpd: fix 'nexthop_set failed' error message often displayed

Merge pull request #16872 from donaldsharp/remove_nl_addraw_l

zebra: Remove nl_addraw_l

tests: adjust topotest after bgp_redistribute_add() fix

In ospf_multi_vrf_bgp_route_leak, the admin distance for the
redistributed ospf route should be 110, and should remain as 110 after
it's imported into another vrf, and then downloaded to zebra.

Signed-off-by: Enke Chen <enchen@paloaltonetworks.com>

bgpd: fix misplaced arguments in bgp_redistribute_add()

Fix the misplaced arguments "bhtype" and "api.distance" in
bgp_redistribute_add().

Signed-off-by: Enke Chen <enchen@paloaltonetworks.com>

zebra: Remove nl_addraw_l

This function is never used.  So let's remove it.

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

Merge pull request #16848 from enkechen-panw/ecomm-val

bgpd: define val in ecommunity_val as uint8_t

Merge pull request #16868 from sri-mohan1/srib-24-frr-a

bgpd: changes for code maintainability

yang: use relative path for route-map

f1ea52bee9 ("yang: use relative path instead of absolute one for
route-map") has introduced a route-map optimization but some paths
were not changed.

Use relative path instead absolute one everywhere.

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

bgpd: 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 #16860 from LabNConsulting/aceelindem/ospf-sr-algo-vulnerability

ospfd: Fix heap corruption vulnerability when parsing SR-Algorithm TLV

bgpd: define val in ecommunity_val as uint8_t

The type of the val field in ecommunity_val is used inconsistently
in a number of places. It should be defined as uint8_t.

Signed-off-by: Enke Chen <enchen@paloaltonetworks.com>

Merge pull request #16857 from donaldsharp/logger_warning

tests: use logger.warning not logger.warn

ospfd: Fix heap corruption vulnerability when parsing SR-Algorithm TLV

When parsing the SR-Algorithm TLV in the OSPF Router Information Opaque
LSA, assure that not more than the maximum number of supported
algorithms are copied from the TLV.

Signed-off-by: Acee Lindem <acee@lindem.com>

Merge pull request #16846 from donaldsharp/zebra_evpn_mac_cleanup

Zebra evpn mac cleanup

bgpd: cancel events once in peer_free()

Don't need to cancel scheduled events twice in a row - just
once.

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

Merge pull request #16856 from donaldsharp/ospf_vrf_debugs

ospfd: Add vrf name to debug functions in ospf_zebra.c

Merge pull request #16841 from mjstapp/fix_bgp_script

bgpd: remove bgpd.h from bgp script header

Merge pull request #16816 from opensourcerouting/feature/bgp_dual_as

bgpd: Implement BGP dual-as feature

tests: use logger.warning not logger.warn

Apparently logger.warn is being deprecated.  So let's
switch over to logger.warning.  Clearly it's better

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

Merge pull request #16720 from opensourcerouting/fix/default_originate_not_needed_if_not_enabled

bgpd: Do not scan update-groups if default-originate timer is set to 0

ospfd: Add vrf name to debug functions in ospf_zebra.c

This file had a bunch of debugs that were operating on
specific vrf's but not telling you which vrf this was
happening in.  Add code to allow operator debugging
to understand what vrf is being worked on.

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

zebra: In zebra_evpn_mac.c remove bad comments

Adding comments that tell what a variable is doing in
the middle of a function call makes it extremely hard
to read the formatting.  Remove.

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

zebra: Reindent some badly formatted functions in zebra_evpn_mac.c

Fix some badly formatted code to fit better on the screen.

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

zebra: Reframe zebra_evpn_mac.c to be properly formatted

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

Merge pull request #16849 from opensourcerouting/fix/remove_rpki_cmd

bgpd: Remove deprecated RPKI command

Merge pull request #16840 from LabNConsulting/chopps/yang-lib

add ietf-yang-library support

Merge pull request #16852 from sri-mohan1/srib-24-frr-a

bgpd: changes for code maintainability

Merge pull request #16850 from enkechen-panw/show-stat

bgpd: add counters for redistributed and aggregated routes

bgpd: changes for code maintainability

these changes are for improving the code maintainability and readability

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

tests: add test for new ietf-yang-library support

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

mgmtd: add ietf-yang-library support

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

bgpd: add counters for redistributed and aggregated routes

Add counters for redistributed routes, and local aggregates to the
output of "show ip bgp statistics".

Signed-off-by: Enke Chen <enchen@paloaltonetworks.com>

bgpd: Remove deprecated RPKI command

`rpki cache <A.B.C.D|WORD> ...`

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

Merge pull request #16774 from donaldsharp/bgp_give_some_ordering_increases

Bgp give some ordering increases

Merge pull request #16809 from louis-6wind/fix-rcap-double-free

isisd: fix rcap tlv double-free crash

Merge pull request #16839 from LabNConsulting/chopps/edit-reply

Add `created` and `changed` booleans to `edit-reply` message

bgpd: remove bgpd.h from bgp script header

Use forward references instead of including the main bgpd.h
header in the bgp scripting header to reduce external impact
of changes.

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

lib: mgmtd: add `changed` and `created` to edit-reply msg

- This is used for various return values in RESTCONF

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

lib: mgmtd: cleanup error value for native messaging

- Now if positive it's libyang LY_ERR, otherwise it's `-errno` value.

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

mgmtd: allow dest DS "running" if implicit lock+commit

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

lib: constify yang_resolve_snode_xpath results

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

Merge pull request #16828 from enkechen-panw/print-rnh

zebra: include the prefix in nht show command

tests: update isisd fuzz test

Since the previous commit, if a router capability subTLV is not
readable, the previously read subTLVs are kept.

Update of the ISIS fuzz test.

> $ wuschl rebuild tests/isisd/test_fuzz_isis_tlv
> $ gzip -9 tests/isisd/test_fuzz_isis_tlv_tests.h

> $ ./test_fuzz_isis_tlv 2>/dev/null | grep failed
> Test 139 failed, output differs.
> Test 150 failed, output differs.
> 2 of 405 tests failed.
>
> $ ./test_fuzz_isis_tlv 139 2>/dev/null
> Test 139 failed, output differs.
> Expected output:
> Unpack log:
> Unpacking 564 bytes of TLVs...
>   Unpacking TLV...
>     Found TLV of type 193 and len 13.
>     Skipping unknown TLV 193 (13 bytes)
>   Unpacking TLV...
>     Found TLV of type 0 and len 0.
>     Skipping unknown TLV 0 (0 bytes)
>   Unpacking TLV...
>     Found TLV of type 0 and len 0.
>     Skipping unknown TLV 0 (0 bytes)
>   Unpacking TLV...
>     Found TLV of type 242 and len 12.
>     Unpacking Router Capability TLV...
>     WARNING: Router Capability subTLV length too large compared to expected size
> Unpacked TLVs:
> Received output:
> Unpack log:
> Unpacking 564 bytes of TLVs...
>   Unpacking TLV...
>     Found TLV of type 193 and len 13.
>     Skipping unknown TLV 193 (13 bytes)
>   Unpacking TLV...
>     Found TLV of type 0 and len 0.
>     Skipping unknown TLV 0 (0 bytes)
>   Unpacking TLV...
>     Found TLV of type 0 and len 0.
>     Skipping unknown TLV 0 (0 bytes)
>   Unpacking TLV...
>     Found TLV of type 242 and len 12.
>     Unpacking Router Capability TLV...
>     WARNING: Router Capability subTLV length too large compared to expected size
> Unpacked TLVs:
> Router Capability: 253.212.128.242 , D:1, S:1
>
> $ ./test_fuzz_isis_tlv 150 2>/dev/null
> Test 150 failed, output differs.
> Expected output:
> Unpack log:
> Unpacking 403 bytes of TLVs...
>   Unpacking TLV...
>     Found TLV of type 129 and len 13.
>     Unpacking Protocols Supported TLV...
>       Protocols Supported: 73, 16, 255, 255, 255, 101, 10, 11, 11, 11, 11, 11, 11
>   Unpacking TLV...
>     Found TLV of type 11 and len 11.
>     Skipping unknown TLV 11 (11 bytes)
>   Unpacking TLV...
>     Found TLV of type 242 and len 12.
>     Unpacking Router Capability TLV...
>     WARNING: Router Capability subTLV length too large compared to expected size
> Unpacked TLVs:
> Protocols Supported: 73, 16, 255, 255, 255, 101, 10, 11, 11, 11, 11, 11, 11
> Received output:
> Unpack log:
> Unpacking 403 bytes of TLVs...
>   Unpacking TLV...
>     Found TLV of type 129 and len 13.
>     Unpacking Protocols Supported TLV...
>       Protocols Supported: 73, 16, 255, 255, 255, 101, 10, 11, 11, 11, 11, 11, 11
>   Unpacking TLV...
>     Found TLV of type 11 and len 11.
>     Skipping unknown TLV 11 (11 bytes)
>   Unpacking TLV...
>     Found TLV of type 242 and len 12.
>     Unpacking Router Capability TLV...
>     WARNING: Router Capability subTLV length too large compared to expected size
> Unpacked TLVs:
> Protocols Supported: 73, 16, 255, 255, 255, 101, 10, 11, 11, 11, 11, 11, 11
> Router Capability: 253.212.128.242 , D:1, S:1

Link: https://pypi.org/project/wuschl/
Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>

isisd: fix rcap tlv double-free crash

A double-free crash happens when a subTLV of the "Router Capability"
TLV is not readable and a previous "Router Capability" TLV was read.

rcap was supposed to be freed later by isis_free_tlvs() ->
free_tlv_router_cap(). In 78774bbcd5 ("isisd: add isis flex-algo lsp
advertisement"), this was not the case because rcap was not saved to
tlvs->router_cap when the function returned early because of a subTLV
length issue.

Always set tlvs->router_cap to free the memory.

Note that this patch has the consequence that in case of subTLV error,
the previously read "Router Capability" subTLVs are kept in memory.

Fixes: 49efc80d34 ("isisd: Ensure rcap is freed in error case")
Fixes: 78774bbcd5 ("isisd: add isis flex-algo lsp advertisement")
Reported-by: Iggy Frankovic <iggyfran@amazon.com>
Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>

Merge pull request #16835 from cscarpitta/add-sid-structure-to-seg6local-nh

bgpd, lib: Include SID structure in seg6local nexthop

bgpd: Include structure when removing End.DT46 SID

Include SID structure information when removing an SRv6 End.DT46 SID
from the forwarding plane.

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

bgpd: Include structure when removing End.DT4/6 SID

Include SID structure information when removing an SRv6 End.DT4 or End.DT6 SID
from the forwarding plane.

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

bgpd: Include structure when installing End.DT46 SID

Include SID structure information when installing an SRv6 End.DT46 SID
in the forwarding plane.

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

bgpd: Include structure when installing End.DT4/6 SID

Include SID structure information when installing an SRv6 End.DT6 or End.DT4 SID
in the forwarding plane.

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

lib: Include SID structure in seg6local nexthop

Include SID structure information in seg6local nexthop data structure.

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>

zebra: include the prefix in nht show command

Include the prefix in "show ip nht" and "show ipv6 nht".

Signed-off-by: Enke Chen <enchen@paloaltonetworks.com>

Merge pull request #16827 from LabNConsulting/chopps/fix-on-error-with-xdist

improvements for xdist mode

Merge pull request #16829 from Jafaral/vty-json

pimd: use vty_json

tests: fix spawning shells/vtysh on error in xdist mode

- Also fix the above and CLI when running in munet native mode

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

tests: update munet 0.14.14, 0.14.15

- improvement to remote CLI
- 0.14.15 fixed a small bug in 0.14.14

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

pimd: use vty_json

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

tests: update munet 0.14.13

- Improve remote CLI operation [improves xdist mode]

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

tests: update munet 0.14.12

- Adds retry_sleep (i.e., interval) parameter to native @retry decorator
- Fix --stdout and --stderr munet CLI args

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

Merge pull request #16438 from Jafaral/pim-10.1-cand-rp

PIM candidate BSR/RP support

Merge pull request #16818 from ak503/ospf_gr

ospfd: missing '[no]ip ospf graceful-restart hello-delay <N>' commands

Merge pull request #16815 from opensourcerouting/fix/adjust_wait_count_default_timers_if_too_low

tests: Adjust minimum wait/count timers for run_and_expect() if they are too low

Merge pull request #16808 from garyachy/nhrp_duplicate_authentication_extension_field

nhrpd: fixes duplicate auth extension

ospfd: fix missing '[no]ip ospf graceful-restart hello-delay <N>' commands

Signed-off-by: Dmitrii Turlupov <turlupov@bk.ru>

tests: Test `neighbor X no-prepend replace-as dual-as`

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

doc: Document `neighbox X local-as Y replace-as no-prepend dual-as`

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

bgpd: Implement BGP dual-as feature

This is helpful for migrations, etc.

The neighbor is configured with:

```
router bgp 65000
 neighbor X local-as 65001 no-prepend replace-as dual-as
```

Neighbor X can use either 65000, or 65001 to peer with.

Closes: https://github.com/FRRouting/frr/issues/13928
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>