Honor standard communities (no_export, no_advertise, no_export_subconfed)
when exporting routes.
This commit is contained in:
parent
56a2bed46b
commit
bd2d8190dd
2 changed files with 43 additions and 4 deletions
|
@ -297,6 +297,42 @@ bgp_new_bucket(struct bgp_proto *p, ea_list *new, unsigned hash)
|
||||||
return b;
|
return b;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static int
|
||||||
|
bgp_export_check(struct bgp_proto *p, ea_list *new)
|
||||||
|
{
|
||||||
|
eattr *a;
|
||||||
|
struct adata *d;
|
||||||
|
|
||||||
|
/* Check if next hop is valid */
|
||||||
|
a = ea_find(new, EA_CODE(EAP_BGP, BA_NEXT_HOP));
|
||||||
|
if (!a || ipa_equal(p->next_hop, *(ip_addr *)a->u.ptr))
|
||||||
|
{
|
||||||
|
DBG("\tInvalid NEXT_HOP\n");
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* Check if we aren't forbidden to export the route by communities */
|
||||||
|
a = ea_find(new, EA_CODE(EAP_BGP, BA_COMMUNITY));
|
||||||
|
if (a)
|
||||||
|
{
|
||||||
|
d = a->u.ptr;
|
||||||
|
if (int_set_contains(d, BGP_COMM_NO_ADVERTISE))
|
||||||
|
{
|
||||||
|
DBG("\tNO_ADVERTISE\n");
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
if (!p->is_internal &&
|
||||||
|
(int_set_contains(d, BGP_COMM_NO_EXPORT) ||
|
||||||
|
int_set_contains(d, BGP_COMM_NO_EXPORT_SUBCONFED)))
|
||||||
|
{
|
||||||
|
DBG("\tNO_EXPORT\n");
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return 1;
|
||||||
|
}
|
||||||
|
|
||||||
static struct bgp_bucket *
|
static struct bgp_bucket *
|
||||||
bgp_get_bucket(struct bgp_proto *p, ea_list *old, ea_list *tmp, int originate)
|
bgp_get_bucket(struct bgp_proto *p, ea_list *old, ea_list *tmp, int originate)
|
||||||
{
|
{
|
||||||
|
@ -375,10 +411,7 @@ bgp_get_bucket(struct bgp_proto *p, ea_list *old, ea_list *tmp, int originate)
|
||||||
return NULL;
|
return NULL;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Check if next hop is valid */
|
if (!bgp_export_check(p, new))
|
||||||
a = ea_find(new, EA_CODE(EAP_BGP, BA_NEXT_HOP));
|
|
||||||
ASSERT(a);
|
|
||||||
if (ipa_equal(p->next_hop, *(ip_addr *)a->u.ptr))
|
|
||||||
return NULL;
|
return NULL;
|
||||||
|
|
||||||
/* Create new bucket */
|
/* Create new bucket */
|
||||||
|
|
|
@ -151,4 +151,10 @@ int bgp_rx(struct birdsock *sk, int size);
|
||||||
#define BS_OPENCONFIRM 4
|
#define BS_OPENCONFIRM 4
|
||||||
#define BS_ESTABLISHED 5
|
#define BS_ESTABLISHED 5
|
||||||
|
|
||||||
|
/* Well-known communities */
|
||||||
|
|
||||||
|
#define BGP_COMM_NO_EXPORT 0xffffff01 /* Don't export outside local AS / confed. */
|
||||||
|
#define BGP_COMM_NO_ADVERTISE 0xffffff02 /* Don't export at all */
|
||||||
|
#define BGP_COMM_NO_EXPORT_SUBCONFED 0xffffff03 /* NO_EXPORT even in local confederation */
|
||||||
|
|
||||||
#endif
|
#endif
|
||||||
|
|
Loading…
Reference in a new issue