Skip to content

Update static_zebra.c#31

Open
pettershao-ragilenetworks wants to merge 1 commit intosonic-net:frr/8.2from
pettershao-ragilenetworks:pettershao-ragilenetworks-patch-static-route-nexthop-num
Open

Update static_zebra.c#31
pettershao-ragilenetworks wants to merge 1 commit intosonic-net:frr/8.2from
pettershao-ragilenetworks:pettershao-ragilenetworks-patch-static-route-nexthop-num

Conversation

@pettershao-ragilenetworks

Limit the number of next hops for static routes

@pettershao-ragilenetworks
Copy link
Author

Why I did it
The declared array size( struct zapi_nexthop nexthops[MULTIPATH_NUM] ) is MULTIPATH_NUM. If the configuration exceeds MULTIPATH_NUM, the BGP container will crash

How I did it
Limit the number of nexthops for static route

How to verify it
Configure next hops over MULTIPATH_NUM for static route, and make sure nexthops are valid.

@pettershao-ragilenetworks
Copy link
Author

@ton31337 @donaldsharp help check and merge this, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant