diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2018-04-11 14:56:44 +0200 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2018-04-25 18:39:17 +0200 |
| commit | d0fa84c228262e18d14207af4cb7b4e0a8ffdafb (patch) | |
| tree | 034e7aee0a5eb9dc1911a698919e77396a2a574c /tests/bgpd/test_mp_attr.c | |
| parent | bgpd: add import vrf route-map command (diff) | |
| download | frr-d0fa84c228262e18d14207af4cb7b4e0a8ffdafb.tar.xz frr-d0fa84c228262e18d14207af4cb7b4e0a8ffdafb.zip | |
tests: Fix crash in test_mp_attr
Some recent changes in BGP now require that the
peer's nexthop have a valid ifp when we are looking
at:
case BGP_ATTR_NHLEN_IPV6_GLOBAL_AND_LL:
case BGP_ATTR_NHLEN_VPNV6_GLOBAL_AND_LL:
This assumption makes sense for this type of Nexthop Attribute.
So for the test let's jimmy up a `fake` enough interface pointer
so that the actual test we can focus on what we are actually
testing.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'tests/bgpd/test_mp_attr.c')
| -rw-r--r-- | tests/bgpd/test_mp_attr.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/tests/bgpd/test_mp_attr.c b/tests/bgpd/test_mp_attr.c index 34c35cfcc..9e0cb1f5c 100644 --- a/tests/bgpd/test_mp_attr.c +++ b/tests/bgpd/test_mp_attr.c @@ -1033,6 +1033,7 @@ static as_t asn = 100; int main(void) { + struct interface ifp; struct peer *peer; int i, j; @@ -1065,6 +1066,9 @@ int main(void) peer->status = Established; peer->curr = stream_new(BGP_MAX_PACKET_SIZE); + ifp.ifindex = 0; + peer->nexthop.ifp = &ifp; + for (i = AFI_IP; i < AFI_MAX; i++) for (j = SAFI_UNICAST; j < SAFI_MAX; j++) { peer->afc[i][j] = 1; |