nodejs
diff --git a/Collapse file
‎src/node_crypto.cc‎
Copy file name to clipboardExpand all lines: src/node_crypto.cc
+5-3Lines changed: 5 additions & 3 deletions b/Collapse file
‎src/node_crypto.cc‎
Copy file name to clipboardExpand all lines: src/node_crypto.cc
+5-3Lines changed: 5 additions & 3 deletions
diff --git a/Collapse file
‎test/parallel/test-crypto-key-objects.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-crypto-key-objects.js
+7Lines changed: 7 additions & 0 deletions b/Collapse file
‎test/parallel/test-crypto-key-objects.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-crypto-key-objects.js
+7Lines changed: 7 additions & 0 deletions
@@ -2669,7 +2669,7 @@ static bool IsSupportedAuthenticatedMode(const EVP_CIPHER_CTX* ctx) {
 template <typename T>
 static T* MallocOpenSSL(size_t count) {
   void* mem = OPENSSL_malloc(MultiplyWithOverflowCheck(count, sizeof(T)));
-  CHECK_NOT_NULL(mem);
+  CHECK_IMPLIES(mem == nullptr, count == 0);
   return static_cast<T*>(mem);
 }
 
@@ -2827,7 +2827,8 @@ static EVPKeyPointer ParsePrivateKey(const PrivateKeyEncodingConfig& config,
 
   if (config.format_ == kKeyFormatPEM) {
     BIOPointer bio(BIO_new_mem_buf(key, key_len));
-    CHECK(bio);
+    if (!bio)
+      return pkey;
 
     char* pass = const_cast<char*>(config.passphrase_.get());
     pkey.reset(PEM_read_bio_PrivateKey(bio.get(),
@@ -2842,7 +2843,8 @@ static EVPKeyPointer ParsePrivateKey(const PrivateKeyEncodingConfig& config,
       pkey.reset(d2i_PrivateKey(EVP_PKEY_RSA, nullptr, &p, key_len));
     } else if (config.type_.ToChecked() == kKeyEncodingPKCS8) {
       BIOPointer bio(BIO_new_mem_buf(key, key_len));
-      CHECK(bio);
+      if (!bio)
+        return pkey;
       char* pass = const_cast<char*>(config.passphrase_.get());
       pkey.reset(d2i_PKCS8PrivateKey_bio(bio.get(),
                                          nullptr,
 
@@ -105,3 +105,10 @@ const privatePem = fixtures.readSync('test_rsa_privkey.pem', 'ascii');
     }
   }
 }
+
+{
+  // This should not cause a crash: https://github.com/nodejs/node/issues/25247
+  assert.throws(() => {
+    createPrivateKey({ key: '' });
+  }, /null/);
+}
Original file line number	Diff line number	Diff line change
`@@ -105,3 +105,10 @@ const privatePem = fixtures.readSync('test_rsa_privkey.pem', 'ascii');`
`105`	`105`	`}`
`106`	`106`	`}`
`107`	`107`	`}`
	`108`	`+`
	`109`	`+{`
	`110`	`+ // This should not cause a crash: https://github.com/nodejs/node/issues/25247`
	`111`	`+ assert.throws(() => {`
	`112`	`+ createPrivateKey({ key: '' });`
	`113`	`+ }, /null/);`
	`114`	`+}`