From e64deda873027d7e0a841fda47e741aa55cf2498 Mon Sep 17 00:00:00 2001 From: Paul Elliott Date: Thu, 9 Sep 2021 14:07:23 +0100 Subject: [PATCH] Add missing check to multipart decrypt Ensure that the test actually does something, rather than skipping both parts, also add comment to this effect. Signed-off-by: Paul Elliott --- tests/suites/test_suite_psa_crypto.function | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/tests/suites/test_suite_psa_crypto.function b/tests/suites/test_suite_psa_crypto.function index a88108715..19b687e17 100644 --- a/tests/suites/test_suite_psa_crypto.function +++ b/tests/suites/test_suite_psa_crypto.function @@ -3494,6 +3494,8 @@ void aead_multipart_encrypt( int key_type_arg, data_t *key_data, size_t ad_part_len = 0; size_t data_part_len = 0; + /* Ensure that either one part of the test or the other is done, i.e this + * test does something. */ TEST_ASSERT( do_test_ad_chunked || do_test_data_chunked ); /* Temporary whilst we have algorithms that cannot support chunking */ @@ -3588,6 +3590,10 @@ void aead_multipart_decrypt( int key_type_arg, data_t *key_data, size_t ad_part_len = 0; size_t data_part_len = 0; + /* Ensure that either one part of the test or the other is done, i.e this + * test does something. */ + TEST_ASSERT( do_test_ad_chunked || do_test_data_chunked ); + /* Temporary whilst we have algorithms that cannot support chunking */ if( do_test_ad_chunked == 1 ) {