From ec6d28114fda2805a2b85ce5981137fa97be2f8a Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Fri, 15 Jul 2022 21:04:37 +0000 Subject: [PATCH] crypto: remove unneeded guard createCFRGRaw() will always return an object. The error for an undefined return value will never occur. --- lib/internal/crypto/cfrg.js | 2 -- lib/internal/crypto/keys.js | 2 +- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/lib/internal/crypto/cfrg.js b/lib/internal/crypto/cfrg.js index 6910aa84134a57..ba39bc117b89ff 100644 --- a/lib/internal/crypto/cfrg.js +++ b/lib/internal/crypto/cfrg.js @@ -291,8 +291,6 @@ async function cfrgImportKey( case 'raw': { verifyAcceptableCfrgKeyUse(name, 'public', usagesSet); keyObject = createCFRGRawKey(name, keyData, true); - if (keyObject === undefined) - throw lazyDOMException('Unable to import CFRG key', 'OperationError'); break; } } diff --git a/lib/internal/crypto/keys.js b/lib/internal/crypto/keys.js index 5d00727813250a..881cf00f051fcd 100644 --- a/lib/internal/crypto/keys.js +++ b/lib/internal/crypto/keys.js @@ -85,7 +85,7 @@ for (const m of [[kKeyEncodingPKCS1, 'pkcs1'], [kKeyEncodingPKCS8, 'pkcs8'], encodingNames[m[0]] = m[1]; // Creating the KeyObject class is a little complicated due to inheritance -// and that fact that KeyObjects should be transferrable between threads, +// and the fact that KeyObjects should be transferrable between threads, // which requires the KeyObject base class to be implemented in C++. // The creation requires a callback to make sure that the NativeKeyObject // base class cannot exist without the other KeyObject implementations.