bgpd: fix bug in ecommunity_match

The offset used to compare extended communities should be increased by
steps of ECOMMUNITY_SIZE and not one by one.

diff --git a/bgpd/bgp_ecommunity.c b/bgpd/bgp_ecommunity.c
index e55ce3528c1c5653848219b0bba6411ac2e51010..926e2650a234bf3f95e6f1523daf613715241b62 100644 (file)
--- a/bgpd/bgp_ecommunity.c
+++ b/bgpd/bgp_ecommunity.c
@@ -770,7 +770,9 @@ ecommunity_match (const struct ecommunity *ecom1,
   /* Every community on com2 needs to be on com1 for this to match */
   while (i < ecom1->size && j < ecom2->size)
     {
-      if (memcmp (ecom1->val + i, ecom2->val + j, ECOMMUNITY_SIZE) == 0)
+      if (memcmp (ecom1->val + i * ECOMMUNITY_SIZE,
+                  ecom2->val + j * ECOMMUNITY_SIZE,
+                  ECOMMUNITY_SIZE) == 0)
         j++;
       i++;
     }
@@ -780,4 +782,3 @@ ecommunity_match (const struct ecommunity *ecom1,
   else
     return 0;
 }
-