From d6a8a5a54d02cc05534c170fa76c3db4db08dd13 Mon Sep 17 00:00:00 2001 From: David Benjamin Date: Mon, 24 Jul 2017 17:46:44 -0400 Subject: [PATCH] Remove obsolete TODOs. Looks like they're using the pool now. Change-Id: Ieeb1cacb9cb039d35ff091bc9742262f0fc5b146 Reviewed-on: https://boringssl-review.googlesource.com/18364 Reviewed-by: Adam Langley --- ssl/ssl_asn1.cc | 2 -- 1 file changed, 2 deletions(-) diff --git a/ssl/ssl_asn1.cc b/ssl/ssl_asn1.cc index d360a7aa0..a1c53d5bd 100644 --- a/ssl/ssl_asn1.cc +++ b/ssl/ssl_asn1.cc @@ -678,7 +678,6 @@ UniquePtr SSL_SESSION_parse(CBS *cbs, } if (has_peer) { - /* TODO(agl): this should use the |SSL_CTX|'s pool. */ CRYPTO_BUFFER *buffer = CRYPTO_BUFFER_new_from_CBS(&peer, pool); if (buffer == NULL || !sk_CRYPTO_BUFFER_push(ret->certs, buffer)) { @@ -696,7 +695,6 @@ UniquePtr SSL_SESSION_parse(CBS *cbs, return nullptr; } - /* TODO(agl): this should use the |SSL_CTX|'s pool. */ CRYPTO_BUFFER *buffer = CRYPTO_BUFFER_new_from_CBS(&cert, pool); if (buffer == NULL || !sk_CRYPTO_BUFFER_push(ret->certs, buffer)) {