Before importing a certificate, free any previous one. Bug 1648
authorJeremy Harris <jgh146exb@wizmail.org>
Sun, 21 Jun 2015 17:17:09 +0000 (18:17 +0100)
committerJeremy Harris <jgh146exb@wizmail.org>
Sun, 21 Jun 2015 17:17:09 +0000 (18:17 +0100)
Because the SSL libraries do not use Exim's heap management
this was a memory-leak in "exim -bp".

src/src/deliver.c
src/src/tlscert-gnu.c
src/src/tlscert-openssl.c

index ec030fe..543a618 100644 (file)
@@ -3135,15 +3135,17 @@ while (!done)
       break;
 
       case '2':
-      addr->peercert = NULL;
       if (*ptr)
        (void) tls_import_cert(ptr, &addr->peercert);
+      else
+       addr->peercert = NULL;
       break;
 
       case '3':
-      addr->ourcert = NULL;
       if (*ptr)
        (void) tls_import_cert(ptr, &addr->ourcert);
+      else
+       addr->ourcert = NULL;
       break;
 
 # ifndef DISABLE_OCSP
index dc290b8..40f49d3 100644 (file)
@@ -51,10 +51,14 @@ tls_import_cert(const uschar * buf, void ** cert)
 {
 void * reset_point = store_get(0);
 gnutls_datum_t datum;
-gnutls_x509_crt_t crt;
+gnutls_x509_crt_t crt = *(gnutls_x509_crt_t *)cert;
 int fail = 0;
 
-gnutls_global_init();
+if (crt)
+  gnutls_x509_crt_deinit(crt);
+else
+  gnutls_global_init();
+
 gnutls_x509_crt_init(&crt);
 
 datum.data = string_unprinting(US buf);
index 165a3cf..f2e482b 100644 (file)
@@ -55,9 +55,11 @@ tls_import_cert(const uschar * buf, void ** cert)
 void * reset_point = store_get(0);
 const uschar * cp = string_unprinting(US buf);
 BIO * bp;
-X509 * x;
+X509 * x = *(X509 **)cert;
 int fail = 0;
 
+if (x) X509_free(x);
+
 bp = BIO_new_mem_buf(US cp, -1);
 if (!(x = PEM_read_bio_X509(bp, NULL, 0, NULL)))
   {