From 5b30e28eb45cb707afe73fa3d06d2ed9b56ff04f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tobias=20Nie=C3=9Fen?= Date: Mon, 14 Mar 2022 20:15:17 +0100 Subject: [PATCH] src: check EC_POINT_get_affine_coordinates result PR-URL: https://github.com/nodejs/node/pull/42304 Reviewed-By: Rich Trott Reviewed-By: James M Snell Reviewed-By: Anna Henningsen Reviewed-By: Darshan Sen --- src/crypto/crypto_ec.cc | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/crypto/crypto_ec.cc b/src/crypto/crypto_ec.cc index 7f1ba4dbff6036..2d738bb9b62e11 100644 --- a/src/crypto/crypto_ec.cc +++ b/src/crypto/crypto_ec.cc @@ -733,7 +733,11 @@ Maybe ExportJWKEcKey( BignumPointer x(BN_new()); BignumPointer y(BN_new()); - EC_POINT_get_affine_coordinates(group, pub, x.get(), y.get(), nullptr); + if (!EC_POINT_get_affine_coordinates(group, pub, x.get(), y.get(), nullptr)) { + ThrowCryptoError(env, ERR_get_error(), + "Failed to get elliptic-curve point coordinates"); + return Nothing(); + } if (target->Set( env->context(),