bgpd: Free ->raw_data from Hard Notification message after we use it

==175785== 0 bytes in 1 blocks are definitely lost in loss record 1 of 88
==175785==    at 0x483DD99: calloc (in /usr/lib/x86_64-linux-gnu/valgrind/vgpreload_memcheck-amd64-linux.so)
==175785==    by 0x492EB8E: qcalloc (in /usr/local/lib/libfrr.so.0.0.0)
==175785==    by 0x269823: bgp_notify_decapsulate_hard_reset (in /usr/lib/frr/bgpd)
==175785==    by 0x26C85D: bgp_notify_receive (in /usr/lib/frr/bgpd)
==175785==    by 0x26E94E: bgp_process_packet (in /usr/lib/frr/bgpd)
==175785==    by 0x4985349: thread_call (in /usr/local/lib/libfrr.so.0.0.0)
==175785==    by 0x491D521: frr_run (in /usr/local/lib/libfrr.so.0.0.0)
==175785==    by 0x1EBEE8: main (in /usr/lib/frr/bgpd)
==175785==

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

Merge pull request #11579 from opensourcerouting/fix/revert_isis_fast_reroute

isisd: Revert fast-reroute recent changes

Revert "isisd: apply fast-reroute on an adjacency failure"

This reverts commit 71252973234e70ffa3cb5eec43d4efc1be3d3331.

Revert "isisd: remove previous labels before fast-reroute"

This reverts commit a254fa54e3d5c31befb40164fc70deab3cf639ad.

Revert "isisd: allow nexthop lookup on interface only"

This reverts commit a11742dc53e31d1379cb4f7b9b7b750614e30928.

Revert "isisd: apply fast-reroute when an interface falls down"

This reverts commit 62e458278b3d261f492ffeac7ed70b32272a43dd.

Revert "isisd: avoid fast-reroute on down adjacency when the interface is down"

This reverts commit 08e4960ab48c1c9e4227e40b286d5cbc4ac581f0.

Revert "isisd: avoid fast-reroute from running twice on a down interface event"

This reverts commit d526476184999275c318b13f92ae6b38a0250052.

Revert "topotests: isis-tilfa add a switchover test after linkdown"

This reverts commit bac0d1e6068ae46502bd5b348765faea554ebf72.

Revert "topotests: isis-tilfa add a switchover test after BFD down"

This reverts commit e83a52698b31fde2532b4a5a8c5223ba5e858ef2.

Revert "isisd: fix metric calculation of classic lfa backup prefixes"

This reverts commit e45958e9cfccd6dfc108a7ef5cb73775e8ef2e68.

Revert "isisd: fix SPF scheduling on IPv6 only topology"

This reverts commit d95cd33545006004424736d20a0226b5cd9fb47a.

Revert "topotests: isis-lfa add a switchover test after linkdown"

This reverts commit 5558ac984fa3adc5beb66a1945ad169705a28a37.

Revert "topotests: isis-lfa add a switchover test after hello timeout"

This reverts commit c99ef489629ffa057b312756771693e417309995.

Revert "topotests: isis-lfa add a switchover test after neighbor clear"

This reverts commit 9cdc099bb5d1fa0c3949847759469bfc444a885c.

Revert "topotests: isis-lfa add a switchover test after BFD down"

This reverts commit c8d2816ee5245ac2a13cbb36299f2f0c23f226e3.

Merge pull request #11570 from opensourcerouting/pim6-bsr-clear

pim6d: add `clear ipv6 pim bsr-data`

Merge pull request #11569 from opensourcerouting/gcc-plugin-shortcomings-220708

bgpd: work around GCC/plugin shortcomings

Merge pull request #11568 from anlancs/fix/minor-4

bgpd: fix missing rmac value in debug

Merge pull request #10962 from louis-6wind/lfa-netlink

isisd: apply fast-reroute as soon an interface or an adjacency falls down

pim6d: add `clear ipv6 pim bsr-data`

This is needed for ANVL between testruns, and already in IPv4 too.

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

bgpd: work around GCC/plugin shortcomings

