diff options
| author |   jsing <jsing@openbsd.org> | 2014-06-07 14:37:35 +0000 |
|---|---|---|
| committer | jsing <jsing@openbsd.org> | 2014-06-07 14:37:35 +0000 |
| commit | e51a8815943aa06057e71825213ca317d2e1ff83 (patch) | |
| tree | dc6719bfc58f6652375ab4099fb2173779117b07 /lib/libssl/s3_lib.c | |
| parent | BIO_free has an implicit NULL check, so do not bother checking for NULL (diff) | |
| download | wireguard-openbsd-e51a8815943aa06057e71825213ca317d2e1ff83.tar.xz wireguard-openbsd-e51a8815943aa06057e71825213ca317d2e1ff83.zip | |
ssl3_free_digest_list() has its own NULL check.
Diffstat (limited to 'lib/libssl/s3_lib.c')
| -rw-r--r-- | lib/libssl/s3_lib.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/lib/libssl/s3_lib.c b/lib/libssl/s3_lib.c index 6165b2a1207..3058b66a0d9 100644 --- a/lib/libssl/s3_lib.c +++ b/lib/libssl/s3_lib.c @@ -2338,8 +2338,7 @@ ssl3_free(SSL *s) if (s->s3->tmp.ca_names != NULL) sk_X509_NAME_pop_free(s->s3->tmp.ca_names, X509_NAME_free); BIO_free(s->s3->handshake_buffer); - if (s->s3->handshake_dgst) - ssl3_free_digest_list(s); + ssl3_free_digest_list(s); OPENSSL_cleanse(s->s3, sizeof *s->s3); free(s->s3); s->s3 = NULL; @@ -2382,9 +2381,8 @@ ssl3_clear(SSL *s) BIO_free(s->s3->handshake_buffer); s->s3->handshake_buffer = NULL; } - if (s->s3->handshake_dgst) { - ssl3_free_digest_list(s); - } + ssl3_free_digest_list(s); + memset(s->s3, 0, sizeof *s->s3); s->s3->rbuf.buf = rp; s->s3->wbuf.buf = wp; |