bgpd: Call bgp_dest_unlock_node() inside bgp_adj_in_remove()

[ton31337]

No description provided.

[LabN-CI]

🛑 Basic BGPD CI results: FAILURE

Results table

_	_
Result	FAILURE git merge/8836 6a840fd frr.github Build
Date	06/11/2021
Start	04:11:03
Finish	04:12:18
Run-Time	01:15
Total
Pass
Fail
Valgrind-Errors
Valgrind-Loss
Details	vncregress-2021-06-11-04:11:03.txt
Log	make-2021-06-11-04:11:03.out.bz2
Memory

For details, please contact louberger

[donaldsharp]

this looks correct to me.

[NetDEF-CI]

Continuous Integration Result: SUCCESSFUL

Congratulations, this patch passed basic tests

Tested-by: NetDEF / OpenSourceRouting.org CI System

CI System Testrun URL: https://ci1.netdef.org/browse/FRR-FRRPULLREQ-19552/

This is a comment from an automated CI system.
For questions and feedback in regards to this CI system, please feel free to email
Martin Winter - mwinter (at) opensourcerouting.org.

[rwestphal]

Looks good to me. Since bgp_adj_in_set() does the node locking, it makes sense to do the unlocking on bgp_adj_in_remove().

[mwinter-osr]

@Mergifyio backport stable/8.0

[mergify]

Command backport stable/8.0: success

Backports have been created

• #9168 bgpd: Call bgp_dest_unlock_node() inside bgp_adj_in_remove() (backport #8836) has been created for branch stable/8.0