Commit ff17b39
crypto: fix public key encryption internals
Coincidentally, the old version works as well since the padding
parameter is never null, but it is semantically incorrect.
PR-URL: #22780
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>1 parent 6786730 commit ff17b39
1 file changed
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -3890,7 +3890,7 @@ void PublicKeyCipher::Cipher(const FunctionCallbackInfo<Value>& args) { | |||
| 3890 | 3890 | bool r = Cipher<operation, EVP_PKEY_cipher_init, EVP_PKEY_cipher>( | |
| 3891 | 3891 | kbuf, | |
| 3892 | 3892 | klen, | |
| 3893 | - args.Length() >= 3 && !args[2]->IsNull() ? *passphrase : nullptr, | ||
| 3893 | + args.Length() >= 4 && !args[3]->IsNull() ? *passphrase : nullptr, | ||
| 3894 | 3894 | padding, | |
| 3895 | 3895 | reinterpret_cast<const unsigned char*>(buf), | |
| 3896 | 3896 | len, | |
0 commit comments