The GCC extended printf format checking plugin runs into some GCC
shortcomings regarding casts on printf function parameters.  While this
can be fixed with a small GCC patch, patching GCC is "nontrivial" to say
the least.  Luckily, it happens that this is /almost/ not an issue for
the FRR source base.

Since we fix SA "misunderstandings" too, let's just fix places where the
format checking plugin runs into this limitation to keep things working
extra smoothly.

(It's not a huge effort either, these two spots in bgpd are the only
places that trigger the plugin limitation, and it's been "clean" before
that for more than a year if my memory is right.)

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

Merge pull request #10592 from patrasar/master_pimv6_bsm

Merge pull request #11553 from patrasar/clear_ip_mroute

pimd, pim6d: Fix description for clear ip[v6] mroute command

bgpd: fix missing rmac value in debug

`attr.rmac` is not set in debug as expected for its wrong place in code.

Just move the debug process (`bgp_debug_zebra(NULL)`) after possible `rmac`
value is set.

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

pim6d: removed pim6_stubs.c file

pim6_stub.c file is no more needed.

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: Adjust pim_show_bsm_db for IPV6

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: IPV6 adjust in hash_calc_on_grp_rp

Signed-off-by: sarita patra <saritap@vmware.com>

Revert "pim6d: IPv6-adjust BSM code"

This reverts commit 809c11a9d87e0d12a5c3d3ca6906bc398a58de6c.

pim6d: remove unused variable neigh_src_str

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: running pim_bsm.c for pimv6

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: add pimv6 support for bsmmsg_grpinfo

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: add pimv6 support for bsmmsg_rpinfo

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: add pimv6 support for bsm_hdr

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: Add pim_addr rp_addr in bsm_rpinfo

Signed-off-by: sarita patra <saritap@vmware.com>

Merge pull request #11541 from opensourcerouting/zebra-mr-netlink

zebra: fix various multicast routing netlink API bits

Merge pull request #11545 from opensourcerouting/fix/memory_leak_for_bmp_listener

bgpd: Free memory for BMP listeners when deleting BGP instance

Merge pull request #11549 from opensourcerouting/fix/logrotate

tools: Add missing bfdd to logrotate config

pim6d: bsr nht handling for IPV6

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: modify in_addr current_bsr to pim_addr current_bsr

Signed-off-by: sarita patra <saritap@vmware.com>

Merge pull request #11552 from opensourcerouting/pkgfix

Package build fixes

Merge pull request #10715 from patrasar/pim_rpf_prefix_to_pimaddr

pim6d: modify pim_rpf from struct prefix to pim_addr

pim6d: Fix description for "clear ipv6 mroute" cmd

Signed-off-by: Sarita Patra <saritap@vmware.com>

pimd: Fix description for "clear ip mroute" cmd

Signed-off-by: Sarita Patra <saritap@vmware.com>

snapcraft: Version must be quoted in snap store

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

debian: Fix version requirement for rtrlib to allow distro specific names

Signed-off-by: Martin Winter <mwinter@opensourcerouting.org>

zebra: debug decode RTA_EXPIRES and RTA_MFC_STATS

Just adding two more attributes to decode and show nicely in netlink
msgdump debug output.

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

zebra: fix remaining MR RTM_GETROUTE oddities

The IPv6 version needs rtm_src_len and rtm_dst_len filled in due to
strict validation.  IPv4 also has this requirement, but zebra is running
in non-strict mode there so the kernel accepts it...

Also the table ID hack is IPv4 only.

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

tools: Add missing bfdd to logrotate config

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

Merge pull request #11530 from ARShreenidhi/default_originate_testcase

tests : bgp default originate test cases

Merge pull request #11548 from paulc39/check_hostname_in_resolver_resolve

lib: check hostname in resolver_resolve

lib: check hostname in resolver_resolve

resolver_resolve should check hostname is null or not.

if ares_gethostbyname() get null hostname string, the hostname string will access a null pointer and crash.

Signed-off-by: kevinshen <kevinshen@inspur.com>

