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.

Sign up for GitHub

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

Jump to bottom

bgpd: Validate both nexthop information (NEXTHOP and NLRI) (backport #17435) #17441

Open
wants to merge 2 commits into
base: stable/10.0
Choose a base branch
from
Open

bgpd: Validate both nexthop information (NEXTHOP and NLRI) (backport #17435) #17441

wants to merge 2 commits into from
+201 −21

Commits on Nov 17, 2024

  1. tests: Check if IPv6 prefix has a valid nexthop if multiple NLRIs exist 

    Signed-off-by: Donatas Abraitis <[email protected]>
(cherry picked from commit 229466e)
    @ton31337 @mergify
    ton31337 authored and mergify[bot] committed Nov 17, 2024
    Configuration menu
    Copy the full SHA
    e127cae View commit details
    Browse the repository at this point in the history

  2. bgpd: Validate both nexthop information (NEXTHOP and NLRI) 

    If we receive an IPv6 prefix e.g.: 2001:db8:100::/64 with nextop: 0.0.0.0, and
mp_nexthop: fc00::2, we should not treat this with an invalid nexthop because
of 0.0.0.0. We MUST check for MP_REACH attribute also and decide later if we
have at least one a valid nexthop.

Signed-off-by: Donatas Abraitis <[email protected]>
(cherry picked from commit a0d2734)

# Conflicts:
#	bgpd/bgp_route.c
    @ton31337 @mergify
    ton31337 authored and mergify[bot] committed Nov 17, 2024
    Configuration menu
    Copy the full SHA
    027d813 View commit details
    Browse the repository at this point in the history