Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use l_lpm_lookup to query the policy id, and the maximum integer value is 2147483647. Is it normal? #632

Closed
ShawnLeung87 opened this issue Mar 1, 2023 · 2 comments · Fixed by #633
Closed

Use l_lpm_lookup to query the policy id, and the maximum integer value is 2147483647. Is it normal? #632

ShawnLeung87 opened this issue Mar 1, 2023 · 2 comments · Fixed by #633
Labels
bug
Milestone
Version 1.1

Comments

@ShawnLeung87
Copy link

In the current program version, lpm is replaced by the fib method to query the policy id. When the prefix does not exist, the maximum integer value 2147483647 will be returned. This situation will not occur in the old version of the lpm method
AltraMayor added a commit that referenced this issue Mar 1, 2023
@AltraMayor
gt: fix l_lpm_lookup() and l_lpm6_lookup()
8002687 
This commit makes l_lpm_lookup() and l_lpm6_lookup() report
error codes.

This commit closes #632.
@AltraMayor AltraMayor mentioned this issue Mar 1, 2023
Merged
@AltraMayor AltraMayor added the bug label Mar 1, 2023
@AltraMayor AltraMayor added this to the Version 1.1 milestone Mar 1, 2023
@AltraMayor
Copy link
Owner

Thank you for reporting this bug.

Could you test pull request #633 to make sure it solves this issue?

@ShawnLeung87
Copy link
Author

ShawnLeung87 commented Mar 2, 2023 •
edited

The return value is already normal, the same as the return value of the previous version

Thank you for reporting this bug.

Could you test pull request #633 to make sure it solves this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
Version 1.1
Development

Successfully merging a pull request may close this issue.

gt: fix l_lpm_lookup() and l_lpm6_lookup() AltraMayor/gatekeeper
2 participants
@AltraMayor @ShawnLeung87