Skip to content

Commit a221017

Browse files
codebytereMylesBorins
authored andcommitted
crypto: cast oaepLabel to unsigned char*
OpenSSL uses a macro without typechecking; since C++ does not implicitly cast void* this is needed to conform Node.js to the OpenSSL documentation. PR-URL: #30917 Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
1 parent 663a6b4 commit a221017

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/node_crypto.cc

+3-2
Original file line numberDiff line numberDiff line change
@@ -5510,8 +5510,9 @@ bool PublicKeyCipher::Cipher(Environment* env,
55105510
// OpenSSL takes ownership of the label, so we need to create a copy.
55115511
void* label = OPENSSL_memdup(oaep_label, oaep_label_len);
55125512
CHECK_NOT_NULL(label);
5513-
if (0 >= EVP_PKEY_CTX_set0_rsa_oaep_label(ctx.get(), label,
5514-
oaep_label_len)) {
5513+
if (0 >= EVP_PKEY_CTX_set0_rsa_oaep_label(ctx.get(),
5514+
reinterpret_cast<unsigned char*>(label),
5515+
oaep_label_len)) {
55155516
OPENSSL_free(label);
55165517
return false;
55175518
}

0 commit comments

Comments
 (0)