From 42f926ece7ebfdb3ff896b93d362cdeb0cd00d83 Mon Sep 17 00:00:00 2001 From: Fedor Indutny Date: Thu, 27 Jun 2013 19:53:29 +0200 Subject: [PATCH] crypto: fix memory leak in LoadPKCS12 X509_STORE_add_cert increment reference of passed `x509` cert, `X509_free` must be called to avoid memory leak. This is a back-port of commit c1db1ec from the master branch. --- src/node_crypto.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/src/node_crypto.cc b/src/node_crypto.cc index d3a7dcd76..b1140b866 100644 --- a/src/node_crypto.cc +++ b/src/node_crypto.cc @@ -640,6 +640,7 @@ Handle SecureContext::LoadPKCS12(const Arguments& args) { X509_STORE_add_cert(sc->ca_store_, x509); SSL_CTX_add_client_CA(sc->ctx_, x509); + X509_free(x509); } EVP_PKEY_free(pkey);