summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2025-04-16Merge pull request #18670 from FRRouting/mergify/bp/stable/10.3/pr-18667stable/10.3Jafar Al-Gharaibeh
isisd: fix srv6_sid memory leak (backport #18667)
2025-04-16isisd: fix srv6_sid memory leakLouis Scalbert
Seen with isis_srv6_topo1 topotest. > ==178793==ERROR: LeakSanitizer: detected memory leaks > > Direct leak of 56 byte(s) in 1 object(s) allocated from: > #0 0x7f3f63cb4a57 in __interceptor_calloc ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:154 > #1 0x7f3f6366f8dd in qcalloc lib/memory.c:105 > #2 0x561b810c62b7 in isis_srv6_sid_alloc isisd/isis_srv6.c:243 > #3 0x561b8111f944 in isis_zebra_srv6_sid_notify isisd/isis_zebra.c:1534 > #4 0x7f3f637df9d7 in zclient_read lib/zclient.c:4845 > #5 0x7f3f637779b2 in event_call lib/event.c:2011 > #6 0x7f3f63642ff1 in frr_run lib/libfrr.c:1216 > #7 0x561b81018bf2 in main isisd/isis_main.c:360 > #8 0x7f3f63029d8f in __libc_start_call_main ../sysdeps/nptl/libc_start_call_main.h:58 Fixes: 0af0f4616d ("isisd: Receive SRv6 SIDs notifications from zebra") Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com> (cherry picked from commit 25c813ac382ba79270f40b85e168cdbcad499e2d)
2025-04-16Merge pull request #18666 from FRRouting/mergify/bp/stable/10.3/pr-18662Mark Stapp
tests: Fix northbound endian use in a unit-test (backport #18662)
2025-04-16tests: add nb test binary to .gitignoreMark Stapp
Add a northbound unit-test binary product to .gitignore Signed-off-by: Mark Stapp <mjs@cisco.com> (cherry picked from commit b256f2f1e90fd40ffb791d79dbdcaf71c3ab1a04)
2025-04-16tests: use little-endian order for libyang apiMark Stapp
Use the expected - little-endian - byte-order for a param to one of the libyang apis; tests fail on LE architectures otherwise. Signed-off-by: Mark Stapp <mjs@cisco.com> (cherry picked from commit da8fce3830baa2fd4d70050a3f7677d6e8bd5ad4)
2025-04-13Merge pull request #18650 from FRRouting/mergify/bp/stable/10.3/pr-18649Donald Sharp
Rpki testing and bug fix (backport #18649)
2025-04-13tests: Add more tests to bgp_rpki_topo1 testDonald Sharp
Looking at the gcov of the rpki code, I noticed that there was some functionality that is not covered in our test suites. Add the functionality. Signed-off-by: Donald Sharp <donaldsharp72@gmail.com> (cherry picked from commit dbff585b411edba20fc73b5e509ef9c1bc0697b2)
2025-04-13bgpd: Prevent crash when issuing a show rpki connectionsDonald Sharp
When attempting to check rpki status and the connection has been turned off, let's check to see if we are connected before we ask the rpki subsystem, else we will get a crash in the rpki library. Signed-off-by: Donald Sharp <donaldsharp72@gmail.com> (cherry picked from commit dcf43ae009ffecf206fb8cf8896eb5cd616ba4e5)
2025-04-12Merge pull request #18646 from FRRouting/mergify/bp/stable/10.3/pr-18628Carmine Scarpitta
tools: fix reload script for SRv6 locators and formats (backport #18628)
2025-04-11tools: fix reload script for SRv6 locators and formatsRajasekar Raja
Current code implementation does not have a "no" form of handling for the following commands under segment-routing srv6 - no formats - no locators - no prefix <> under locator XYZ Fix the handling of segment-routing srv6 locators and formats commands - Ignore "no formats" and "no locators" command - replace "no prefix" under locator XYZ as "no locator XYZ" as prefix is a mandatory property of locator Signed-off-by: Rajasekar Raja <rajasekarr@nvidia.com> (cherry picked from commit ce06d35fa983c67d25d93f5c2533610490313699)
2025-04-10Merge pull request #18636 from FRRouting/mergify/bp/stable/10.3/pr-18548Jafar Al-Gharaibeh
babeld: fix incorrect type assignment in parse_request_subtlv (backport #18548)
2025-04-10babeld: fix incorrect type assignment in parse_request_subtlvzmw12306
parse_request_subtlv accesses type using fixed offset instead of current position. Signed-off-by: zmw12306 <zmw12306@gmail.com> (cherry picked from commit 1571607c6bcba8f6861bdb90f472fe534df78a80)
2025-04-08Merge pull request #18618 from FRRouting/mergify/bp/stable/10.3/pr-18583Jafar Al-Gharaibeh
babeld: check valid babel port (backport #18583)
2025-04-08Merge pull request #18612 from FRRouting/mergify/bp/stable/10.3/pr-18526Jafar Al-Gharaibeh
pimd: Fix memory leak on shutdown (backport #18526)
2025-04-08Merge pull request #18616 from FRRouting/mergify/bp/stable/10.3/pr-18598Jafar Al-Gharaibeh
nhrpd: Add Hop Count Validation Before Forwarding in nhrp_peer_recv() (backport #18598)
2025-04-09Merge pull request #18619 from FRRouting/mergify/bp/stable/10.3/pr-18614Donatas Abraitis
bgpd: On shutdown free up memory leak found by topotest (backport #18614)
2025-04-08bgpd: On shutdown free up memory leak found by topotestDonald Sharp
This commit fixes two types of problems: a) Avoidance of cleaning up memory when a instance is hidden, thus causing it never to be freed on shutdown b) In some instances bgp_create is called 2 times for some code. We are double allocating memory and dropping it on the second allocation. Signed-off-by: Donald Sharp <sharpd@nvidia.com> (cherry picked from commit b18c309015f3b91bc65af2f64ed3e6f845b855eb)
2025-04-08babeld: check valid babel portzmw12306
Add checking for port == 6696. Signed-off-by: zmw12306 <zmw12306@gmail.com> (cherry picked from commit 6f88868f325f0269198fb8e0c00b40a7bbe0e53f)
2025-04-08nhrpd: Add Hop Count Validation Before Forwarding in nhrp_peer_recv()zmw12306
According to [RFC 2332, Section 5.1], if an NHS receives a packet that it would normally forward and the hop count is zero, it must send an error indication back to the source and drop the packet. Signed-off-by: zmw12306 <zmw12306@gmail.com> (cherry picked from commit 7c877164820bfc1cd7f84b3f7529f716b0f897cd)
2025-04-08Merge pull request #18609 from ↵Jafar Al-Gharaibeh
cscarpitta/fix/backport_check_sid_loc_block_beforehand staticd: Avoid requesting SRv6 sid from zebra when loc and sid block don't match (manual backport #18580)
2025-04-08pimd: Fix memory leak on shutdownDonald Sharp
The gm_join_list has a setup where it attempts to only create the list upon need and deletes it when the list is empty. On interface shutdown it was calling the function to empty the list but it was not empty so the list was being left at the end. Just add a bit of code to really clean up the list in the shutdown case. Direct leak of 40 byte(s) in 1 object(s) allocated from: 0 0x7f84850b83b7 in __interceptor_calloc ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:77 1 0x7f8484c391c4 in qcalloc lib/memory.c:106 2 0x7f8484c1ad36 in list_new lib/linklist.c:49 3 0x55d982827252 in pim_if_gm_join_add pimd/pim_iface.c:1354 4 0x55d982852b59 in lib_interface_gmp_address_family_join_group_create pimd/pim_nb_config.c:4499 5 0x7f8484c6a5d3 in nb_callback_create lib/northbound.c:1512 6 0x7f8484c6a5d3 in nb_callback_configuration lib/northbound.c:1910 7 0x7f8484c6bb51 in nb_transaction_process lib/northbound.c:2042 8 0x7f8484c6c164 in nb_candidate_commit_apply lib/northbound.c:1381 9 0x7f8484c6c39f in nb_candidate_commit lib/northbound.c:1414 10 0x7f8484c6cf1c in nb_cli_classic_commit lib/northbound_cli.c:57 11 0x7f8484c72f67 in nb_cli_apply_changes_internal lib/northbound_cli.c:195 12 0x7f8484c73a2e in nb_cli_apply_changes lib/northbound_cli.c:251 13 0x55d9828bd30f in interface_ip_igmp_join_magic pimd/pim_cmd.c:5436 14 0x55d9828bd30f in interface_ip_igmp_join pimd/pim_cmd_clippy.c:6366 15 0x7f8484bb5cbd in cmd_execute_command_real lib/command.c:1003 16 0x7f8484bb5fdc in cmd_execute_command lib/command.c:1062 17 0x7f8484bb6508 in cmd_execute lib/command.c:1228 18 0x7f8484cfb6ec in vty_command lib/vty.c:626 19 0x7f8484cfbc3f in vty_execute lib/vty.c:1389 20 0x7f8484cff9f0 in vtysh_read lib/vty.c:2408 21 0x7f8484cec846 in event_call lib/event.c:1984 22 0x7f8484c1a10a in frr_run lib/libfrr.c:1246 23 0x55d9828fc765 in main pimd/pim_main.c:166 24 0x7f848470c249 in __libc_start_call_main ../sysdeps/nptl/libc_start_call_main.h: Signed-off-by: Donald Sharp <sharpd@nvidia.com> (cherry picked from commit 521b58945ca5c8c75ccdb1f7f1cb9e2eb10ab83f)
2025-04-08staticd: Avoid requesting SRv6 sid from zebra when loc and sid block dont matchCarmine Scarpitta
Currently, when the locator block and sid block differs, staticd would still go ahead and request zebra to allocate the SID which it does if there is atleast one match (from any locators). Only when staticd tries to install the route, it sees that the locator block and sid block are different and avoids installing the route. Fix: Check if the locator block and sid block match before even requesting Zebra to allocate one. Signed-off-by: Rajasekar Raja <rajasekarr@nvidia.com> (cherry picked from commit dbd9fed0b30dd1d3475686f71f87d326eeafd26c)
2025-04-04Merge pull request #18576 from FRRouting/mergify/bp/stable/10.3/pr-18567Donatas Abraitis
pimd: Initialize gm proxy to false (backport #18567)
2025-04-03pimd: Initialize gm proxy to falseNathan Bahr
Signed-off-by: Nathan Bahr <nbahr@atcorp.com> (cherry picked from commit 153d9ea3b99d07015191924866ae14bb0525365a)
2025-04-03Merge pull request #18573 from FRRouting/mergify/bp/stable/10.3/pr-18572Jafar Al-Gharaibeh
tests: Fix typo when configuring delayopen timer (backport #18572)
2025-04-03tests: Fix typo when configuring delayopen timerDonatas Abraitis
`"` was accidentally added, and random tests failures happening. Fixes: a4f61b78dd382c438ff4fec2fda7450ecc890edf ("tests: Check if routes are marked as stale and retained with N-bit for GR") Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org> (cherry picked from commit 55d88ee3de422ff6dc206c6ebe5ba96b3ff67967)
2025-04-03Merge pull request #18569 from FRRouting/mergify/bp/stable/10.3/pr-18494Donatas Abraitis
lib: Return duplicate prefix-list entry test (backport #18494)
2025-04-03Merge pull request #18565 from FRRouting/mergify/bp/stable/10.3/pr-18561Donatas Abraitis
lib: Return duplicate ipv6 prefix-list entry test (backport #18561)
2025-04-03lib: Return duplicate prefix-list entry testDonatas Abraitis
If we do e.g.: ip prefix-list PL_LoopbackV4 permit 10.1.0.32/32 ip prefix-list PL_LoopbackV4 permit 10.1.0.32/32 ip prefix-list PL_LoopbackV4 permit 10.1.0.32/32 We end up, having duplicate records with a different sequence number only. Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org> (cherry picked from commit 8384d41144496019725c1e250abd0ceea854341f)
2025-04-02lib: Return duplicate ipv6 prefix-list entry testDonatas Abraitis
Fixes: 8384d41144496019725c1e250abd0ceea854341f ("lib: Return duplicate prefix-list entry test") Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org> (cherry picked from commit 24ae7cd30a055dc17fc9d75762320e1359e005b2)
2025-04-01Merge pull request #18527 from FRRouting/mergify/bp/stable/10.3/pr-18502Russ White
bgpd: Set the label for MP_UNREACH_NLRI 0x800000 instead of 0x000000 (backport #18502)
2025-04-01Merge pull request #18511 from FRRouting/mergify/bp/stable/10.3/pr-18498Russ White
bgpd: Retain the routes if we do a clear with N-bit set for Graceful-Restart (backport #18498)
2025-03-30Merge pull request #18509 from FRRouting/mergify/bp/stable/10.3/pr-18503Donatas Abraitis
ospf6d: Fix LSA memory leaks related to graceful restart (backport #18503)
2025-03-28tests: Use label 0x800000 instead of 0x000000 for BMP testsmergify/bp/stable/10.3/pr-18502Donatas Abraitis
Related-to: 94e2aadf7187d7d695babce21033b5bc8e454f25 ("bgpd: Set the label for MP_UNREACH_NLRI 0x800000 instead of 0x000000") Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org> (cherry picked from commit e69459c7144568381c05bbaf962adecd914975d5)
2025-03-28bgpd: Set the label for MP_UNREACH_NLRI 0x800000 instead of 0x000000Donatas Abraitis
RFC8277 says: The procedures in [RFC3107] for withdrawing the binding of a label or sequence of labels to a prefix are not specified clearly and correctly. => How to Explicitly Withdraw the Binding of a Label to a Prefix Suppose a BGP speaker has announced, on a given BGP session, the binding of a given label or sequence of labels to a given prefix. Suppose it now wishes to withdraw that binding. To do so, it may send a BGP UPDATE message with an MP_UNREACH_NLRI attribute. The NLRI field of this attribute is encoded as follows: 0 1 2 3 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | Length | Compatibility | +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ | Prefix ~ ~ | +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+ Figure 4: NLRI for Withdrawal Upon transmission, the Compatibility field SHOULD be set to 0x800000. Upon reception, the value of the Compatibility field MUST be ignored. [RFC3107] also made it possible to withdraw a binding without specifying the label explicitly, by setting the Compatibility field to 0x800000. However, some implementations set it to 0x000000. In order to ensure backwards compatibility, it is RECOMMENDED by this document that the Compatibility field be set to 0x800000, but it is REQUIRED that it be ignored upon reception. In FRR case where a single label is used per-prefix, we should send 0x800000, and not 0x000000. Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org> (cherry picked from commit 94e2aadf7187d7d695babce21033b5bc8e454f25)
2025-03-28bgpd: Remove unused defines from bgp_label.hDonatas Abraitis
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org> (cherry picked from commit 42b9d985ccced6ee4ccc70b139c78cd47867b2ba)
2025-03-28tests: Fix wait times in test_ospf6_gr_topo1 topotestMartin Buck
Increase wait times to at least the minimum wait time accepted by topotest.run_and_expect(). Also change poll interval to 1s, no point in doings this more frequently. Finally, slightly improve the topology diagram to also include area numbers. Signed-off-by: Martin Buck <mb-tmp-tvguho.pbz@gromit.dyndns.org> (cherry picked from commit b73e3ae69dfd2050dc061b81040a4748d8992ddd)
2025-03-28ospf6d: Fix LSA memory leaks related to graceful restartMartin Buck
Fixes leaks reported by ospf6_gr_topo1 topotest. Signed-off-by: Martin Buck <mb-tmp-tvguho.pbz@gromit.dyndns.org> (cherry picked from commit 0db0e7fbd7c6d6c5ba32c741385bc1b2d1acfa77)
2025-03-28tests: Check if routes are marked as stale and retained with N-bit for GRmergify/bp/stable/10.3/pr-18498Donatas Abraitis
Related-to: b7c657d4e065f310fcf6454abae1a963c208c3b8 ("bgpd: Retain the routes if we do a clear with N-bit set for Graceful-Restart") Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org> (cherry picked from commit a4f61b78dd382c438ff4fec2fda7450ecc890edf)
2025-03-28bgpd: Retain the routes if we do a clear with N-bit set for Graceful-RestartDonatas Abraitis
On receiving side we already did the job correctly, but the peer which initiates the clear does not retain the other's routes. This commit fixes that. Fixes: 20170775da3a3c5d41aba714d0c1d5a29b0da61c ("bgpd: Activate Graceful-Restart when receiving CEASE/HOLDTIME notifications") Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org> (cherry picked from commit b7c657d4e065f310fcf6454abae1a963c208c3b8)
2025-03-27Merge pull request #18522 from opensourcerouting/fix/revert_18478Mark Stapp
Revert backport for crash when modifying SRv6 SID for stable/10.3 branch
2025-03-27Revert "staticd: Fix crash that occurs when modifying an SRv6 SID"Donatas Abraitis
This reverts commit c80c2b2a4226640464a38f8ee9f8b2b4be8bd094.
2025-03-27Revert "tests: Add test case to verify SRv6 SID modify"Donatas Abraitis
This reverts commit 8f1313392f70ff2e7e8f924e238564879d5e3aeb.
2025-03-25Merge pull request #18478 from FRRouting/mergify/bp/stable/10.3/pr-18467Russ White
staticd: Fix a crash that occurs when modifying an SRv6 SID (backport #18467)
2025-03-25Merge pull request #18485 from FRRouting/mergify/bp/stable/10.3/pr-18483Donald Sharp
bgpd: Fix holdtime not working properly when busy (backport #18483)
2025-03-25bgpd: Fix holdtime not working properly when busymergify/bp/stable/10.3/pr-18483Donald Sharp
Commit: cc9f21da2218d95567eff1501482ce58e6600f54 Modified the bgp_fsm code to dissallow the extension of the hold time when the system is under extremely heavy load. This was a attempt to remove the return code but it was too aggressive and messed up this bit of code. Put the behavior back that was introduced in: d0874d195d0127009a7d9c06920c52c95319eff9 Signed-off-by: Donald Sharp <sharpd@nvidia.com> (cherry picked from commit 9a26a56c5188fd1c95e244932bc17f97b9051935)
2025-03-24tests: Add test case to verify SRv6 SID modifyCarmine Scarpitta
This commit adds a test case that modifies a SID and verifies that the RIB is as expected. Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com> (cherry picked from commit 23403e01a3ece0b206bc8acf36c08b4bb538cb79)
2025-03-24staticd: Fix crash that occurs when modifying an SRv6 SIDCarmine Scarpitta
When the user modifies an SRv6 SID and then removes all SIDs, staticd crashes: ``` 2025/03/23 08:37:22.691860 STATIC: lib/memory.c:74: mt_count_free(): assertion (mt->n_alloc) failed STATIC: Received signal 6 at 1742715442 (si_addr 0x8200007cf0); aborting... STATIC: zlog_signal+0x390 fcc704a844b8 ffffd7450390 /usr/lib/frr/libfrr.so.0 (mapped at 0xfcc704800000) STATIC: core_handler+0x1f8 fcc704b79990 ffffd7450590 /usr/lib/frr/libfrr.so.0 (mapped at 0xfcc704800000) STATIC: ---- signal ---- STATIC: ? fcc705c008f8 ffffd74507a0 linux-vdso.so.1 (mapped at 0xfcc705c00000) STATIC: pthread_key_delete+0x1a0 fcc70458f1f0 ffffd7451a00 /lib/aarch64-linux-gnu/libc.so.6 (mapped at 0xfcc704510000) STATIC: raise+0x1c fcc70454a67c ffffd7451ad0 /lib/aarch64-linux-gnu/libc.so.6 (mapped at 0xfcc704510000) STATIC: abort+0xe4 fcc704537130 ffffd7451af0 /lib/aarch64-linux-gnu/libc.so.6 (mapped at 0xfcc704510000) STATIC: _zlog_assert_failed+0x3c4 fcc704c407c8 ffffd7451c40 /usr/lib/frr/libfrr.so.0 (mapped at 0xfcc704800000) STATIC: mt_count_free+0x12c fcc704a93c74 ffffd7451dc0 /usr/lib/frr/libfrr.so.0 (mapped at 0xfcc704800000) STATIC: qfree+0x28 fcc704a93fa0 ffffd7451e70 /usr/lib/frr/libfrr.so.0 (mapped at 0xfcc704800000) STATIC: static_srv6_sid_free+0x1c adc1df8fa544 ffffd7451e90 /usr/lib/frr/staticd (mapped at 0xadc1df8a0000) STATIC: delete_static_srv6_sid+0x14 adc1df8faafc ffffd7451eb0 /usr/lib/frr/staticd (mapped at 0xadc1df8a0000) STATIC: list_delete_all_node+0x104 fcc704a60eec ffffd7451ed0 /usr/lib/frr/libfrr.so.0 (mapped at 0xfcc704800000) STATIC: list_delete+0x8c fcc704a61054 ffffd7451f00 /usr/lib/frr/libfrr.so.0 (mapped at 0xfcc704800000) STATIC: static_srv6_cleanup+0x20 adc1df8fabdc ffffd7451f20 /usr/lib/frr/staticd (mapped at 0xadc1df8a0000) STATIC: sigint+0x40 adc1df8be544 ffffd7451f30 /usr/lib/frr/staticd (mapped at 0xadc1df8a0000) STATIC: frr_sigevent_process+0x148 fcc704b79460 ffffd7451f40 /usr/lib/frr/libfrr.so.0 (mapped at 0xfcc704800000) STATIC: event_fetch+0x1c4 fcc704bc0834 ffffd7451f60 /usr/lib/frr/libfrr.so.0 (mapped at 0xfcc704800000) STATIC: frr_run+0x650 fcc704a5d230 ffffd7452080 /usr/lib/frr/libfrr.so.0 (mapped at 0xfcc704800000) STATIC: main+0x1d0 adc1df8be75c ffffd7452270 /usr/lib/frr/staticd (mapped at 0xadc1df8a0000) STATIC: __libc_init_first+0x7c fcc7045373fc ffffd74522b0 /lib/aarch64-linux-gnu/libc.so.6 (mapped at 0xfcc704510000) STATIC: __libc_start_main+0x98 fcc7045374cc ffffd74523c0 /lib/aarch64-linux-gnu/libc.so.6 (mapped at 0xfcc704510000) STATIC: _start+0x30 adc1df8be0f0 ffffd7452420 /usr/lib/frr/staticd (mapped at 0xadc1df8a0000) ``` Tracking this down, the crash occurs because every time we modify a SID, staticd executes some callbacks to modify the SID and finally it calls `apply_finish`, which re-adds the SID to the list `srv6_sids`. This leads to having the same SID multiple times in the `srv6_sids` list. When we delete all SIDs, staticd attempts to deallocate the same SID multiple times, which leads to the crash. This commit fixes the issue by moving the code that adds the SID to the list from the `apply_finish` callback to the `create` callback. This ensures that the SID is inserted into the list only once, when it is created. For all subsequent modifications, the SID is modified but not added to the list. Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com> (cherry picked from commit 6037ea350c98fbce60d0a287720cd4e60f7a21ec)
2025-03-23Merge pull request #18463 from FRRouting/mergify/bp/stable/10.3/pr-18378Donald Sharp
bgpd: fix `set evpn gateway-ip ipv[46]` route-map (backport #18378)
2025-03-23bgpd: fix evpn attributes being dropped on inputmergify/bp/stable/10.3/pr-18378Tuetuopay
All assignments of the EVPN attributes (ESI and Gateway IP) are gated behind the peer being set up for inbound soft-reconfiguration. There are no actual reasons for this limitation, so let's perform the EVPN attribute assignment no matter what when soft reconfiguration is not enabled. Fixes: 6e076ba5231 ("bgpd: Fix for ain->attr corruption during path update") Signed-off-by: Tuetuopay <tuetuopay@me.com> (cherry picked from commit 7320659f78cbe86dd983d5101831120fc14583d7)