| commit | e12c4378e94d4332a5878ae2bcfc9f3f5f264f2b | [log] [tgz] |
|---|---|---|
| author | David Benjamin <davidben@chromium.org> | Wed Apr 01 14:16:09 2015 -0400 |
| committer | Adam Langley <agl@google.com> | Wed Apr 01 18:22:23 2015 +0000 |
| tree | 56c8743589bd858b971ae52b2692685e7477af38 | |
| parent | 5a3162a521f3e3fa038553a7940b4edd173ead23 [diff] |
Fix leak in ssl_test. SSL_CIPHER_get_rfc_name still returns an allocated string. Change-Id: Ie2f14626c1ff22d0ea613b22439b7de5c04c9062 Reviewed-on: https://boringssl-review.googlesource.com/4190 Reviewed-by: Adam Langley <agl@google.com>
diff --git a/ssl/ssl_test.cc b/ssl/ssl_test.cc index d077810..22018bb 100644 --- a/ssl/ssl_test.cc +++ b/ssl/ssl_test.cc
@@ -413,7 +413,9 @@ if (cipher == NULL) { return false; } - out->assign(SSL_CIPHER_get_rfc_name(cipher)); + char *rfc_name = SSL_CIPHER_get_rfc_name(cipher); + out->assign(rfc_name); + OPENSSL_free(rfc_name); return true; }