Skip to content

Commit 4e98e4e

Browse files
committed
src: switch crypto APIs to use Maybe<void>
nodejs/node#54775
1 parent af714ec commit 4e98e4e

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

‎patches/node/fix_handle_boringssl_and_openssl_incompatibilities.patch‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -297,12 +297,12 @@ index 23b2b8c56dec8ac600b8f14b78d9e80b7fa3ed3b..e7a8fe4181542252d9142ea9460cacc5
297297
if (params->saltLength != nullptr) {
298298
- if (ASN1_INTEGER_get_int64(&salt_length, params->saltLength) != 1) {
299299
- ThrowCryptoError(env, ERR_get_error(), "ASN1_INTEGER_get_in64 error");
300-
- return Nothing<bool>();
300+
- return Nothing<void>();
301301
- }
302302
+ // TODO(codebytere): Upstream a shim to BoringSSL?
303303
+ // if (ASN1_INTEGER_get_int64(&salt_length, params->saltLength) != 1) {
304304
+ // ThrowCryptoError(env, ERR_get_error(), "ASN1_INTEGER_get_in64 error");
305-
+ // return Nothing<bool>();
305+
+ // return Nothing<void>();
306306
+ // }
307307
}
308308

0 commit comments

Comments
 (0)