Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bgpd: A couple more bgpd crash fixes for malformed packets (backport #14716) #14735

Merged
merged 2 commits into from
Nov 6, 2023

bgpd: Ignore handling NLRIs if we received MP_UNREACH_NLRI

250844b
Select commit
Loading
Failed to load commit list.
Merged

bgpd: A couple more bgpd crash fixes for malformed packets (backport #14716) #14735

bgpd: Ignore handling NLRIs if we received MP_UNREACH_NLRI
250844b
Select commit
Loading
Failed to load commit list.
NetDEF CI Pull Requests Testing / TopoTests Ubuntu 18.04 arm8 Part 5 succeeded Nov 5, 2023 in 19m 46s

Execution finished at ci09-FRR-PULLREQ3-TOPO5U18ARM8-551 after 1149.194 seconds

Bamboo CI Link
Execution details at https://ci1.netdef.org/browse/FRR-PULLREQ3-TOPO5U18ARM8-551

Execution Summary

Failures
All tests passed

Success and Not classified
✔️ bgp_max_med_on_startup.test_bgp_max_med_on_startup test_bgp_max_med_on_startup (10.413 s)
✔️ bgp_maximum_prefix_invalid_update.test_bgp_maximum_prefix_invalid_update test_bgp_maximum_prefix_invalid (6.352 s)
✔️ bgp_maximum_prefix_out.test_bgp_maximum_prefix_out test_bgp_maximum_prefix_out (23.369 s)
✔️ bgp_minimum_holdtime.test_bgp_minimum_holdtime test_bgp_minimum_holdtime (5.493 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_address_unambiguous_within_each_vrf_p0 (31.649 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_ambiguous_overlapping_addresses_in_different_vrfs_p0 (2.178 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_static_routes_associated_to_specific_vrfs_p0 (2.495 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_vrf_with_unique_physical_interface_p0 (4.536 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_prefixes_leaking_p0 (4.881 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_static_routes_advertised_within_specific_vrf_p0 (3.874 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_end_to_end_traffic_isolation_p0 (29.339 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_static_routes_for_inter_vrf_route_leaking_p0 (1.805 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_inter_vrf_and_intra_vrf_communication_iBGP_p0 (1.845 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_inter_vrf_and_intra_vrf_communication_eBGP_p0 (1.861 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_route_map_within_vrf_to_alter_bgp_attribute_nexthop_p0 (5.431 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_route_map_within_vrf_to_alter_bgp_attribute_p0[locPrf] (12.306 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_route_map_within_vrf_to_alter_bgp_attribute_p0[weight] (12.473 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_route_map_within_vrf_to_alter_bgp_attribute_p0[metric] (12.452 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_route_map_within_vrf_to_alter_bgp_attribute_aspath_p0 (12.356 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_route_map_within_vrf_to_alter_bgp_attribute_lcomm_p0 (4.856 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_route_map_match_traffic_based_on_vrf_p0 (6.074 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_vrf_lite_with_static_bgp_originated_routes_p0 (11.237 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_prefix_list_to_permit_deny_prefixes_p0 (6.240 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_route_map_set_and_match_tag_p0 (2.238 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_route_map_set_and_match_metric_p0 (2.007 s)
✔️ bgp_multi_vrf_topo1.test_bgp_multi_vrf_topo1 test_route_map_set_and_match_community_p0 (9.522 s)
✔️ bgp_multi_vrf_topo2.test_bgp_multi_vrf_topo2 test_vrf_with_multiple_links_p1 (108.744 s)
✔️ bgp_multi_vrf_topo2.test_bgp_multi_vrf_topo2 test_shut_noshut_p1 (28.350 s)
✔️ bgp_multi_vrf_topo2.test_bgp_multi_vrf_topo2 test_vrf_vlan_routing_table_p1 (23.827 s)
✔️ bgp_multi_vrf_topo2.test_bgp_multi_vrf_topo2 test_vrf_route_leaking_next_hop_interface_flapping_p1 (3.148 s)
✔️ bgp_multi_vrf_topo2.test_bgp_multi_vrf_topo2 test_restart_bgpd_daemon_p1 (11.806 s)
✔️ bgp_multi_vrf_topo2.test_bgp_multi_vrf_topo2 test_delete_and_re_add_vrf_p1 (38.066 s)
✔️ bgp_multi_vrf_topo2.test_bgp_multi_vrf_topo2 test_vrf_name_significance_p1 (61.379 s)
✔️ bgp_multi_vrf_topo2.test_bgp_multi_vrf_topo2 test_restart_frr_services_p1 (15.319 s)
✔️ bgp_multiview_topo1.test_bgp_multiview_topo1 test_router_running (10.082 s)
✔️ bgp_multiview_topo1.test_bgp_multiview_topo1 test_bgp_converge (10.420 s)
✔️ bgp_multiview_topo1.test_bgp_multiview_topo1 test_bgp_routingTable (0.342 s)
⚠️ bgp_multiview_topo1.test_bgp_multiview_topo1 test_shutdown_check_memleak(1.613 s)

/root/share/topotests/bgp_multiview_topo1/test_bgp_multiview_topo1.py:216: Memory leak test/report is disabled

✔️ bgp_multiview_topo1.test_bgp_multiview_topo1 test_shutdown_check_memleak (1.613 s)
✔️ bgp_multiview_topo1.test_bgp_multiview_topo1 test_shutdown_check_memleak (1.613 s)
✔️ bgp_path_attribute_discard.test_bgp_path_attribute_discard test_bgp_path_attribute_discard (5.911 s)
⚠️ bgp_path_attribute_discard.test_bgp_path_attribute_discard test_memory_leak(1.174 s)

/root/share/topotests/bgp_path_attribute_discard/test_bgp_path_attribute_discard.py:147: Memory leak test/report is disabled

✔️ bgp_path_attribute_discard.test_bgp_path_attribute_discard test_memory_leak (1.174 s)
✔️ bgp_path_attribute_discard.test_bgp_path_attribute_discard test_memory_leak (1.174 s)
✔️ bgp_path_attribute_treat_as_withdraw.test_bgp_path_attribute_treat_as_withdraw test_bgp_path_attribute_treat_as_withdraw (8.291 s)
⚠️ bgp_path_attribute_treat_as_withdraw.test_bgp_path_attribute_treat_as_withdraw test_memory_leak(1.902 s)

/root/share/topotests/bgp_path_attribute_treat_as_withdraw/test_bgp_path_attribute_treat_as_withdraw.py:142: Memory leak test/report is disabled

✔️ bgp_path_attribute_treat_as_withdraw.test_bgp_path_attribute_treat_as_withdraw test_memory_leak (1.902 s)
✔️ bgp_path_attribute_treat_as_withdraw.test_bgp_path_attribute_treat_as_withdraw test_memory_leak (1.902 s)
✔️ bgp_path_attributes_topo1.test_bgp_path_attributes test_next_hop_attribute (22.384 s)
✔️ bgp_path_attributes_topo1.test_bgp_path_attributes test_aspath_attribute (11.050 s)
✔️ bgp_path_attributes_topo1.test_bgp_path_attributes test_localpref_attribute (11.046 s)
✔️ bgp_path_attributes_topo1.test_bgp_path_attributes test_weight_attribute (11.103 s)
✔️ bgp_path_attributes_topo1.test_bgp_path_attributes test_origin_attribute (6.005 s)
✔️ bgp_path_attributes_topo1.test_bgp_path_attributes test_med_attribute (12.955 s)
✔️ bgp_path_attributes_topo1.test_bgp_path_attributes test_admin_distance (16.248 s)
✔️ bgp_path_selection.test_bgp_path_selection test_bgp_path_selection_ecmp (6.951 s)
✔️ bgp_path_selection.test_bgp_path_selection test_bgp_path_selection_vpn_ecmp (1.905 s)
✔️ bgp_path_selection.test_bgp_path_selection test_bgp_path_selection_metric (0.259 s)
✔️ bgp_path_selection.test_bgp_path_selection test_bgp_path_selection_vpn_metric (3.093 s)
✔️ bgp_peer_graceful_shutdown.test_bgp_peer_graceful_shutdown test_bgp_orf (8.904 s)
✔️ bgp_peer_group.test_bgp_peer-group test_bgp_peer_group (8.936 s)
✔️ bgp_peer_type_multipath_relax.test_bgp_peer-type_multipath-relax test_bgp_peer_type_multipath_relax (20.436 s)
⚠️ bgp_peer_type_multipath_relax.test_bgp_peer-type_multipath-relax test_memory_leak(1.460 s)

/root/share/topotests/bgp_peer_type_multipath_relax/test_bgp_peer-type_multipath-relax.py:365: Memory leak test/report is disabled

✔️ bgp_peer_type_multipath_relax.test_bgp_peer-type_multipath-relax test_memory_leak (1.460 s)
✔️ bgp_peer_type_multipath_relax.test_bgp_peer-type_multipath-relax test_memory_leak (1.460 s)
✔️ bgp_prefix_list_any.test_bgp_prefix_list_any test_bgp_route_map_prefix_list (21.548 s)
✔️ bgp_prefix_list_topo1.test_prefix_lists test_ip_prefix_lists_in_permit (12.455 s)
✔️ bgp_prefix_list_topo1.test_prefix_lists test_ip_prefix_lists_out_permit (3.085 s)
✔️ bgp_prefix_list_topo1.test_prefix_lists test_ip_prefix_lists_in_deny_and_permit_any (0.737 s)
✔️ bgp_prefix_list_topo1.test_prefix_lists test_delete_prefix_lists (0.768 s)
✔️ bgp_prefix_list_topo1.test_prefix_lists test_ip_prefix_lists_out_deny_and_permit_any (3.055 s)
✔️ bgp_prefix_list_topo1.test_prefix_lists test_modify_prefix_lists_in_permit_to_deny (9.365 s)
✔️ bgp_prefix_list_topo1.test_prefix_lists test_modify_prefix_lists_in_deny_to_permit (9.300 s)
✔️ bgp_prefix_list_topo1.test_prefix_lists test_modify_prefix_lists_out_permit_to_deny (9.329 s)
✔️ bgp_prefix_list_topo1.test_prefix_lists test_modify_prefix_lists_out_deny_to_permit (9.356 s)
✔️ bgp_prefix_list_topo1.test_prefix_lists test_ip_prefix_lists_implicit_deny (5.076 s)
✔️ bgp_prefix_list_topo1.test_prefix_modify test_bug_prefix_lists_deny_to_permit_p1 (33.746 s)
✔️ bgp_prefix_sid.test_bgp_prefix_sid test_r1_receive_and_advertise_prefix_sid_type1 (7.286 s)
✔️ bgp_prefix_sid.test_bgp_prefix_sid test_peer2_receive_prefix_sid_type1 (1.657 s)
✔️ bgp_prefix_sid2.test_bgp_prefix_sid2 test_r1_rib (7.472 s)
✔️ bgp_recursive_route_ebgp_multi_hop.test_bgp_recursive_route_ebgp_multi_hop test_recursive_routes_iBGP_peer_p1 (35.034 s)
✔️ bgp_recursive_route_ebgp_multi_hop.test_bgp_recursive_route_ebgp_multi_hop test_next_hop_as_self_ip_p1 (18.460 s)
✔️ bgp_recursive_route_ebgp_multi_hop.test_bgp_recursive_route_ebgp_multi_hop test_next_hop_with_recursive_lookup_p1 (37.383 s)
✔️ bgp_recursive_route_ebgp_multi_hop.test_bgp_recursive_route_ebgp_multi_hop test_BGP_path_attributes_default_values_p1 (10.985 s)
✔️ bgp_recursive_route_ebgp_multi_hop.test_bgp_recursive_route_ebgp_multi_hop test_BGP_peering_bw_loopback_and_physical_p1 (20.075 s)
✔️ bgp_recursive_route_ebgp_multi_hop.test_bgp_recursive_route_ebgp_multi_hop test_BGP_active_standby_preemption_and_ecmp_p1 (18.389 s)
✔️ bgp_recursive_route_ebgp_multi_hop.test_bgp_recursive_route_ebgp_multi_hop test_password_authentication_for_eBGP_and_iBGP_peers_p1 (18.383 s)
✔️ bgp_reject_as_sets.test_bgp_reject_as_sets test_bgp_reject_as_sets (9.091 s)
✔️ bgp_remove_private_as.test_bgp_remove_private_as test_bgp_remove_private_as (22.152 s)
✔️ bgp_remove_private_as_route_map.test_bgp_remove_private_as_route_map test_bgp_remove_private_as_route_map (11.164 s)
⚠️ bgp_rfapi_basic_sanity.test_bgp_rfapi_basic_sanity test_memory_leak(7.324 s)

/root/share/topotests/lib/ltemplate.py:197: Memory leak test/report is disabled

✔️ bgp_rfapi_basic_sanity.test_bgp_rfapi_basic_sanity test_memory_leak (7.324 s)
✔️ bgp_rfapi_basic_sanity.test_bgp_rfapi_basic_sanity test_memory_leak (7.324 s)
✔️ bgp_rfapi_basic_sanity.test_bgp_rfapi_basic_sanity test_add_routes (1.379 s)
✔️ bgp_rfapi_basic_sanity.test_bgp_rfapi_basic_sanity test_adjacencies (16.888 s)
✔️ bgp_rfapi_basic_sanity.test_bgp_rfapi_basic_sanity test_check_routes (1.529 s)
✔️ bgp_rfapi_basic_sanity.test_bgp_rfapi_basic_sanity test_check_close (0.792 s)
✔️ bgp_rfapi_basic_sanity.test_bgp_rfapi_basic_sanity test_check_timeout (0.291 s)
✔️ bgp_rfapi_basic_sanity.test_bgp_rfapi_basic_sanity test_cleanup_all (5.122 s)
⚠️ bgp_rfapi_basic_sanity_config2.test_bgp_rfapi_basic_sanity_config2 test_memory_leak(8.331 s)

/root/share/topotests/lib/ltemplate.py:197: Memory leak test/report is disabled

✔️ bgp_rfapi_basic_sanity_config2.test_bgp_rfapi_basic_sanity_config2 test_memory_leak (8.331 s)
✔️ bgp_rfapi_basic_sanity_config2.test_bgp_rfapi_basic_sanity_config2 test_memory_leak (8.331 s)
✔️ bgp_rfapi_basic_sanity_config2.test_bgp_rfapi_basic_sanity_config2 test_add_routes (1.400 s)
✔️ bgp_rfapi_basic_sanity_config2.test_bgp_rfapi_basic_sanity_config2 test_adjacencies (18.548 s)
✔️ bgp_rfapi_basic_sanity_config2.test_bgp_rfapi_basic_sanity_config2 test_check_routes (1.513 s)
✔️ bgp_rfapi_basic_sanity_config2.test_bgp_rfapi_basic_sanity_config2 test_check_close (0.790 s)
✔️ bgp_rfapi_basic_sanity_config2.test_bgp_rfapi_basic_sanity_config2 test_check_timeout (0.298 s)
✔️ bgp_rfapi_basic_sanity_config2.test_bgp_rfapi_basic_sanity_config2 test_cleanup_all (5.140 s)
✔️ bgp_route_aggregation.test_bgp_aggregation test_route_summarisation_with_summary_only_p1 (25.071 s)
✔️ bgp_route_aggregation.test_bgp_aggregation test_route_summarisation_with_as_set_p1 (17.255 s)