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

crypto: Add more keylen sanity checks in pbkdf2 #3029

Closed
wants to merge 1 commit into from
Closed

crypto: Add more keylen sanity checks in pbkdf2 #3029

wants to merge 1 commit into from

Commits on Sep 23, 2015

  1. crypto: add more keylen sanity checks in pbkdf2 

    issue nodejs#2987 makes the point that crypto.pbkdf2 should not fail silently
and accept invalid but numeric values like NaN and Infinity. We already
check if the keylen is lower than 0, so extending that to NaN and
Infinity should make sense.

Fixes: nodejs#2987
    Johann committed Sep 23, 2015
    Configuration menu
    Copy the full SHA
    5550518 View commit details
    Browse the repository at this point in the history