crypto: make pbkdf2 use checkIsArrayBufferView

This commit updates pbkdf2 to use checkIsArrayBufferView from
internal/crypto/util.

PR-URL: #20251
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Tiancheng "Timothy" Gu <[email protected]>

Author: danbev

lib/internal/crypto/pbkdf2.js

@@ -7,10 +7,10 @@ const {
   ERR_OUT_OF_RANGE
 } = require('internal/errors').codes;
 const {
+  checkIsArrayBufferView,
   getDefaultEncoding,
   toBuf
 } = require('internal/crypto/util');
-const { isArrayBufferView } = require('internal/util/types');
 const {
   PBKDF2
 } = process.binding('crypto');
@@ -39,19 +39,8 @@ function _pbkdf2(password, salt, iterations, keylen, digest, callback) {
   if (digest !== null && typeof digest !== 'string')
     throw new ERR_INVALID_ARG_TYPE('digest', ['string', 'null'], digest);
 
-  password = toBuf(password);
-  salt = toBuf(salt);
-
-  if (!isArrayBufferView(password)) {
-    throw new ERR_INVALID_ARG_TYPE('password',
-                                   ['string', 'Buffer', 'TypedArray'],
-                                   password);
-  }
-
-  if (!isArrayBufferView(salt)) {
-    throw new ERR_INVALID_ARG_TYPE('salt',
-                                   ['string', 'Buffer', 'TypedArray'], salt);
-  }
+  password = checkIsArrayBufferView('password', toBuf(password));
+  salt = checkIsArrayBufferView('salt', toBuf(salt));
 
   if (typeof iterations !== 'number')
     throw new ERR_INVALID_ARG_TYPE('iterations', 'number', iterations);

test/parallel/test-crypto-pbkdf2.js

@@ -123,7 +123,8 @@ assert.throws(
   });
 
 [1, {}, [], true, undefined, null].forEach((input) => {
-  const msgPart2 = `Buffer, or TypedArray. Received type ${typeof input}`;
+  const msgPart2 = 'Buffer, TypedArray, or DataView.' +
+                   ` Received type ${typeof input}`;
   assert.throws(
     () => crypto.pbkdf2(input, 'salt', 8, 8, 'sha256', common.mustNotCall()),
     {