January 2018 - dev - lists.frrouting.org

no soft bgp clear is triggered on modification to prefixlist / routemap
by François 02 Aug '18

02 Aug '18

Hello, This issue has been discussed here : https://github.com/FRRouting/frr/issues/126 To sum up, when a route-map is modified (tried 2 things: adding a new entry and modifying prefix-list involved), no soft bgp clear is performed, so it looks like other peers never get new results (the "show advertised route" output is correct). Looks like "no route-map ... " and "route-map ..." again does the trick. My question is: do someone has an idea on how to fix that? I can give a shot, but I would appreciate some inputs ;) Ps: I use version 2 of FRR (because Vincent Jardin told that version 3 was not production ready at FRNog 29 ;)) Thanks! -- François

3 6

BGP Route Leaking for VRF-Lite and VPN's
by Donald Sharp 15 Mar '18

15 Mar '18

All - During discussions this morning w/ Lou we discovered that we are both working towards a BGP solution for Route Leaking. We have enough differences between our approaches that we need to spend some time consolidating our approach. The current plan is to have the meeting at 3:30 EDT this Friday Jan 19. If you would like to attend please let me know and I will put you on the invite for it. donald

4 15

Manpage content
by Quentin Young 01 Feb '18

01 Feb '18

Hi folks, As part of the work I’m doing to convert all of our docs to RST, I’ve now reached the point where I am converting over the manpages from groff and want some community input on content. As it stands the manpages have not been significantly updated since ~2004 and contain little more than a brief summary of option flags. Now that all of the user documentation is in RST, as far as I can see we have two options: 1. Keep the manpages separate, with their current minimal content. Example (converted) manpage source: (bgpd.rst) http://ix.io/F0E <http://ix.io/F0E> (common-options.rst) http://ix.io/F0G <http://ix.io/F0G> 2. Add the standard SYNOPSIS, DESCRIPTION, OPTIONS etc. sections, as in the source above, to the top of each chapter in the user documentation that covers a daemon, and then generate manpages from the same source we use to generate the various other manuals (web, info, pdf, etc). The only downside I see to option #2 is that some of the manpages have small novels written inside them (for example the 24 paragraph section on MED in the BGP chapter), which degrades their usefulness as the “quick reference” people tend to use them for these days. Whereas the upside is 13 less (albeit small) files to update when things change and a significant increase in the amount of overall information in the manpages. Thoughts?

3 3

MPLS use cases and Linux Kernel Interactions
by Donald Sharp 01 Feb '18

01 Feb '18

All - We plan to have a MPLS use case and Linux Kernel Interactions meeting this Wed at 10am EDT. If you would like to attend please let me know and I'll get you an invite. thanks! donald

2 3

ABOUT TO TRANSFER YOUR FUND
by Mrs Robin Sanders 31 Jan '18

31 Jan '18

1 0

AT&T to Release “dNOS” Software Framework Into Open Source to Accelerate Network White Box Adoption
by Olivier Dugeon 31 Jan '18

31 Jan '18

Hi all, Do you see this news: http://about.att.com/story/dnos_software_framework_into_open_source.html As AT&T mention the asset of Vyatta, do you know if they continue to use Quagga or if they have switch to FRR ? Perhaps it is time to contact AT&T as in their previous white paper about dNOS, they are very criticism against Quagga. Regards Olivier

2 1

New Defects reported by Coverity Scan for freerangerouting/frr
by scan-admin＠coverity.com 31 Jan '18

31 Jan '18

