diff options
| author | Matt Caswell <matt@openssl.org> | 2017-06-23 10:10:51 +0100 |
|---|---|---|
| committer | Matt Caswell <matt@openssl.org> | 2017-06-23 17:22:38 +0100 |
| commit | e3908501ca32e6629eed046f2ec44b66ecd1854b (patch) | |
| tree | 1e2820be729fbba7e69bda90bbaa809983f030ce /test/constant_time_test.c | |
| parent | 019e47ce564e9d57ed85c4ebe0672518b6a349f5 (diff) | |
| download | openssl-new-e3908501ca32e6629eed046f2ec44b66ecd1854b.tar.gz | |
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 <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/3755)
Diffstat (limited to 'test/constant_time_test.c')
| -rw-r--r-- | test/constant_time_test.c | 4 |
1 files 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)); } |