From ff0426cc94df5e6dd25b8cfd6f9f7c840264a400 Mon Sep 17 00:00:00 2001 From: Emilia Kasper Date: Thu, 13 Jul 2017 18:30:56 +0200 Subject: [PATCH] Remove resolved TODO Fixed in 5b8fa431ae8eb5a18ba913494119e394230d4b70 [ci skip] Reviewed-by: Richard Levitte Reviewed-by: Matt Caswell (Merged from https://github.com/openssl/openssl/pull/3924) --- crypto/rsa/rsa_pk1.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/crypto/rsa/rsa_pk1.c b/crypto/rsa/rsa_pk1.c index efb16a02e6..5bc91c46d5 100644 --- a/crypto/rsa/rsa_pk1.c +++ b/crypto/rsa/rsa_pk1.c @@ -226,8 +226,6 @@ int RSA_padding_check_PKCS1_type_2(unsigned char *to, int tlen, * We can't continue in constant-time because we need to copy the result * and we cannot fake its length. This unavoidably leaks timing * information at the API boundary. - * TODO(emilia): this could be addressed at the call site, - * see BoringSSL commit 0aa0767340baf925bda4804882aab0cb974b2d26. */ if (!good) { mlen = -1; -- 2.25.1