Commit d72a0ac2 authored by Ondrej Zajicek's avatar Ondrej Zajicek
Browse files

Fixes serious bug in route attribute handing. 

ea_same() sometimes returns true for different route attributes,
which caused that hash table in BGP does not work correctly and
some routes were sent with different attributes.
parent 80f0d676

nest/rt-attr.c

+1 −1
Original line number Diff line number Diff line
@@ -310,7 +310,7 @@ ea_same(ea_list *x, ea_list *y) 
	  a->flags != b->flags ||

	  a->type != b->type ||

	  ((a->type & EAF_EMBEDDED) ? a->u.data != b->u.data :

	   (a->u.ptr->length != b->u.ptr->length || memcmp(a->u.ptr, b->u.ptr, a->u.ptr->length))))

	   (a->u.ptr->length != b->u.ptr->length || memcmp(a->u.ptr->data, b->u.ptr->data, a->u.ptr->length))))

	return 0;

    }

  return 1;

CRA Git | Maintained and supported by SUSTech CRA and CCSE