openconfig / lemming

An OpenConfig reference device implementation
Apache License 2.0
31 stars 9 forks source link

Address infinite loop when recursively resolving routes. #460

Closed robshakir closed 3 months ago

robshakir commented 3 months ago
* (M) sysrib/{server,server_test,sysrib,sysrib_test}.go
   - Avoid infinite stack depth when a route recurses onto itself.
     Previously, looking up a next-hop or next-hop interface would
     not track the visited nodes, and hence infinite recursion could
     occur. This PR adds tracking of these changes into the RIB
     lookup to avoid this.
github-actions[bot] commented 3 months ago

Pull Request Test Coverage Report for Build 10175027886

Details


Files with Coverage Reduction New Missed Lines %
sysrib/static.go 1 90.41%
<!-- Total: 1 -->
Totals Coverage Status
Change from base Build 10101306638: 0.004%
Covered Lines: 18825
Relevant Lines: 658631

💛 - Coveralls