bgpd: Free memory for BMP listeners when deleting BGP instance

When using `no router bgp` we MUST free the memory for the listeners too.

Replicate with:
```
router bgp 100
bmp targets server1
bmp listener 0.0.0.0 port 65535
no router bgp
router bgp 100
bmp targets server1
bmp listener 0.0.0.0 port 65535
no router bgp
router bgp 100
bmp targets server1
bmp listener 0.0.0.0 port 65535
no router bgp
```

Before:
```
Type                          : Current#   Size       Total     Max#  MaxBytes
BMP targets                   :        0    240           0        1       248
BMP targets name              :        0      8           0        1        24
BMP listener                  :       12    152        1824       12      1824
BMP instance state            :        0     88           0        1        88
```

After:
```
Type                          : Current#   Size       Total     Max#  MaxBytes
BMP targets                   :        0    240           0        1       248
BMP targets name              :        0      8           0        1        24
BMP listener                  :        0    152           0        1       152
BMP instance state            :        0     88           0        1        88
```

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

Merge pull request #11542 from opensourcerouting/bfd-coverity

bfdd: fix coverity scan issues

Merge pull request #11540 from opensourcerouting/typesafe-assume-explain

lib: use assume() for SA fixing, add explainer

bfdd: fix coverity memory overrun

Use the destination for the operator `sizeof()` instead of the source
which could (and is) be bigger than destination.

We are not truncating any data here it just happens that the zebra
interface data structure hardware address can be bigger due to different
types of interface.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

bfdd: fix coverity scan resource leak

Close the descriptor if something fails and we don't return it.

Signed-off-by: Rafael Zalamena <rzalamena@opensourcerouting.org>

pim6d: Pass pim_addr for pim_nht callbacks.

For pim callbacks, we pass pim_addr as value, not pointer.
So making it consistent for pim_nht callbacks.

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Adjust rpf_addr in show commands

Signed-off-by: sarita patra <saritap@vmware.com>

Merge pull request #11521 from patrasar/pimv6_drpriority

pim6d: Fix [no] ipv6 pim drpriority command

pimd: Handle rpf_addr in pim_ifchannel code

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Handle rpf_addr in pim nht

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Handle rpf_addr in register send/recv

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: Handle rpf_addr in join/prune processing

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: Handle rpf_addr in pim_msg_build_jp_groups()

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: Handle rpf_addr in on_neighbor_jp_timer()

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: Handle rpf_addr in pim_if_addr_add()

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: Adjust rpf_addr in pim_tib code

In this PR, we are handling the pim_tib code
after rpf_addr modified from prefix to pim_addr.

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: Adjust rpf_addr in pim_vxlan_orig_mr_up_add

In this PR, we are handling the pim_vxlan_orig_mr_up_add()
after rpf_addr modified from prefix to pim_addr.

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: Adjust rpf_addr in pim_zebra

In this PR, we are handling the pim_zebra code
after rpf_addr modified from prefix to pim_addr.

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: Adjust rpf_addr in pim_upstream

In this PR, we are handling the pim_upstream
after rpf_addr modified from prefix to pim_addr.

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: Adjust rpf_addr in rp_info

In this PR, we are handling the rp_info
after rpf_addr modified from prefix
to pim_addr.

Signed-off-by: sarita patra <saritap@vmware.com>

pim6d: Adjust rpf_addr rpf code

In this PR, we are handling the pim_rpf code
after rpf_addr modified from prefix
to pim_addr.

Signed-off-by: sarita patra <saritap@vmware.com>

pimd: modify rpf_addr from prefix to pim_addr

Currently rpf_addr is of type struct prefix, but it's always
a /32 host prefix. So changing it to pim_addr in order to support both
IPV4 and IPV6.

Signed-off-by: sarita patra <saritap@vmware.com>

Merge pull request #9307 from SaiGomathiN/sai-nht

pimd: json support for nexthop

zebra: IIF/OIF are not used in MR RTM_GETROUTE

The multicast routing RTM_GETROUTE command does not use IIF/OIF
attributes, and the IPv6 version will refuse them with an error due to
being new netlink API and thus using strict validation.

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

zebra: netlink MR is a rtmsg, not ndmsg

These two structs happen to be the same size and have the family field
in the same spot, but the correct one to use here is rtmsg not ndmsg.

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

zebra: correctly ignore multicast nl msgs

zebra does not care about _notifications_ from the kernel regarding
multicast routing;  we only use the MR netlink API to request stats from
the kernel by actively sending a RTM_GETROUTE.

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

lib: use assume() for SA fixing, add explainer

Literally 4 minutes after hitting merge on Mark's previous fix for this
I remembered we have an `assume()` macro in compiler.h which seems like
the right tool for this.

(... and if I'm touching it, I might as well add a little text
explaining what's going on here.)

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

Merge pull request #11534 from mjstapp/fix_typesafe_sa

lib: fix SA warning in typesafe code

Merge pull request #11524 from kuldeepkash/multicast_pim_bsm

tests: Fix for multicast_pim_bsm_topo1 key error

lib: fix SA warning in typesafe code

There's a nagging SA warning, at least with the scan-build
version we use in the FRR CI.

Signed-off-by: Mark Stapp <mstapp@nvidia.com>

Merge pull request #11519 from opensourcerouting/feature/turn_on_reserved_ip_ranges

Allow using IPv4 (Class E) IP ranges for daemons

Merge pull request #11494 from ak503/ospf_bfd

ospfd: set/unset bfd profile

Merge pull request #11502 from donaldsharp/zebra_dplane_fini

zebra: make rib_process_dplane_results own ctx freeing

Merge pull request #11508 from louis-6wind/lsp-parse

isisd: fix infinite loop when parsing LSPs

Merge pull request #11515 from opensourcerouting/fix/cond_advertisements_update_type

bgpd: Start conditional advertisements instantly if we configure them

Merge pull request #11517 from louis-6wind/advert-map

topostest: bgp_conditional_advertisement cleanup

tests : bgp default originate testcases

this commit containes 2 testcases that covers
1. Default originate behaviour on restarting the BGP daemon and FRR router
2. Default Originate behaviour on shut no-shutting the interface

Signed-off-by: ARShreenidhi <rshreenidhi@vmware.com>

Merge pull request #11527 from mobash-rasool/fixes

pimd: Remove unused extern pim_channel_oil_list

Merge pull request #11371 from rampxxxx/feat_bfd_ipv4_echo_fwd_path

bfdd: add IPv4 BFD Echo support that loops pkts in forwarding plane

pimd: Remove unused extern pim_channel_oil_list

Removing the extern declaration of pim_channel_oil_list since
its not required anymore.

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

tests: Fix for multicast_pim_bsm_topo1 key error

Issue was reported by Donald, we were hitting
with key not found error and execution was
stopped, which is fixed by this PR.

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

Merge pull request #11507 from donaldsharp/setsockopt_changes

pim6d: fix no ipv6 pim drpriority

Signed-off-by: Sarita Patra <saritap@vmware.com>

pim6d: configure ipv6 pim drpriority

Signed-off-by: Sarita Patra <saritap@vmware.com>

Merge pull request #11514 from donaldsharp/zebra_odds_and_ends

Zebra odds and ends

Merge pull request #11520 from donaldsharp/zebra_debug_queue

zebra: Add a subqueue2str function to give more useful data in debugs

zebra: Add a subqueue2str function to give more useful data in debugs

New output example:

2022-07-03 09:40:29.310 [DEBG] zebra: [JF0K0-DVHWH] rib_meta_queue_add: (0:254):4.5.6.8/32: queued rn 0x55937f586ee0 into sub-queue Kernel Routes
2022-07-03 09:40:29.321 [DEBG] zebra: [HH6N2-PDCJS] default(0:254):4.5.6.8/32 rn 0x55937f586ee0 dequeued from sub-queue Kernel Routes

Let's make it a bit more human readable.

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

Merge pull request #11505 from SaiGomathiN/watermark

pim6d: Fix the todo items in ipv6 watermark limit CLI