kris/boringssl
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Remove some redundant OPENSSL_cleanses.

Browse Source 
Anything heap-allocated is automatically cleansed.

Change-Id: I88034251bcba7a3e74c0d1ec887dff5a4c16fa8b
Reviewed-on: https://boringssl-review.googlesource.com/20664
Commit-Queue: Steven Valdez <svaldez@google.com>
Reviewed-by: Steven Valdez <svaldez@google.com>
CQ-Verified: CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org>
This commit is contained in:
David Benjamin 2017-09-21 02:58:03 -04:00 committed by CQ bot account: commit-bot@chromium.org
parent b1b76aee3c
commit b7e5b08a20
1 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ssl/handshake.cc
View File

@ -144,12 +144,6 @@ SSL_HANDSHAKE::SSL_HANDSHAKE(SSL *ssl_arg)
} }
SSL_HANDSHAKE::~SSL_HANDSHAKE() { SSL_HANDSHAKE::~SSL_HANDSHAKE() {
OPENSSL_cleanse(secret, sizeof(secret));
OPENSSL_cleanse(early_traffic_secret, sizeof(early_traffic_secret));
OPENSSL_cleanse(client_handshake_secret, sizeof(client_handshake_secret));
OPENSSL_cleanse(server_handshake_secret, sizeof(server_handshake_secret));
OPENSSL_cleanse(client_traffic_secret_0, sizeof(client_traffic_secret_0));
OPENSSL_cleanse(server_traffic_secret_0, sizeof(server_traffic_secret_0));
OPENSSL_free(cookie); OPENSSL_free(cookie);
OPENSSL_free(key_share_bytes); OPENSSL_free(key_share_bytes);
OPENSSL_free(ecdh_public_key); OPENSSL_free(ecdh_public_key);