]> git.ipfire.org Git - thirdparty/bird.git/commit
Filter: make bgpmask literals real constructors
authorJan Maria Matejka <mq@ucw.cz>
Wed, 28 Feb 2018 15:57:50 +0000 (16:57 +0100)
committerJan Maria Matejka <mq@ucw.cz>
Wed, 14 Mar 2018 10:34:29 +0000 (11:34 +0100)
commite8bc64e308586b6502090da2775af84cd760ed0d
treefd111a1534336be4d3fa115ed6fd5688e6be43d3
parent74bfd2f97c0a95b6fb73a67d9334e54a90695c58
Filter: make bgpmask literals real constructors

The bgpmask literals can include expressions. This is OK but they have
to be interpreted as soon as the code is run, not in the time the code
is used as value.

This led to strange behavior like rewriting bgpmasks when they shan't
be rewritten:

function mask_generator(int as)
{
return [= * as * =];
}

function another()
bgpmask m1;
bgpmask m2;
{
m1 = mask_generator(10);
m2 = mask_generator(20);
if (m1 == m2) {
print("strange"); # this would happen
}
}

Moreover, sending this to CLI would cause stack overflow and knock down the
whole BIRD, as soon as there is at least one route to execute the given
filter on.

show route filter bgpmask mmm; bgppath ppp; { ppp = +empty+; mmm = [= (ppp ~ mmm) =]; print(mmm); accept; }

The magic match operator (~) inside the bgpmask literal would try to
resolve mmm, which points to the same bgpmask so it would resolve
itself, call the magic match operator and vice versa.

After this patch, the bgpmask literal will get resolved as soon as it's
assigned to mmm and it also will return a type error as bool is not
convertible to ASN in BIRD.
filter/config.Y
filter/filter.c
filter/filter.h
nest/a-path.c