proxmox-mirrors/mirror_frr
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_frr synced 2025-08-07 06:03:10 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

bgpd: fix calculation of update-group hash with maximum-prefix-out

Browse Source 
Take into account the maximum-prefix-out value when calculating the
update-group hash.

Fixes: fde246e835 ("bgpd: Add an option to limit outgoing prefixes")
Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
This commit is contained in:
Louis Scalbert 2021-10-20 16:15:22 +02:00
parent c748ab2567
commit e02672605b
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
bgpd/bgp_updgrp.c
View File

@ -306,6 +306,7 @@ static void *updgrp_hash_alloc(void *p)
* 14. encoding both global and link-local nexthop? * 14. encoding both global and link-local nexthop?
* 15. If peer is configured to be a lonesoul, peer ip address * 15. If peer is configured to be a lonesoul, peer ip address
* 16. Local-as should match, if configured. * 16. Local-as should match, if configured.
* 17. maximum-prefix-out
* ) * )
*/ */
static unsigned int updgrp_hash_key_make(const void *p) static unsigned int updgrp_hash_key_make(const void *p)
@ -340,6 +341,7 @@ static unsigned int updgrp_hash_key_make(const void *p)
key = jhash_1word(peer->v_routeadv, key); key = jhash_1word(peer->v_routeadv, key);
key = jhash_1word(peer->change_local_as, key); key = jhash_1word(peer->change_local_as, key);
key = jhash_1word(peer->max_packet_size, key); key = jhash_1word(peer->max_packet_size, key);
key = jhash_1word(peer->pmax_out[afi][safi], key);
if (peer->group) if (peer->group)
key = jhash_1word(jhash(peer->group->name, key = jhash_1word(jhash(peer->group->name,
@ -453,6 +455,9 @@ static bool updgrp_hash_cmp(const void *p1, const void *p2)
if (pe1->change_local_as != pe2->change_local_as) if (pe1->change_local_as != pe2->change_local_as)
return false; return false;
if (pe1->pmax_out[afi][safi] != pe2->pmax_out[afi][safi])
return false;
/* flags like route reflector client */ /* flags like route reflector client */
if ((flags1 & PEER_UPDGRP_AF_FLAGS) != (flags2 & PEER_UPDGRP_AF_FLAGS)) if ((flags1 & PEER_UPDGRP_AF_FLAGS) != (flags2 & PEER_UPDGRP_AF_FLAGS))
return false; return false;