From: Matt Caswell Date: Fri, 23 Jun 2017 09:10:51 +0000 (+0100) Subject: Fix the constant time 64 test X-Git-Tag: OpenSSL_1_1_1-pre1~1215 X-Git-Url: https://git.openssl.org/?p=openssl.git;a=commitdiff_plain;h=e3908501ca32e6629eed046f2ec44b66ecd1854b;hp=019e47ce564e9d57ed85c4ebe0672518b6a349f5 Fix the constant time 64 test We were adding more tests than we had data for due to use of sizeof instead of OSSL_NELEM. I also changed the 8 bit tests for consistency, although they were already working. Reviewed-by: Rich Salz (Merged from https://github.com/openssl/openssl/pull/3755) --- diff --git a/test/constant_time_test.c b/test/constant_time_test.c index 422c04ca27..fdd4075c77 100644 --- a/test/constant_time_test.c +++ b/test/constant_time_test.c @@ -332,6 +332,6 @@ void register_tests(void) ADD_TEST(test_sizeofs); ADD_ALL_TESTS(test_binops, OSSL_NELEM(test_values)); ADD_ALL_TESTS(test_signed, OSSL_NELEM(signed_test_values)); - ADD_ALL_TESTS(test_8values, sizeof(test_values_8)); - ADD_ALL_TESTS(test_64values, sizeof(test_values_64)); + ADD_ALL_TESTS(test_8values, OSSL_NELEM(test_values_8)); + ADD_ALL_TESTS(test_64values, OSSL_NELEM(test_values_64)); }