From cc744b9b26bfa433a029676f991bac6b5b095d5c Mon Sep 17 00:00:00 2001 From: David Benjamin Date: Sat, 16 Sep 2017 03:16:17 -0400 Subject: [PATCH] crypto: remove unnecessary SSLerr calls These are OpenSSL-internal APIs that are no longer accessible in 1.1.0 and weren't necessary. OpenSSL will push its own errors and, if it doesn't, the calling code would handle it anyway. PR-URL: https://github.com/nodejs/node/pull/16130 Reviewed-By: Ben Noordhuis Reviewed-By: Rod Vagg --- src/node_crypto.cc | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/node_crypto.cc b/src/node_crypto.cc index ecef5dd98333c5..ad0edcf1ce4994 100644 --- a/src/node_crypto.cc +++ b/src/node_crypto.cc @@ -668,7 +668,6 @@ int SSL_CTX_use_certificate_chain(SSL_CTX* ctx, x = PEM_read_bio_X509_AUX(in, nullptr, NoPasswordCallback, nullptr); if (x == nullptr) { - SSLerr(SSL_F_SSL_CTX_USE_CERTIFICATE_CHAIN_FILE, ERR_R_PEM_LIB); return 0; } @@ -679,7 +678,6 @@ int SSL_CTX_use_certificate_chain(SSL_CTX* ctx, // Read extra certs STACK_OF(X509)* extra_certs = sk_X509_new_null(); if (extra_certs == nullptr) { - SSLerr(SSL_F_SSL_CTX_USE_CERTIFICATE_CHAIN_FILE, ERR_R_MALLOC_FAILURE); goto done; }