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.

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

Prevent endless loops cause by incorrect SNMP implementations #51

Open
exhuma opened this issue Aug 9, 2018 · 0 comments
Open

Prevent endless loops cause by incorrect SNMP implementations #51

exhuma opened this issue Aug 9, 2018 · 0 comments

Comments

@exhuma
Copy link
Owner

exhuma commented Aug 9, 2018

I've run into a bug in a network device which does not behave correctly on GetNext requests. Instead of returning the next OID it will return the same OID as the one requested on some rare occasions.

This causes endless loops on walk and bulkwalk (and thus table requests as well). A check should be introduced that verifies that the OID received by getnext is different from the one passed in. A more precise check would be to check if the returned OID is "larger" than the one passed in.

These checks may have a considerable performance impact on larger walk operations, primarily the check if the OID is increasing. They should therefore be made optional.

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

No branches or pull requests

1 participant