April 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

Fwd: FW: FRRouting - RIP VRF Support.
by Amit Tiwari 26 Jul '18

26 Jul '18

Hi Kunihiro, I have compiled FRRouting master branch and deployed it on my router, bgp and ospf are working fine with vrf. Whereas, in case of RIP, vrf doesn't seems to be working, there is no vrf support even at CLI level. May we know, when are you guys releasing frrouting with full vrf support in RIP. Regards, Amit Kumar Tiwari 9953769181

3 2

Today's Technical Meeting Agenda
by Philippe Guibert 08 May '18

08 May '18

Hi all, For todays call, there will be a discussion about Yang based management. Which approach should be taken to enhance Yang int FRRouting. We, at 6WIND would like to share with you our feedback about Yang approach, and the issues identified. You can find the link below for more information. https://docs.google.com/presentation/d/1rVCpH2VI1TnkAzZInlDUHAp9Zo4PjjzY9qz… Thanks, Philippe

7 18

documentation now on the website
by Quentin Young 02 May '18

02 May '18

Hi folks, Updated documentation is now on the FRR website. User guide: https://frrouting.org/user-guide/ <https://frrouting.org/user-guide/> Developer’s guide: https://frrouting.org/dev-guide/ <https://frrouting.org/dev-guide/> This was built from the tip of master and should be accurate for the upcoming 4.0 release. Eventually I will work on a way to publish versioned documentation for each release. The developer documentation will be rolling. Thanks, Quentin

4 3

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

26 Apr '18

Hi, Please find the latest report on new defect(s) introduced to freerangerouting/frr found with Coverity Scan. 5 new defect(s) introduced to freerangerouting/frr found with Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 5 of 5 defect(s) ** CID 1468416: Error handling issues (CHECKED_RETURN) /bgpd/bgp_rd.c: 213 in form_auto_rd() ________________________________________________________________________________________________________ *** CID 1468416: Error handling issues (CHECKED_RETURN) /bgpd/bgp_rd.c: 213 in form_auto_rd() 207 { 208 char buf[100]; 209 210 prd->family = AF_UNSPEC; 211 prd->prefixlen = 64; 212 sprintf(buf, "%s:%hu", inet_ntoa(router_id), rd_id); >>> CID 1468416: Error handling issues (CHECKED_RETURN) >>> Calling "str2prefix_rd" without checking return value (as is done elsewhere 15 out of 16 times). 213 str2prefix_rd(buf, prd); ** CID 1468415: Null pointer dereferences (FORWARD_NULL) ________________________________________________________________________________________________________ *** CID 1468415: Null pointer dereferences (FORWARD_NULL) /bgpd/bgp_vty_clippy.c: 361 in af_route_map_vpn_imexport() 355 } 356 #if 0 /* anything that can fail? */ 357 if (_failcnt) 358 return CMD_WARNING; 359 #endif 360 #endif >>> CID 1468415: Null pointer dereferences (FORWARD_NULL) >>> Passing null pointer "rmap_str" to "af_route_map_vpn_imexport_magic", which dereferences it. 361 return af_route_map_vpn_imexport_magic(self, vty, argc, argv, no, direction_str, rmap_str); 362 } 363 364 /* af_import_vrf_route_map => "[no] import vrf route-map RMAP$rmap_str" */ 365 DEFUN_CMD_FUNC_DECL(af_import_vrf_route_map) 366 #define funcdecl_af_import_vrf_route_map static int af_import_vrf_route_map_magic(\ ** CID 1468414: Null pointer dereferences (FORWARD_NULL) ________________________________________________________________________________________________________ *** CID 1468414: Null pointer dereferences (FORWARD_NULL) /bgpd/bgp_vty_clippy.c: 409 in af_import_vrf_route_map() 403 } 404 #if 0 /* anything that can fail? */ 405 if (_failcnt) 406 return CMD_WARNING; 407 #endif 408 #endif >>> CID 1468414: Null pointer dereferences (FORWARD_NULL) >>> Passing null pointer "rmap_str" to "af_import_vrf_route_map_magic", which dereferences it. 409 return af_import_vrf_route_map_magic(self, vty, argc, argv, no, rmap_str); 410 } 411 412 /* bgp_imexport_vrf => "[no] import vrf NAME$import_name" */ 413 DEFUN_CMD_FUNC_DECL(bgp_imexport_vrf) 414 #define funcdecl_bgp_imexport_vrf static int bgp_imexport_vrf_magic(\ ** CID 1468413: Null pointer dereferences (FORWARD_NULL) ________________________________________________________________________________________________________ *** CID 1468413: Null pointer dereferences (FORWARD_NULL) /bgpd/bgp_vty_clippy.c: 457 in bgp_imexport_vrf() 451 } 452 #if 0 /* anything that can fail? */ 453 if (_failcnt) 454 return CMD_WARNING; 455 #endif 456 #endif >>> CID 1468413: Null pointer dereferences (FORWARD_NULL) >>> Passing null pointer "import_name" to "bgp_imexport_vrf_magic", which dereferences it. 457 return bgp_imexport_vrf_magic(self, vty, argc, argv, no, import_name); 458 } 459 460 /* bgp_imexport_vpn => "[no] <import|export>$direction_str vpn" */ 461 DEFUN_CMD_FUNC_DECL(bgp_imexport_vpn) 462 #define funcdecl_bgp_imexport_vpn static int bgp_imexport_vpn_magic(\ ** CID 1465490: Error handling issues (CHECKED_RETURN) /pimd/pim_igmp_mtrace.c: 817 in igmp_mtrace_recv_qry_req() ________________________________________________________________________________________________________ *** CID 1465490: Error handling issues (CHECKED_RETURN) /pimd/pim_igmp_mtrace.c: 817 in igmp_mtrace_recv_qry_req() 811 } 812 /* 813 * 6.4 Forwarding Traceroute Requests: 814 * Previous-hop router not known, 815 * packet is sent to an appropriate multicast address 816 */ >>> CID 1465490: Error handling issues (CHECKED_RETURN) >>> Calling "inet_aton" without checking return value (as is done elsewhere 115 out of 117 times). 817 inet_aton(MCAST_ALL_ROUTERS, &nh_addr); 818 } 819 820 /* 6.2.2 8. If this router is the Rendez-vous Point */ 821 if (pim_rp_i_am_rp(pim, mtracep->grp_addr)) { 822 mtrace_rsp_set_fwd_code(rspp, MTRACE_FWD_CODE_REACHED_RP); ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V0…

