Correct tls_config_clear_keys() behaviour.

Previously this incorrectly called tls_keypair_clear(), which results in
the private key being cleared, along with the certificate, OCSP staple and
pubkey hash. This breaks OCSP stapling if tls_config_clear_keys() is called
following tls_configure(), as is done by httpd.

Fix this by calling tls_keypair_clear_key() so that only the private key is
cleared, leaving the other public data untouched. While here, remove
tls_keypair_clear() and fold the necessary parts into tls_keypair_free().

ok beck@

1 files changed, 14 insertions, 23 deletions

diff --git a/src/lib/libtls/tls_keypair.c b/src/lib/libtls/tls_keypair.c
index 03e7f4ad76..a98e5c2e7e 100644
--- a/src/lib/libtls/tls_keypair.c
+++ b/src/lib/libtls/tls_keypair.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: tls_keypair.c,v 1.5 2018/02/10 04:57:35 jsing Exp $ */
+/* $OpenBSD: tls_keypair.c,v 1.6 2018/04/07 16:35:34 jsing Exp $ */
 /*
  * Copyright (c) 2014 Joel Sing <jsing@openbsd.org>
  *
@@ -29,14 +29,6 @@ tls_keypair_new(void)
         return calloc(1, sizeof(struct tls_keypair));
 }
 
-static void
-tls_keypair_clear_key(struct tls_keypair *keypair)
-{
-        freezero(keypair->key_mem, keypair->key_len);
-        keypair->key_mem = NULL;
-        keypair->key_len = 0;
-}
-
 static int
 tls_keypair_pubkey_hash(struct tls_keypair *keypair, struct tls_error *error)
 {
@@ -64,6 +56,14 @@ tls_keypair_pubkey_hash(struct tls_keypair *keypair, struct tls_error *error)
         return (rv);
 }
 
+void
+tls_keypair_clear_key(struct tls_keypair *keypair)
+{
+        freezero(keypair->key_mem, keypair->key_len);
+        keypair->key_mem = NULL;
+        keypair->key_len = 0;
+}
+
 int
 tls_keypair_set_cert_file(struct tls_keypair *keypair, struct tls_error *error,
     const char *cert_file)
@@ -117,25 +117,16 @@ tls_keypair_set_ocsp_staple_mem(struct tls_keypair *keypair,
 }
 
 void
-tls_keypair_clear(struct tls_keypair *keypair)
-{
-        struct tls_error error;
-
-        tls_keypair_set_cert_mem(keypair, &error, NULL, 0);
-        tls_keypair_set_key_mem(keypair, &error, NULL, 0);
-        tls_keypair_set_ocsp_staple_mem(keypair, &error, NULL, 0);
-
-        free(keypair->pubkey_hash);
-        keypair->pubkey_hash = NULL;
-}
-
-void
 tls_keypair_free(struct tls_keypair *keypair)
 {
         if (keypair == NULL)
                 return;
 
-        tls_keypair_clear(keypair);
+        tls_keypair_clear_key(keypair);
+
+        free(keypair->cert_mem);
+        free(keypair->ocsp_staple);
+        free(keypair->pubkey_hash);
 
         free(keypair);
 }