From: Matt Caswell Date: Wed, 19 Oct 2016 16:37:22 +0000 (+0100) Subject: Remove a stray TODO that has already been fixed X-Git-Tag: OpenSSL_1_1_1-pre1~3203 X-Git-Url: https://git.openssl.org/gitweb/?p=openssl.git;a=commitdiff_plain;h=699ae85915f83f91bf5d5af45dd4888217005461 Remove a stray TODO that has already been fixed Reviewed-by: Rich Salz --- diff --git a/ssl/bio_ssl.c b/ssl/bio_ssl.c index e2769e1d6a..596df240d3 100644 --- a/ssl/bio_ssl.c +++ b/ssl/bio_ssl.c @@ -103,12 +103,7 @@ static int ssl_read(BIO *b, char *buf, size_t size, size_t *readbytes) BIO_clear_retry_flags(b); - if (size > INT_MAX) - size = INT_MAX; - - ret = SSL_read(ssl, buf, size); - if (ret > 0) - *readbytes = ret; + ret = SSL_read_ex(ssl, buf, size, readbytes); switch (SSL_get_error(ssl, ret)) { case SSL_ERROR_NONE: diff --git a/ssl/record/ssl3_record.c b/ssl/record/ssl3_record.c index d58e5bcdb1..c7f0d987c3 100644 --- a/ssl/record/ssl3_record.c +++ b/ssl/record/ssl3_record.c @@ -1034,7 +1034,6 @@ int tls1_mac(SSL *ssl, SSL3_RECORD *rec, unsigned char *md, int send) * are hashing because that gives an attacker a timing-oracle. */ /* Final param == not SSLv3 */ - /* TODO(size_t): Convert this call */ if (ssl3_cbc_digest_record(mac_ctx, md, &md_size, header, rec->input,