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

fixed a bug for str case conversion before str compare #1209

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
1 participant
@jeswinMathai

jeswinMathai commented May 7, 2018

without conversion, the small case "md5" string fails to match, condition fails and the result variable remains uninitialized, due to which "attempt to index a nil value" error is thrown on line 812. the authorization string is converted to uppercase before comparison everywhere except on line 807

fixed a bug for str conversion before str compare
without conversion, the small case "md5" string fails to match and result variable remains uninitialized, "attempt to index a nil value" error is thrown on line 812. At all rest of the places the string is converted to uppercase before comparision except on line 807

@nmap-bot nmap-bot closed this in 670e24d May 14, 2018

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