]> git.proxmox.com Git - mirror_frr.git/commit
bgpd: remove ATTR_NEXT_HOP for redistributed ipv6 nexthops
authorPhilippe Guibert <philippe.guibert@6wind.com>
Wed, 1 Feb 2023 15:52:28 +0000 (16:52 +0100)
committerPhilippe Guibert <philippe.guibert@6wind.com>
Tue, 9 May 2023 19:00:57 +0000 (21:00 +0200)
commit83b50eff9aeae7e3c4ef2aa001ddd310c983f0f0
treee0820ee00a95105890e1fb787dae73783459b54a
parent9fa282eeb64362e5b12cfd09bd41d24ce6e5c04b
bgpd: remove ATTR_NEXT_HOP for redistributed ipv6 nexthops

This commit addresses an issue with an MPLS VPN network
redistributing static routes that are exported to the VPN,
and where the labels are allocated per next-hop.

For that purpose, the nexthop of the static routes is
checked against the nexthop tracking. The validation
of a valid nexthop will trigger the use of a unique
label for all prefixes using that destination.

However, the nexthop fails to be validated, with the
following message:

 > evaluate_paths: prefix 172:31::14/128 (vrf vrf1), ignoring path due to
 > martian or self-next-hop

The reason is due to the way the attr is created.
By default, the ATTR_NEXTHOP attribute is set for
all prefixes, whereas this flag should only be valid
for IPv4. In the case there is an IPv6 nexthop, remove
the ATTR_NEXTHOP flag.

Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
bgpd/bgp_route.c