proxmox-mirrors/mirror_frr
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_frr synced 2025-08-13 21:10:28 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

bgpd: fix crash on "no import vrf" if no default bgp instance

Browse Source 
Tripped over a crash running the cli_crawler that occurred when the
sequence was doing "import vrf NAME" and "no import vrf NAME" inside
a vrf but a default bgp instance had not been created.  This fix
auto-creates the default instance if the "import vrf NAME" is
entered and a default instance does not exist.

Ticket: CM-20532
Signed-off-by: Don Slice <dslice@cumulusnetworks.com>
Reviewed-by: Donald Sharp <sharpd@cumulusnetworks.com>
This commit is contained in:
Don Slice 2018-04-11 16:12:39 +00:00 committed by Donald Sharp
parent e49840c895
commit 79ef8664a5
1 changed files with 19 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
bgpd/bgp_vty.c
View File

@ -6664,7 +6664,9 @@ DEFPY (bgp_imexport_vrf,
{
VTY_DECLVAR_CONTEXT(bgp, bgp);
struct listnode *node;
struct bgp *vrf_bgp;
struct bgp *vrf_bgp, *bgp_default;
int32_t ret = 0;
as_t as = bgp->as;
bool remove = false;
int32_t idx = 0;
char *vname;
@ -6678,19 +6680,26 @@ DEFPY (bgp_imexport_vrf,
afi = bgp_node_afi(vty);
safi = bgp_node_safi(vty);
bgp_default = bgp_get_default();
if (!bgp_default) {
/* Auto-create assuming the same AS */
ret = bgp_get(&bgp_default, &as, NULL,
BGP_INSTANCE_TYPE_DEFAULT);
if (ret) {
vty_out(vty,
"VRF default is not configured as a bgp instance\n");
return CMD_WARNING;
}
}
vrf_bgp = bgp_lookup_by_name(import_name);
if (!vrf_bgp) {
int32_t ret = 0;
as_t as = bgp->as;
if (strcmp(import_name, BGP_DEFAULT_NAME) == 0) {
vrf_bgp = bgp_get_default();
if (!vrf_bgp)
ret = bgp_get(&vrf_bgp, &as, NULL, BGP_INSTANCE_TYPE_DEFAULT);
} else {
if (strcmp(import_name, BGP_DEFAULT_NAME) == 0)
vrf_bgp = bgp_default;
else
/* Auto-create assuming the same AS */
ret = bgp_get(&vrf_bgp, &as, import_name, bgp_type);
}
if (ret) {
vty_out(vty,