I was looking at the bgp code and noticed that we have different places
that we store nexthop information:

In 'struct attr'
  struct in_addr nexthop;                            <------IPv4 Nexthop
address

In 'struct attr_extra':

#ifdef HAVE_IPV6
  struct in6_addr mp_nexthop_global;        <-----IPv6 Nexthop Global
address
  struct in6_addr mp_nexthop_local;          <-----IPv6 Nexthop Link Local
address
#endif /* HAVE_IPV6 */
#ifdef HAVE_IPV6
  struct in6_addr mp_nexthop_global;        <-----VPNV4 Nexthop Address
  struct in6_addr mp_nexthop_local;          <-----Never Used
#endif /* HAVE_IPV6 */


Does anyone know the historical reasoning for this?  Does anyone mind if I
combine these data structures, in a somewhat more logical manner?

donald
_______________________________________________
Quagga-dev mailing list
[email protected]
https://lists.quagga.net/mailman/listinfo/quagga-dev

Reply via email to