]> git.puffer.fish Git - matthieu/frr.git/commitdiff
bgpd: fix 6vpe nexthop
authorLouis Scalbert <louis.scalbert@6wind.com>
Thu, 15 Feb 2024 12:28:02 +0000 (13:28 +0100)
committerMergify <37929162+mergify[bot]@users.noreply.github.com>
Wed, 28 Feb 2024 09:35:43 +0000 (09:35 +0000)
6vPE enables the announcement of IPv6 VPN prefixes through an IPv4 BGP
session. In this scenario, the next hop addresses for these prefixes are
represented in an IPv4-mapped IPv6 format, noted as ::ffff:[IPv4]. This
format indicates to the peer that it should route these IPv6 addresses
using information from the IPv4 nexthop. For example:

> Path Attribute - MP_REACH_NLRI
> [...]
>     Address family identifier (AFI): IPv6 (2)
>     Subsequent address family identifier (SAFI): Labeled VPN Unicast (128)
>     Next hop:  RD=0:0 IPv6=::ffff:192.0.2.5 RD=0:0 Link-local=fe80::501d:42ff:feef:b021
>     Number of Subnetwork points of attachment (SNPA): 0

This rule is set out in RFC4798:

> The IPv4 address of the egress 6PE router MUST be encoded as an
> IPv4-mapped IPv6 address in the BGP Next Hop field.

However, in some situations, bgpd sends a standard nexthop IPv6 address
instead of an IPv4-mapped IPv6 address because the outgoing interface for
the BGP session has a valid IPv6 address. This is problematic because
the peer router may not be able to route the nexthop IPv6 address (ie.
if the outgoing interface has not IPv6).

Fix the issue by always sending a IPv4-mapped IPv6 address as nexthop
when the BGP session is on IPv4 and address family IPv6.

Link: https://datatracker.ietf.org/doc/html/rfc4798#section-2
Fixes: 92d6f76 ("lib,zebra,bgpd: Fix for nexthop as IPv4 mapped IPv6 address")
Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
(cherry picked from commit 0325116a27258e1df773a046e8668a029bead60c)

bgpd/bgp_updgrp_packet.c

index 7502bf2ec6dcf05ab7979e12793601df5b9d4365..bff52c80d8c78f3da0235b596f0df0fd83c15e54 100644 (file)
@@ -523,11 +523,16 @@ struct stream *bpacket_reformat_for_peer(struct bpacket *pkt,
                        gnh_modified = 1;
                }
 
-               if (IN6_IS_ADDR_UNSPECIFIED(mod_v6nhg)) {
-                       if (peer->nexthop.v4.s_addr != INADDR_ANY) {
-                               ipv4_to_ipv4_mapped_ipv6(mod_v6nhg,
-                                                        peer->nexthop.v4);
-                       }
+               if (peer->nexthop.v4.s_addr != INADDR_ANY &&
+                   (IN6_IS_ADDR_UNSPECIFIED(mod_v6nhg) ||
+                    (peer->connection->su.sa.sa_family == AF_INET &&
+                     paf->afi == AFI_IP6))) {
+                       /* set a IPv4 mapped IPv6 address if no global IPv6
+                        * address is found or if announcing IPv6 prefix
+                        * over an IPv4 BGP session.
+                        */
+                       ipv4_to_ipv4_mapped_ipv6(mod_v6nhg, peer->nexthop.v4);
+                       gnh_modified = 1;
                }
 
                if (IS_MAPPED_IPV6(&peer->nexthop.v6_global)) {