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

Drop some explicit SSLKeyShare destructors.

Browse Source 
We zero out memory in |OPENSSL_free| already.

Change-Id: I84a0f3cdfadd4544c0fade1d3d727baa6496ffe5
Reviewed-on: https://boringssl-review.googlesource.com/c/33446
Commit-Queue: Adam Langley <agl@google.com>
Commit-Queue: David Benjamin <davidben@google.com>
Reviewed-by: David Benjamin <davidben@google.com>
This commit is contained in:
Adam Langley 2018-12-03 14:37:44 -08:00 committed by CQ bot account: commit-bot@chromium.org
parent 96b05ed487
commit e6ad7a027f
1 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ssl/ssl_key_share.cc
View File

@ -38,7 +38,6 @@ namespace {
class ECKeyShare : public SSLKeyShare {
public:
ECKeyShare(int nid, uint16_t group_id) : nid_(nid), group_id_(group_id) {}
~ECKeyShare() override {}
uint16_t GroupID() const override { return group_id_; }
@ -159,9 +158,6 @@ class ECKeyShare : public SSLKeyShare {
class X25519KeyShare : public SSLKeyShare {
public:
X25519KeyShare() {}
~X25519KeyShare() override {
OPENSSL_cleanse(private_key_, sizeof(private_key_));
}
uint16_t GroupID() const override { return SSL_CURVE_X25519; }