1 0

Invite to FRRouting slack
by quentusrex＠gmail.com 24 Apr '18

24 Apr '18

My name is William King, CTO of Flowroute located in Seattle, WA. Looking to evaluate migrating from BIRD to FRRouting, and better connect with the community. -William

2 1

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

20 Apr '18

Hi, Please find the latest report on new defect(s) introduced to freerangerouting/frr found with Coverity Scan. 3 new defect(s) introduced to freerangerouting/frr found with Coverity Scan. 1 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 3 of 3 defect(s) ** CID 1468032: Null pointer dereferences (FORWARD_NULL) ________________________________________________________________________________________________________ *** CID 1468032: Null pointer dereferences (FORWARD_NULL) /pbrd/pbr_vty_clippy.c: 217 in pbr_map_nexthop() 211 } 212 #if 1 /* anything that can fail? */ 213 if (_failcnt) 214 return CMD_WARNING; 215 #endif 216 #endif >>> CID 1468032: Null pointer dereferences (FORWARD_NULL) >>> Passing null pointer "addr" to "pbr_map_nexthop_magic", which dereferences it. 217 return pbr_map_nexthop_magic(self, vty, argc, argv, no, addr, addr_str, intf, name); 218 } 219 220 /* pbr_policy => "[no] pbr-policy NAME$mapname" */ 221 DEFUN_CMD_FUNC_DECL(pbr_policy) 222 #define funcdecl_pbr_policy static int pbr_policy_magic(\ ** CID 1468031: Integer handling issues (OVERFLOW_BEFORE_WIDEN) /pbrd/pbr_zebra.c: 497 in pbr_send_pbr_map() ________________________________________________________________________________________________________ *** CID 1468031: Integer handling issues (OVERFLOW_BEFORE_WIDEN) /pbrd/pbr_zebra.c: 497 in pbr_send_pbr_map() 491 492 void pbr_send_pbr_map(struct pbr_map_sequence *pbrms, 493 struct pbr_map_interface *pmi, bool install) 494 { 495 struct pbr_map *pbrm = pbrms->parent; 496 struct stream *s; >>> CID 1468031: Integer handling issues (OVERFLOW_BEFORE_WIDEN) >>> Potentially overflowing expression "1 << pmi->install_bit" with type "int" (32 bits, signed) is evaluated using 32-bit arithmetic, and then used in a context that expects an expression of type "uint64_t" (64 bits, unsigned). 497 uint64_t is_installed = 1 << pmi->install_bit; 498 499 is_installed &= pbrms->installed; 500 501 DEBUGD(&pbr_dbg_zebra, "%s: for %s %d(%" PRIu64 ")", 502 __PRETTY_FUNCTION__, pbrm->name, install, is_installed); ** CID 1468030: Null pointer dereferences (FORWARD_NULL) ________________________________________________________________________________________________________ *** CID 1468030: Null pointer dereferences (FORWARD_NULL) /lib/nexthop_group_clippy.c: 67 in ecmp_nexthops() 61 } 62 #if 1 /* anything that can fail? */ 63 if (_failcnt) 64 return CMD_WARNING; 65 #endif 66 #endif >>> CID 1468030: Null pointer dereferences (FORWARD_NULL) >>> Passing null pointer "addr" to "ecmp_nexthops_magic", which dereferences it. 67 return ecmp_nexthops_magic(self, vty, argc, argv, no, addr, addr_str, intf, name); ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, 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 17 Apr '18

17 Apr '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 1467945: Memory - corruptions (UNINIT) /zebra/redistribute.c: 534 in zebra_add_import_table_entry() ________________________________________________________________________________________________________ *** CID 1467945: Memory - corruptions (UNINIT) /zebra/redistribute.c: 534 in zebra_add_import_table_entry() 528 if (rmap_name) 529 ret = zebra_import_table_route_map_check( 530 afi, re->type, &rn->p, re->ng.nexthop, re->vrf_id, 531 re->tag, rmap_name); 532 533 if (ret != RMAP_MATCH) { >>> CID 1467945: Memory - corruptions (UNINIT) >>> Using uninitialized value "same". 534 UNSET_FLAG(same->flags, ZEBRA_FLAG_SELECTED); 535 zebra_del_import_table_entry(rn, re); 536 return 0; 537 } 538 539 prefix_copy(&p, &rn->p); ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u2389337.ct.sendgrid.net/wf/click?upn=08onrYu34A-2BWcWUl-2F-2BfV0V0…

1 0

PR without a review for 18 days
by Donald Sharp 16 Apr '18

16 Apr '18

This PR has been without a review for 18 days, without a review: https://github.com/FRRouting/frr/pull/1988 I am planning to push this in at 10:30 AM EDT April 17. Speak now if you want to look at this. thanks! donald

3 2

another slack invite request
by Ronald van der Pol 13 Apr '18

13 Apr '18

Can someone invite me on the slack channel? Thanks. rvdp

2 1