[Fix] check parameters before the "no Promise" bailout

Author: ljharb

lib/async.js

@@ -88,6 +88,13 @@ module.exports = function (password, salt, iterations, keylen, digest, callback)
 		digest = undefined;
 	}
 
+	checkParameters(iterations, keylen);
+	password = toBuffer(password, defaultEncoding, 'Password');
+	salt = toBuffer(salt, defaultEncoding, 'Salt');
+	if (typeof callback !== 'function') {
+		throw new Error('No callback provided to pbkdf2');
+	}
+
 	digest = digest || 'sha1';
 	var algo = toBrowser[digest.toLowerCase()];
 
@@ -105,13 +112,6 @@ module.exports = function (password, salt, iterations, keylen, digest, callback)
 		return;
 	}
 
-	checkParameters(iterations, keylen);
-	password = toBuffer(password, defaultEncoding, 'Password');
-	salt = toBuffer(salt, defaultEncoding, 'Salt');
-	if (typeof callback !== 'function') {
-		throw new Error('No callback provided to pbkdf2');
-	}
-
 	resolvePromise(checkNative(algo).then(function (resp) {
 		if (resp) {
 			return browserPbkdf2(password, salt, iterations, keylen, algo);