Hi, Please find the latest report on new defect(s) introduced to freerangerouting/frr found with Coverity Scan. 2 new defect(s) introduced to freerangerouting/frr found with Coverity Scan. 4 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 2 of 2 defect(s) ** CID 1464494: Null pointer dereferences (NULL_RETURNS) /zebra/zebra_vty.c: 2023 in ipv6_route_address_interface_magic() ________________________________________________________________________________________________________ *** CID 1464494: Null pointer dereferences (NULL_RETURNS) /zebra/zebra_vty.c: 2023 in ipv6_route_address_interface_magic() 2017 if (!nh_zvrf) { 2018 vty_out(vty, "%% nexthop vrf %s is not defined\n", 2019 nexthop_vrf); 2020 return CMD_WARNING_CONFIG_FAILED; 2021 } 2022 >>> CID 1464494: Null pointer dereferences (NULL_RETURNS) >>> Assigning: "zvrf" = null return value from "zebra_vrf_lookup_by_name". 2023 zvrf = zebra_vrf_lookup_by_name(vrf); 2024 if (!nh_zvrf) { 2025 vty_out(vty, "%% nexthop vrf %s is not defined\n", 2026 vrf); 2027 return CMD_WARNING_CONFIG_FAILED; 2028 } ** CID 1464493: Null pointer dereferences (NULL_RETURNS) /zebra/zebra_vty.c: 2114 in ipv6_route_magic() ________________________________________________________________________________________________________ *** CID 1464493: Null pointer dereferences (NULL_RETURNS) /zebra/zebra_vty.c: 2114 in ipv6_route_magic() 2108 if (!nh_zvrf) { 2109 vty_out(vty, "%% nexthop vrf %s is not defined\n", 2110 nexthop_vrf); 2111 return CMD_WARNING_CONFIG_FAILED; 2112 } 2113 >>> CID 1464493: Null pointer dereferences (NULL_RETURNS) >>> Assigning: "zvrf" = null return value from "zebra_vrf_lookup_by_name". 2114 zvrf = zebra_vrf_lookup_by_name(vrf); 2115 if (!nh_zvrf) { 2116 vty_out(vty, "%% nexthop vrf %s is not defined\n", 2117 vrf); 2118 return CMD_WARNING_CONFIG_FAILED; 2119 } ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V0… To manage Coverity Scan email notifications for "frr(a)lists.nox.tf", click https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V0…

1 0

New Defects reported by Coverity Scan for freerangerouting/frr
by scan-admin＠coverity.com 26 Jan '18

26 Jan '18

Hi, Please find the latest report on new defect(s) introduced to freerangerouting/frr found with Coverity Scan. 1 new defect(s) introduced to freerangerouting/frr found with Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 1 of 1 defect(s) ** CID 1464403: Control flow issues (DEADCODE) /zebra/zebra_ptm.c: 1011 in zebra_ptm_bfd_client_register() ________________________________________________________________________________________________________ *** CID 1464403: Control flow issues (DEADCODE) /zebra/zebra_ptm.c: 1011 in zebra_ptm_bfd_client_register() 1005 ZEBRA_PTM_BFD_CLIENT_FLAG_REG); 1006 1007 return 0; 1008 1009 stream_failure: 1010 if (out_ctxt) >>> CID 1464403: Control flow issues (DEADCODE) >>> Execution cannot reach this statement: "ptm_lib_cleanup_msg(ptm_hdl...". 1011 ptm_lib_cleanup_msg(ptm_hdl, out_ctxt); 1012 return 0; 1013 } 1014 1015 /* BFD client deregister */ 1016 void zebra_ptm_bfd_client_deregister(int proto) ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V0… To manage Coverity Scan email notifications for "frr(a)lists.nox.tf", click https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V0…

1 0

New Defects reported by Coverity Scan for freerangerouting/frr
by scan-admin＠coverity.com 24 Jan '18

24 Jan '18

