-
Notifications
You must be signed in to change notification settings - Fork 26
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
Support node 4 #23
Comments
Dear Hueniverse,
}; |
What does it have to do with this issue ? |
Not related to this issue , It was only enhancement . |
@msingh025 this function |
@mtharrison thanks a lot , we are on the same page, I have used it in our authentication module, before using it, i read and found, we are comparing strings char by char, I thought, I should break the loop on mismatched chars, so I added a extra line inside loop |
@msingh025 why should you break on mismatched chars? |
@mtharrison bcz , I do not want iterate char till end of string's length. if const ac and const bc are not equal then defiantly both string a and string b is not equal to each other . Then I should break loop at that point. |
That defeats the whole purpose of this function to be a fixed time comparison. If whether or not two strings are equal influences the running time of the function it is not running in fixed time and you might as well just be doing |
This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions. |
No description provided.
The text was updated successfully, but these errors were encountered: