Commit 509aab5d authored by Ondrej Zajicek's avatar Ondrej Zajicek
Browse files

Fixes serious bug in BGP add-path 

Temporary rta is reused in BGP, while rta_lookup() breaks it.

Thanks to Alexander Chernikov for analysing the problem.
parent af454f9b

proto/bgp/packets.c

+4 −0
Original line number Diff line number Diff line
@@ -999,7 +999,11 @@ bgp_rte_update(struct bgp_proto *p, ip_addr prefix, int pxlen, 
  if (!*a)

    {

      a0->src = *src;



      /* Workaround for rta_lookup() breaking eattrs */

      ea_list *ea = a0->eattrs;

      *a = rta_lookup(a0);

      a0->eattrs = ea;

    }



  net *n = net_get(p->p.table, prefix, pxlen);

CRA Git | Maintained and supported by SUSTech CRA and CCSE