Hi, Please find the latest report on new defect(s) introduced to freerangerouting/frr found with Coverity Scan. 1 new defect(s) introduced to freerangerouting/frr found with Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 1 of 1 defect(s) ** CID 1464359: Null pointer dereferences (FORWARD_NULL) /ospf6d/ospf6_spf.c: 1032 in ospf6_create_single_router_lsa() ________________________________________________________________________________________________________ *** CID 1464359: Null pointer dereferences (FORWARD_NULL) /ospf6d/ospf6_spf.c: 1032 in ospf6_create_single_router_lsa() 1026 lsa->header = (struct ospf6_lsa_header *)new_header; 1027 1028 lsa->lsdb = area->temp_router_lsa_lsdb; 1029 1030 /* Fill Larger LSA Payload */ 1031 end = ospf6_lsdb_head(lsdb, 2, type, adv_router, &rtr_lsa); >>> CID 1464359: Null pointer dereferences (FORWARD_NULL) >>> Comparing "rtr_lsa" to null implies that "rtr_lsa" might be null. 1032 if (rtr_lsa) { 1033 if (!OSPF6_LSA_IS_MAXAGE(rtr_lsa)) { 1034 /* Append first Link State ID LSA */ 1035 lsa_header = (struct ospf6_lsa_header *)rtr_lsa->header; 1036 memcpy(new_header, lsa_header, 1037 ntohs(lsa_header->length)); ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V0… To manage Coverity Scan email notifications for "frr(a)lists.nox.tf", click https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V0…

1 0

New Defects reported by Coverity Scan for freerangerouting/frr
by scan-admin＠coverity.com 24 Jan '18

24 Jan '18

Hi, Please find the latest report on new defect(s) introduced to freerangerouting/frr found with Coverity Scan. 8 new defect(s) introduced to freerangerouting/frr found with Coverity Scan. 3 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 8 of 8 defect(s) ** CID 1464358: Control flow issues (DEADCODE) /zebra/zebra_vty.c: 2107 in ipv6_route_magic() ________________________________________________________________________________________________________ *** CID 1464358: Control flow issues (DEADCODE) /zebra/zebra_vty.c: 2107 in ipv6_route_magic() 2101 nexthop_vrf); 2102 return CMD_WARNING_CONFIG_FAILED; 2103 } 2104 2105 zvrf = zebra_vrf_lookup_by_name(vrf); 2106 if (!nh_zvrf) { >>> CID 1464358: Control flow issues (DEADCODE) >>> Execution cannot reach this statement: "vty_out(vty, "%% nexthop vr...". 2107 vty_out(vty, "%% nexthop vrf %s is not defined\n", 2108 vrf); 2109 return CMD_WARNING_CONFIG_FAILED; 2110 } 2111 2112 return zebra_static_route_leak(vty, zvrf, nh_zvrf, ** CID 1464357: Null pointer dereferences (FORWARD_NULL) /zebra/zebra_vty_clippy.c: 192 in ip_route_blackhole_vrf() ________________________________________________________________________________________________________ *** CID 1464357: Null pointer dereferences (FORWARD_NULL) /zebra/zebra_vty_clippy.c: 192 in ip_route_blackhole_vrf() 186 #if 7 /* anything to parse? */ 187 int _i; 188 #if 1 /* anything that can fail? */ 189 unsigned _fail = 0, _failcnt = 0; 190 #endif 191 const char *no = NULL; >>> CID 1464357: Null pointer dereferences (FORWARD_NULL) >>> Assigning: "prefix" = "NULL". 192 const char *prefix = NULL; 193 struct in_addr mask = { INADDR_ANY }; 194 const char *mask_str = NULL; 195 const char *flag = NULL; 196 long tag = 0; 197 const char *tag_str = NULL; ** CID 1464356: Null pointer dereferences (FORWARD_NULL) /zebra/zebra_vty_clippy.c: 1439 in ipv6_route_vrf() ________________________________________________________________________________________________________ *** CID 1464356: Null pointer dereferences (FORWARD_NULL) /zebra/zebra_vty_clippy.c: 1439 in ipv6_route_vrf() 1433 int _i; 1434 #if 1 /* anything that can fail? */ 1435 unsigned _fail = 0, _failcnt = 0; 1436 #endif 1437 const char *no = NULL; 1438 struct prefix_ipv6 prefix = { }; >>> CID 1464356: Null pointer dereferences (FORWARD_NULL) >>> Assigning: "prefix_str" = "NULL". 1439 const char *prefix_str = NULL; 1440 struct prefix_ipv6 from = { }; 1441 const char *from_str = NULL; 1442 struct in6_addr gate = IN6ADDR_ANY_INIT; 1443 const char *gate_str = NULL; 1444 const char *ifname = NULL; ** CID 1464355: Control flow issues (DEADCODE) /zebra/zebra_vty.c: 2016 in ipv6_route_address_interface_magic() ________________________________________________________________________________________________________ *** CID 1464355: Control flow issues (DEADCODE) /zebra/zebra_vty.c: 2016 in ipv6_route_address_interface_magic() 2010 nexthop_vrf); 2011 return CMD_WARNING_CONFIG_FAILED; 2012 } 2013 2014 zvrf = zebra_vrf_lookup_by_name(vrf); 2015 if (!nh_zvrf) { >>> CID 1464355: Control flow issues (DEADCODE) >>> Execution cannot reach this statement: "vty_out(vty, "%% nexthop vr...". 2016 vty_out(vty, "%% nexthop vrf %s is not defined\n", 2017 vrf); 2018 return CMD_WARNING_CONFIG_FAILED; 2019 } 2020 2021 return zebra_static_route_leak(vty, zvrf, nh_zvrf, ** CID 1464354: Control flow issues (DEADCODE) /zebra/zebra_vty.c: 591 in ip_route_magic() ________________________________________________________________________________________________________ *** CID 1464354: Control flow issues (DEADCODE) /zebra/zebra_vty.c: 591 in ip_route_magic() 585 nexthop_vrf); 586 return CMD_WARNING_CONFIG_FAILED; 587 } 588 589 zvrf = zebra_vrf_lookup_by_name(vrf); 590 if (!nh_zvrf) { >>> CID 1464354: Control flow issues (DEADCODE) >>> Execution cannot reach this statement: "vty_out(vty, "%% nexthop vr...". 591 vty_out(vty, "%% nexthop vrf %s is not defined\n", 592 vrf); 593 return CMD_WARNING_CONFIG_FAILED; 594 } 595 596 return zebra_static_route_leak(vty, zvrf, nh_zvrf, ** CID 1464353: Control flow issues (DEADCODE) /zebra/zebra_vty.c: 487 in ip_route_address_interface_magic() ________________________________________________________________________________________________________ *** CID 1464353: Control flow issues (DEADCODE) /zebra/zebra_vty.c: 487 in ip_route_address_interface_magic() 481 nexthop_vrf); 482 return CMD_WARNING_CONFIG_FAILED; 483 } 484 485 zvrf = zebra_vrf_lookup_by_name(vrf); 486 if (!nh_zvrf) { >>> CID 1464353: Control flow issues (DEADCODE) >>> Execution cannot reach this statement: "vty_out(vty, "%% nexthop vr...". 487 vty_out(vty, "%% nexthop vrf %s is not defined\n", 488 vrf); 489 return CMD_WARNING_CONFIG_FAILED; 490 } 491 492 return zebra_static_route_leak(vty, zvrf, nh_zvrf, ** CID 1464352: Null pointer dereferences (FORWARD_NULL) /zebra/zebra_vty_clippy.c: 1230 in ipv6_route_address_interface_vrf() ________________________________________________________________________________________________________ *** CID 1464352: Null pointer dereferences (FORWARD_NULL) /zebra/zebra_vty_clippy.c: 1230 in ipv6_route_address_interface_vrf() 1224 int _i; 1225 #if 1 /* anything that can fail? */ 1226 unsigned _fail = 0, _failcnt = 0; 1227 #endif 1228 const char *no = NULL; 1229 struct prefix_ipv6 prefix = { }; >>> CID 1464352: Null pointer dereferences (FORWARD_NULL) >>> Assigning: "prefix_str" = "NULL". 1230 const char *prefix_str = NULL; 1231 struct prefix_ipv6 from = { }; 1232 const char *from_str = NULL; 1233 struct in6_addr gate = IN6ADDR_ANY_INIT; 1234 const char *gate_str = NULL; 1235 const char *ifname = NULL; ** CID 1464351: Null pointer dereferences (FORWARD_NULL) /zebra/zebra_vty_clippy.c: 1031 in ipv6_route_blackhole_vrf() ________________________________________________________________________________________________________ *** CID 1464351: Null pointer dereferences (FORWARD_NULL) /zebra/zebra_vty_clippy.c: 1031 in ipv6_route_blackhole_vrf() 1025 int _i; 1026 #if 1 /* anything that can fail? */ 1027 unsigned _fail = 0, _failcnt = 0; 1028 #endif 1029 const char *no = NULL; 1030 struct prefix_ipv6 prefix = { }; >>> CID 1464351: Null pointer dereferences (FORWARD_NULL) >>> Assigning: "prefix_str" = "NULL". 1031 const char *prefix_str = NULL; 1032 struct prefix_ipv6 from = { }; 1033 const char *from_str = NULL; 1034 const char *flag = NULL; 1035 long tag = 0; 1036 const char *tag_str = NULL; ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V0… To manage Coverity Scan email notifications for "frr(a)lists.nox.tf", click https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V0…

1 0