From d72a0ac2396471ffb2b3a580ee986f950e4d23ae Mon Sep 17 00:00:00 2001 From: Ondrej Zajicek Date: Fri, 22 May 2009 15:16:53 +0200 Subject: [PATCH] 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. --- nest/rt-attr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/nest/rt-attr.c b/nest/rt-attr.c index e79cba1f..2318f801 100644 --- a/nest/rt-attr.c +++ b/nest/rt-attr.c @@ -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;