From e3908501ca32e6629eed046f2ec44b66ecd1854b Mon Sep 17 00:00:00 2001 From: Matt Caswell Date: Fri, 23 Jun 2017 10:10:51 +0100 Subject: [PATCH] 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) --- test/constant_time_test.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) 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)); } -- 2.34.1