crypto: fix leak in GetPeerCertificate

nodejs · Dec 3, 2012 · 51d5655 · 51d5655
1 parent 9d02bfb
commit 51d5655
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/src/node_crypto.cc b/src/node_crypto.cc
@@ -1532,6 +1532,15 @@ Handle<Value> Connection::GetPeerCertificate(const Arguments& args) {
         (void) BIO_reset(bio);
     }
 
+    if (pkey != NULL) {
+      EVP_PKEY_free(pkey);
+      pkey = NULL;
+    }
+    if (rsa != NULL) {
+      RSA_free(rsa);
+      rsa = NULL;
+    }
+
     ASN1_TIME_print(bio, X509_get_notBefore(peer_cert));
     BIO_get_mem_ptr(bio, &mem);
     info->Set(valid_from_symbol, String::New(mem->data, mem->length));