From 1d248b67f3e344da4b2c117bb67fd2121b53320c Mon Sep 17 00:00:00 2001 From: Donatas Abraitis Date: Mon, 7 Feb 2022 10:50:33 +0200 Subject: [PATCH] bgpd: Flush temporary attributes after route-map apply Fixes crash: ``` Received signal 6 at 1644222173 (si_addr 0x6f000012d2, PC 0x7f79274cbfb7); aborting... /usr/lib/x86_64-linux-gnu/frr/libfrr.so.0(zlog_backtrace_sigsafe+0x6d) [0x7f79282e079d] /usr/lib/x86_64-linux-gnu/frr/libfrr.so.0(zlog_signal+0xf3) [0x7f79282e0993] /usr/lib/x86_64-linux-gnu/frr/libfrr.so.0(+0xbd881) [0x7f792830b881] /lib/x86_64-linux-gnu/libpthread.so.0(+0x12980) [0x7f7927890980] /lib/x86_64-linux-gnu/libc.so.6(gsignal+0xc7) [0x7f79274cbfb7] /lib/x86_64-linux-gnu/libc.so.6(abort+0x141) [0x7f79274cd921] /lib/x86_64-linux-gnu/libc.so.6(+0x89967) [0x7f7927516967] /lib/x86_64-linux-gnu/libc.so.6(+0x909da) [0x7f792751d9da] /lib/x86_64-linux-gnu/libc.so.6(cfree+0x620) [0x7f7927525050] /usr/lib/frr/bgpd(+0x1e7c1c) [0x55b5c53b1c1c] /usr/lib/frr/bgpd(aspath_free+0x28) [0x55b5c53b1c78] /usr/lib/frr/bgpd(bgp_attr_flush+0x245) [0x55b5c52abc15] /usr/lib/frr/bgpd(subgroup_announce_check+0x1137) [0x55b5c530c5d7] ``` Signed-off-by: Donatas Abraitis (cherry picked from commit 686826e90e5d8a96e6b4b93cef77aa595fd83ef0) --- bgpd/bgp_route.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/bgpd/bgp_route.c b/bgpd/bgp_route.c index d4a63e6369..3f340a9acf 100644 --- a/bgpd/bgp_route.c +++ b/bgpd/bgp_route.c @@ -2116,7 +2116,7 @@ bool subgroup_announce_check(struct bgp_dest *dest, struct bgp_path_info *pi, "%s [Update:SEND] %pFX is filtered by route-map", peer->host, p); - bgp_attr_flush(attr); + bgp_attr_flush(&dummy_attr); return false; } } -- 2.39.5