crypto: handle initEDRaw pkey failure

nodejs · Sep 24, 2021 · b73c0a1 · b73c0a1
1 parent c7da13c
commit b73c0a1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 14 deletions.
diff --git a/lib/internal/crypto/ec.js b/lib/internal/crypto/ec.js
@@ -122,12 +122,11 @@ function createECRawKey(namedCurve, keyData, isPublic) {
       break;
   }
 
-  if (isPublic) {
-    handle.initEDRaw(namedCurve, keyData, kKeyTypePublic);
-    return new PublicKeyObject(handle);
+  const keyType = isPublic ? kKeyTypePublic : kKeyTypePrivate;
+  if (!handle.initEDRaw(namedCurve, keyData, keyType)) {
+    throw lazyDOMException('Failure to generate key object');
   }
 
-  handle.initEDRaw(namedCurve, keyData, kKeyTypePrivate);
   return new PrivateKeyObject(handle);
 }
 

diff --git a/lib/internal/crypto/keys.js b/lib/internal/crypto/keys.js
@@ -435,16 +435,10 @@ function getKeyObjectHandleFromJwk(key, ctx) {
     }
 
     const handle = new KeyObjectHandle();
-    if (isPublic) {
-      handle.initEDRaw(
-        `NODE-${key.crv.toUpperCase()}`,
-        keyData,
-        kKeyTypePublic);
-    } else {
-      handle.initEDRaw(
-        `NODE-${key.crv.toUpperCase()}`,
-        keyData,
-        kKeyTypePrivate);
+
+    const keyType = isPublic ? kKeyTypePublic : kKeyTypePrivate;
+    if (!handle.initEDRaw(`NODE-${key.crv.toUpperCase()}`, keyData, keyType)) {
+      throw new ERR_CRYPTO_INVALID_JWK();
     }
 
     return handle;