From cb0bc3d64ce45f7abd1b99f044f45dfee71fedf3 Mon Sep 17 00:00:00 2001 From: jsing Date: Tue, 24 Jan 2017 01:34:09 +0000 Subject: There is no point in zeroing fields that exist within a struct that is about to be explicit_bzero'd and freed. --- lib/libssl/ssl_sess.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'lib/libssl/ssl_sess.c') diff --git a/lib/libssl/ssl_sess.c b/lib/libssl/ssl_sess.c index f44fdcc4194..4033b1e8f1e 100644 --- a/lib/libssl/ssl_sess.c +++ b/lib/libssl/ssl_sess.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ssl_sess.c,v 1.63 2017/01/23 13:36:13 jsing Exp $ */ +/* $OpenBSD: ssl_sess.c,v 1.64 2017/01/24 01:34:09 jsing Exp $ */ /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) * All rights reserved. * @@ -706,9 +706,7 @@ SSL_SESSION_free(SSL_SESSION *ss) sk_SSL_CIPHER_free(ss->ciphers); free(ss->tlsext_hostname); free(ss->tlsext_tick); - ss->internal->tlsext_ecpointformatlist_length = 0; free(ss->internal->tlsext_ecpointformatlist); - ss->internal->tlsext_ellipticcurvelist_length = 0; free(ss->internal->tlsext_ellipticcurvelist); explicit_bzero(ss->internal, sizeof(*ss->internal)); -- cgit v1.2.3-59-g8ed1b