matthieu/frr
1
0
Fork 0
forked from Mirror/frr
Code Pull requests Activity

bgpd: MTYPE_BGP was being overused split up

Browse source 
The MTYPE_BGP memory type was being over used as
both the handler for the bgp instance itself as
well as memory associated with name strings.
Let's separate out the two.

Signed-off-by: Donald Sharp <sharpd@nvidia.com>
This commit is contained in:
Donald Sharp 2023-11-15 14:52:10 -05:00
parent 318a8c98b4
commit 0b81a7524d
8 changed files with 47 additions and 43 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
bgpd/bgp_evpn.c
View file

@ -6097,7 +6097,7 @@ void bgp_evpn_derive_auto_rd(struct bgp *bgp, struct bgpevpn *vpn)
snprintfrr(buf, sizeof(buf), "%pI4:%hu", &bgp->router_id, vpn->rd_id); snprintfrr(buf, sizeof(buf), "%pI4:%hu", &bgp->router_id, vpn->rd_id);
(void)str2prefix_rd(buf, &vpn->prd); (void)str2prefix_rd(buf, &vpn->prd);
if (vpn->prd_pretty) if (vpn->prd_pretty)
XFREE(MTYPE_BGP, vpn->prd_pretty); XFREE(MTYPE_BGP_NAME, vpn->prd_pretty);
UNSET_FLAG(vpn->flags, VNI_FLAG_RD_CFGD); UNSET_FLAG(vpn->flags, VNI_FLAG_RD_CFGD);
} }
@ -6203,7 +6203,7 @@ void bgp_evpn_free(struct bgp *bgp, struct bgpevpn *vpn)
hash_release(bgp->vni_svi_hash, vpn); hash_release(bgp->vni_svi_hash, vpn);
hash_release(bgp->vnihash, vpn); hash_release(bgp->vnihash, vpn);
if (vpn->prd_pretty) if (vpn->prd_pretty)
XFREE(MTYPE_BGP, vpn->prd_pretty); XFREE(MTYPE_BGP_NAME, vpn->prd_pretty);
QOBJ_UNREG(vpn); QOBJ_UNREG(vpn);
XFREE(MTYPE_BGP_EVPN, vpn); XFREE(MTYPE_BGP_EVPN, vpn);
} }
@ -7080,7 +7080,7 @@ void bgp_evpn_cleanup(struct bgp *bgp)
} }
if (bgp->vrf_prd_pretty) if (bgp->vrf_prd_pretty)
XFREE(MTYPE_BGP, bgp->vrf_prd_pretty); XFREE(MTYPE_BGP_NAME, bgp->vrf_prd_pretty);
} }
/* /*

8
bgpd/bgp_evpn_vty.c
View file

@ -2294,11 +2294,11 @@ static void evpn_configure_vrf_rd(struct bgp *bgp_vrf, struct prefix_rd *rd,
bgp_evpn_handle_vrf_rd_change(bgp_vrf, 1); bgp_evpn_handle_vrf_rd_change(bgp_vrf, 1);
if (bgp_vrf->vrf_prd_pretty) if (bgp_vrf->vrf_prd_pretty)
XFREE(MTYPE_BGP, bgp_vrf->vrf_prd_pretty); XFREE(MTYPE_BGP_NAME, bgp_vrf->vrf_prd_pretty);
/* update RD */ /* update RD */
memcpy(&bgp_vrf->vrf_prd, rd, sizeof(struct prefix_rd)); memcpy(&bgp_vrf->vrf_prd, rd, sizeof(struct prefix_rd));
bgp_vrf->vrf_prd_pretty = XSTRDUP(MTYPE_BGP, rd_pretty); bgp_vrf->vrf_prd_pretty = XSTRDUP(MTYPE_BGP_NAME, rd_pretty);
SET_FLAG(bgp_vrf->vrf_flags, BGP_VRF_RD_CFGD); SET_FLAG(bgp_vrf->vrf_flags, BGP_VRF_RD_CFGD);
/* We have a new RD for VRF. /* We have a new RD for VRF.
@ -2321,7 +2321,7 @@ static void evpn_unconfigure_vrf_rd(struct bgp *bgp_vrf)
bgp_evpn_derive_auto_rd_for_vrf(bgp_vrf); bgp_evpn_derive_auto_rd_for_vrf(bgp_vrf);
UNSET_FLAG(bgp_vrf->vrf_flags, BGP_VRF_RD_CFGD); UNSET_FLAG(bgp_vrf->vrf_flags, BGP_VRF_RD_CFGD);
if (bgp_vrf->vrf_prd_pretty) if (bgp_vrf->vrf_prd_pretty)
XFREE(MTYPE_BGP, bgp_vrf->vrf_prd_pretty); XFREE(MTYPE_BGP_NAME, bgp_vrf->vrf_prd_pretty);
/* We have a new RD for VRF. /* We have a new RD for VRF.
* Advertise all type-5 routes again with the new RD * Advertise all type-5 routes again with the new RD
*/ */
@ -2343,7 +2343,7 @@ static void evpn_configure_rd(struct bgp *bgp, struct bgpevpn *vpn,
/* update RD */ /* update RD */
memcpy(&vpn->prd, rd, sizeof(struct prefix_rd)); memcpy(&vpn->prd, rd, sizeof(struct prefix_rd));
vpn->prd_pretty = XSTRDUP(MTYPE_BGP, rd_pretty); vpn->prd_pretty = XSTRDUP(MTYPE_BGP_NAME, rd_pretty);
SET_FLAG(vpn->flags, VNI_FLAG_RD_CFGD); SET_FLAG(vpn->flags, VNI_FLAG_RD_CFGD);
if (is_vni_live(vpn)) if (is_vni_live(vpn))

1
bgpd/bgp_memory.c
View file

@ -15,6 +15,7 @@
DEFINE_MGROUP(BGPD, "bgpd"); DEFINE_MGROUP(BGPD, "bgpd");
DEFINE_MTYPE(BGPD, BGP, "BGP instance"); DEFINE_MTYPE(BGPD, BGP, "BGP instance");
DEFINE_MTYPE(BGPD, BGP_NAME, "BGP Name data");
DEFINE_MTYPE(BGPD, BGP_LISTENER, "BGP listen socket details"); DEFINE_MTYPE(BGPD, BGP_LISTENER, "BGP listen socket details");
DEFINE_MTYPE(BGPD, BGP_PEER, "BGP peer"); DEFINE_MTYPE(BGPD, BGP_PEER, "BGP peer");
DEFINE_MTYPE(BGPD, BGP_PEER_CONNECTION, "BGP peer connection"); DEFINE_MTYPE(BGPD, BGP_PEER_CONNECTION, "BGP peer connection");

1
bgpd/bgp_memory.h
View file

@ -11,6 +11,7 @@
DECLARE_MGROUP(BGPD); DECLARE_MGROUP(BGPD);
DECLARE_MTYPE(BGP); DECLARE_MTYPE(BGP);
DECLARE_MTYPE(BGP_NAME);
DECLARE_MTYPE(BGP_LISTENER); DECLARE_MTYPE(BGP_LISTENER);
DECLARE_MTYPE(BGP_PEER); DECLARE_MTYPE(BGP_PEER);
DECLARE_MTYPE(BGP_PEER_CONNECTION); DECLARE_MTYPE(BGP_PEER_CONNECTION);

6
bgpd/bgp_mplsvpn_snmp.c
View file

@ -511,8 +511,8 @@ static int bgp_init_snmp_stats(struct bgp *bgp)
{ {
if (is_bgp_vrf_mplsvpn(bgp)) { if (is_bgp_vrf_mplsvpn(bgp)) {
if (bgp->snmp_stats == NULL) { if (bgp->snmp_stats == NULL) {
bgp->snmp_stats = XCALLOC( bgp->snmp_stats = XCALLOC(MTYPE_BGP_NAME,
MTYPE_BGP, sizeof(struct bgp_snmp_stats)); sizeof(struct bgp_snmp_stats));
/* fix up added routes */ /* fix up added routes */
if (bgp->snmp_stats) { if (bgp->snmp_stats) {
bgp->snmp_stats->routes_added = bgp->snmp_stats->routes_added =
@ -523,7 +523,7 @@ static int bgp_init_snmp_stats(struct bgp *bgp)
} }
} else { } else {
if (bgp->snmp_stats) { if (bgp->snmp_stats) {
XFREE(MTYPE_BGP, bgp->snmp_stats); XFREE(MTYPE_BGP_NAME, bgp->snmp_stats);
bgp->snmp_stats = NULL; bgp->snmp_stats = NULL;
} }
} }

2
bgpd/bgp_route.c
View file

@ -6301,7 +6301,7 @@ static void bgp_static_free(struct bgp_static *bgp_static)
route_map_counter_decrement(bgp_static->rmap.map); route_map_counter_decrement(bgp_static->rmap.map);
if (bgp_static->prd_pretty) if (bgp_static->prd_pretty)
XFREE(MTYPE_BGP, bgp_static->prd_pretty); XFREE(MTYPE_BGP_NAME, bgp_static->prd_pretty);
XFREE(MTYPE_ATTR, bgp_static->eth_s_id); XFREE(MTYPE_ATTR, bgp_static->eth_s_id);
XFREE(MTYPE_BGP_STATIC, bgp_static); XFREE(MTYPE_BGP_STATIC, bgp_static);
} }

11
bgpd/bgp_vty.c
View file

@ -1602,8 +1602,9 @@ DEFUN_NOSH (router_bgp,
* - update asnotation if explicitly mentioned * - update asnotation if explicitly mentioned
*/ */
if (CHECK_FLAG(bgp->vrf_flags, BGP_VRF_AUTO)) { if (CHECK_FLAG(bgp->vrf_flags, BGP_VRF_AUTO)) {
XFREE(MTYPE_BGP, bgp->as_pretty); XFREE(MTYPE_BGP_NAME, bgp->as_pretty);
bgp->as_pretty = XSTRDUP(MTYPE_BGP, argv[idx_asn]->arg); bgp->as_pretty = XSTRDUP(MTYPE_BGP_NAME,
argv[idx_asn]->arg);
if (!CHECK_FLAG(bgp->config, BGP_CONFIG_ASNOTATION) && if (!CHECK_FLAG(bgp->config, BGP_CONFIG_ASNOTATION) &&
asnotation != ASNOTATION_UNDEFINED) { asnotation != ASNOTATION_UNDEFINED) {
SET_FLAG(bgp->config, BGP_CONFIG_ASNOTATION); SET_FLAG(bgp->config, BGP_CONFIG_ASNOTATION);
@ -9392,13 +9393,13 @@ DEFPY (af_rd_vpn_export,
bgp_get_default(), bgp); bgp_get_default(), bgp);
if (yes) { if (yes) {
bgp->vpn_policy[afi].tovpn_rd_pretty = bgp->vpn_policy[afi].tovpn_rd_pretty = XSTRDUP(MTYPE_BGP_NAME,
XSTRDUP(MTYPE_BGP, rd_str); rd_str);
bgp->vpn_policy[afi].tovpn_rd = prd; bgp->vpn_policy[afi].tovpn_rd = prd;
SET_FLAG(bgp->vpn_policy[afi].flags, SET_FLAG(bgp->vpn_policy[afi].flags,
BGP_VPN_POLICY_TOVPN_RD_SET); BGP_VPN_POLICY_TOVPN_RD_SET);
} else { } else {
XFREE(MTYPE_BGP, bgp->vpn_policy[afi].tovpn_rd_pretty); XFREE(MTYPE_BGP_NAME, bgp->vpn_policy[afi].tovpn_rd_pretty);
UNSET_FLAG(bgp->vpn_policy[afi].flags, UNSET_FLAG(bgp->vpn_policy[afi].flags,
BGP_VPN_POLICY_TOVPN_RD_SET); BGP_VPN_POLICY_TOVPN_RD_SET);
} }

55
bgpd/bgpd.c
View file

@ -576,8 +576,8 @@ void bgp_confederation_id_set(struct bgp *bgp, as_t as, const char *as_str)
already_confed = bgp_config_check(bgp, BGP_CONFIG_CONFEDERATION); already_confed = bgp_config_check(bgp, BGP_CONFIG_CONFEDERATION);
bgp->confed_id = as; bgp->confed_id = as;
if (bgp->confed_id_pretty) if (bgp->confed_id_pretty)
XFREE(MTYPE_BGP, bgp->confed_id_pretty); XFREE(MTYPE_BGP_NAME, bgp->confed_id_pretty);
bgp->confed_id_pretty = XSTRDUP(MTYPE_BGP, as_str); bgp->confed_id_pretty = XSTRDUP(MTYPE_BGP_NAME, as_str);
bgp_config_set(bgp, BGP_CONFIG_CONFEDERATION); bgp_config_set(bgp, BGP_CONFIG_CONFEDERATION);
/* If we were doing confederation already, this is just an external /* If we were doing confederation already, this is just an external
@ -630,7 +630,7 @@ void bgp_confederation_id_unset(struct bgp *bgp)
struct listnode *node, *nnode; struct listnode *node, *nnode;
bgp->confed_id = 0; bgp->confed_id = 0;
XFREE(MTYPE_BGP, bgp->confed_id_pretty); XFREE(MTYPE_BGP_NAME, bgp->confed_id_pretty);
bgp_config_unset(bgp, BGP_CONFIG_CONFEDERATION); bgp_config_unset(bgp, BGP_CONFIG_CONFEDERATION);
for (ALL_LIST_ELEMENTS(bgp->peer, node, nnode, peer)) { for (ALL_LIST_ELEMENTS(bgp->peer, node, nnode, peer)) {
@ -684,7 +684,7 @@ void bgp_confederation_peers_add(struct bgp *bgp, as_t as, const char *as_str)
bgp->confed_peers[bgp->confed_peers_cnt].as = as; bgp->confed_peers[bgp->confed_peers_cnt].as = as;
bgp->confed_peers[bgp->confed_peers_cnt].as_pretty = bgp->confed_peers[bgp->confed_peers_cnt].as_pretty =
XSTRDUP(MTYPE_BGP, as_str); XSTRDUP(MTYPE_BGP_NAME, as_str);
bgp->confed_peers_cnt++; bgp->confed_peers_cnt++;
if (bgp_config_check(bgp, BGP_CONFIG_CONFEDERATION)) { if (bgp_config_check(bgp, BGP_CONFIG_CONFEDERATION)) {
@ -722,7 +722,7 @@ void bgp_confederation_peers_remove(struct bgp *bgp, as_t as)
for (i = 0; i < bgp->confed_peers_cnt; i++) for (i = 0; i < bgp->confed_peers_cnt; i++)
if (bgp->confed_peers[i].as == as) { if (bgp->confed_peers[i].as == as) {
XFREE(MTYPE_BGP, bgp->confed_peers[i].as_pretty); XFREE(MTYPE_BGP_NAME, bgp->confed_peers[i].as_pretty);
for (j = i + 1; j < bgp->confed_peers_cnt; j++) { for (j = i + 1; j < bgp->confed_peers_cnt; j++) {
bgp->confed_peers[j - 1].as = bgp->confed_peers[j - 1].as =
bgp->confed_peers[j].as; bgp->confed_peers[j].as;
@ -1265,9 +1265,9 @@ static void peer_free(struct peer *peer)
bgp_addpath_set_peer_type(peer, afi, safi, BGP_ADDPATH_NONE, 0); bgp_addpath_set_peer_type(peer, afi, safi, BGP_ADDPATH_NONE, 0);
if (peer->change_local_as_pretty) if (peer->change_local_as_pretty)
XFREE(MTYPE_BGP, peer->change_local_as_pretty); XFREE(MTYPE_BGP_NAME, peer->change_local_as_pretty);
if (peer->as_pretty) if (peer->as_pretty)
XFREE(MTYPE_BGP, peer->as_pretty); XFREE(MTYPE_BGP_NAME, peer->as_pretty);
bgp_peer_connection_free(&peer->connection); bgp_peer_connection_free(&peer->connection);
@ -1862,7 +1862,7 @@ struct peer *peer_create(union sockunion *su, const char *conf_if,
peer->as = remote_as; peer->as = remote_as;
/* internal and external values do not use as_pretty */ /* internal and external values do not use as_pretty */
if (as_str && asn_str2asn(as_str, NULL)) if (as_str && asn_str2asn(as_str, NULL))
peer->as_pretty = XSTRDUP(MTYPE_BGP, as_str); peer->as_pretty = XSTRDUP(MTYPE_BGP_NAME, as_str);
peer->as_type = as_type; peer->as_type = as_type;
peer->local_id = bgp->router_id; peer->local_id = bgp->router_id;
peer->v_holdtime = bgp->default_holdtime; peer->v_holdtime = bgp->default_holdtime;
@ -1987,10 +1987,10 @@ void peer_as_change(struct peer *peer, as_t as, int as_specified,
peer->as = as; peer->as = as;
if (as_specified == AS_SPECIFIED && as_str) { if (as_specified == AS_SPECIFIED && as_str) {
if (peer->as_pretty) if (peer->as_pretty)
XFREE(MTYPE_BGP, peer->as_pretty); XFREE(MTYPE_BGP_NAME, peer->as_pretty);
peer->as_pretty = XSTRDUP(MTYPE_BGP, as_str); peer->as_pretty = XSTRDUP(MTYPE_BGP_NAME, as_str);
} else if (peer->as_type == AS_UNSPECIFIED && peer->as_pretty) } else if (peer->as_type == AS_UNSPECIFIED && peer->as_pretty)
XFREE(MTYPE_BGP, peer->as_pretty); XFREE(MTYPE_BGP_NAME, peer->as_pretty);
peer->as_type = as_specified; peer->as_type = as_specified;
if (bgp_config_check(peer->bgp, BGP_CONFIG_CONFEDERATION) if (bgp_config_check(peer->bgp, BGP_CONFIG_CONFEDERATION)
@ -3290,9 +3290,9 @@ static struct bgp *bgp_create(as_t *as, const char *name,
bgp = XCALLOC(MTYPE_BGP, sizeof(struct bgp)); bgp = XCALLOC(MTYPE_BGP, sizeof(struct bgp));
bgp->as = *as; bgp->as = *as;
if (as_pretty) if (as_pretty)
bgp->as_pretty = XSTRDUP(MTYPE_BGP, as_pretty); bgp->as_pretty = XSTRDUP(MTYPE_BGP_NAME, as_pretty);
else else
bgp->as_pretty = XSTRDUP(MTYPE_BGP, asn_asn2asplain(*as)); bgp->as_pretty = XSTRDUP(MTYPE_BGP_NAME, asn_asn2asplain(*as));
if (asnotation != ASNOTATION_UNDEFINED) { if (asnotation != ASNOTATION_UNDEFINED) {
bgp->asnotation = asnotation; bgp->asnotation = asnotation;
@ -3421,14 +3421,14 @@ static struct bgp *bgp_create(as_t *as, const char *name,
bgp_mplsvpn_nh_label_bind_cache_init(&bgp->mplsvpn_nh_label_bind); bgp_mplsvpn_nh_label_bind_cache_init(&bgp->mplsvpn_nh_label_bind);
if (name) if (name)
bgp->name = XSTRDUP(MTYPE_BGP, name); bgp->name = XSTRDUP(MTYPE_BGP_NAME, name);
event_add_timer(bm->master, bgp_startup_timer_expire, bgp, event_add_timer(bm->master, bgp_startup_timer_expire, bgp,
bgp->restart_time, &bgp->t_startup); bgp->restart_time, &bgp->t_startup);
/* printable name we can use in debug messages */ /* printable name we can use in debug messages */
if (inst_type == BGP_INSTANCE_TYPE_DEFAULT) { if (inst_type == BGP_INSTANCE_TYPE_DEFAULT) {
bgp->name_pretty = XSTRDUP(MTYPE_BGP, "VRF default"); bgp->name_pretty = XSTRDUP(MTYPE_BGP_NAME, "VRF default");
} else { } else {
const char *n; const char *n;
int len; int len;
@ -3440,7 +3440,7 @@ static struct bgp *bgp_create(as_t *as, const char *name,
len = 4 + 1 + strlen(n) + 1; /* "view foo\0" */ len = 4 + 1 + strlen(n) + 1; /* "view foo\0" */
bgp->name_pretty = XCALLOC(MTYPE_BGP, len); bgp->name_pretty = XCALLOC(MTYPE_BGP_NAME, len);
snprintf(bgp->name_pretty, len, "%s %s", snprintf(bgp->name_pretty, len, "%s %s",
(bgp->inst_type == BGP_INSTANCE_TYPE_VRF) (bgp->inst_type == BGP_INSTANCE_TYPE_VRF)
? "VRF" ? "VRF"
@ -4043,7 +4043,8 @@ void bgp_free(struct bgp *bgp)
if (bgp->vpn_policy[afi].rtlist[dir]) if (bgp->vpn_policy[afi].rtlist[dir])
ecommunity_free(&bgp->vpn_policy[afi].rtlist[dir]); ecommunity_free(&bgp->vpn_policy[afi].rtlist[dir]);
if (bgp->vpn_policy[afi].tovpn_rd_pretty) if (bgp->vpn_policy[afi].tovpn_rd_pretty)
XFREE(MTYPE_BGP, bgp->vpn_policy[afi].tovpn_rd_pretty); XFREE(MTYPE_BGP_NAME,
bgp->vpn_policy[afi].tovpn_rd_pretty);
if (bgp->vpn_policy[afi].tovpn_sid_locator != NULL) if (bgp->vpn_policy[afi].tovpn_sid_locator != NULL)
srv6_locator_chunk_free( srv6_locator_chunk_free(
&bgp->vpn_policy[afi].tovpn_sid_locator); &bgp->vpn_policy[afi].tovpn_sid_locator);
@ -4060,10 +4061,10 @@ void bgp_free(struct bgp *bgp)
bgp_srv6_cleanup(bgp); bgp_srv6_cleanup(bgp);
bgp_confederation_id_unset(bgp); bgp_confederation_id_unset(bgp);
XFREE(MTYPE_BGP, bgp->as_pretty); XFREE(MTYPE_BGP_NAME, bgp->as_pretty);
XFREE(MTYPE_BGP, bgp->name); XFREE(MTYPE_BGP_NAME, bgp->name);
XFREE(MTYPE_BGP, bgp->name_pretty); XFREE(MTYPE_BGP_NAME, bgp->name_pretty);
XFREE(MTYPE_BGP, bgp->snmp_stats); XFREE(MTYPE_BGP_NAME, bgp->snmp_stats);
XFREE(MTYPE_BGP, bgp); XFREE(MTYPE_BGP, bgp);
} }
@ -6465,8 +6466,8 @@ int peer_local_as_set(struct peer *peer, as_t as, bool no_prepend,
peer->change_local_as = as; peer->change_local_as = as;
if (as_str) { if (as_str) {
if (peer->change_local_as_pretty) if (peer->change_local_as_pretty)
XFREE(MTYPE_BGP, peer->change_local_as_pretty); XFREE(MTYPE_BGP_NAME, peer->change_local_as_pretty);
peer->change_local_as_pretty = XSTRDUP(MTYPE_BGP, as_str); peer->change_local_as_pretty = XSTRDUP(MTYPE_BGP_NAME, as_str);
} }
(void)peer_sort(peer); (void)peer_sort(peer);
@ -6503,8 +6504,8 @@ int peer_local_as_set(struct peer *peer, as_t as, bool no_prepend,
replace_as); replace_as);
member->change_local_as = as; member->change_local_as = as;
if (as_str) if (as_str)
member->change_local_as_pretty = member->change_local_as_pretty = XSTRDUP(MTYPE_BGP_NAME,
XSTRDUP(MTYPE_BGP, as_str); as_str);
} }
return 0; return 0;
@ -6530,7 +6531,7 @@ int peer_local_as_unset(struct peer *peer)
peer_flag_unset(peer, PEER_FLAG_LOCAL_AS_NO_PREPEND); peer_flag_unset(peer, PEER_FLAG_LOCAL_AS_NO_PREPEND);
peer_flag_unset(peer, PEER_FLAG_LOCAL_AS_REPLACE_AS); peer_flag_unset(peer, PEER_FLAG_LOCAL_AS_REPLACE_AS);
peer->change_local_as = 0; peer->change_local_as = 0;
XFREE(MTYPE_BGP, peer->change_local_as_pretty); XFREE(MTYPE_BGP_NAME, peer->change_local_as_pretty);
} }
/* Check if handling a regular peer. */ /* Check if handling a regular peer. */
@ -6561,7 +6562,7 @@ int peer_local_as_unset(struct peer *peer)
UNSET_FLAG(member->flags, PEER_FLAG_LOCAL_AS_NO_PREPEND); UNSET_FLAG(member->flags, PEER_FLAG_LOCAL_AS_NO_PREPEND);
UNSET_FLAG(member->flags, PEER_FLAG_LOCAL_AS_REPLACE_AS); UNSET_FLAG(member->flags, PEER_FLAG_LOCAL_AS_REPLACE_AS);
member->change_local_as = 0; member->change_local_as = 0;
XFREE(MTYPE_BGP, member->change_local_as_pretty); XFREE(MTYPE_BGP_NAME, member->change_local_as_pretty);
/* Send notification or stop peer depending on state. */ /* Send notification or stop peer depending on state. */
if (BGP_IS_VALID_STATE_FOR_NOTIF(member->connection->status)) { if (BGP_IS_VALID_STATE_FOR_NOTIF(member->connection->status)) {