Simplify the test and description

Previously the check was convoluted. This has been simplified
and given a more appropriate suggestion as per gilles suggestion

Signed-off-by: Joe Subbiani <joe.subbiani@arm.com>
This commit is contained in:
Joe Subbiani 2021-07-14 15:16:09 +01:00
parent da36c9f2ce
commit 113d80c219

View file

@ -630,27 +630,9 @@ void block_cipher_key_type( int type_arg, int block_size_arg )
PSA_KEY_TYPE_CATEGORY_SYMMETRIC ); PSA_KEY_TYPE_CATEGORY_SYMMETRIC );
TEST_EQUAL( PSA_BLOCK_CIPHER_BLOCK_LENGTH( type ), block_size ); TEST_EQUAL( PSA_BLOCK_CIPHER_BLOCK_LENGTH( type ), block_size );
/* PSA_ROUND_UP_TO_MULTIPLE(block_size, length) in crypto_sizes.h /* Check that the block size is a power of 2. This is required, at least,
* Requires block sizes to be a power of 2. for PSA_ROUND_UP_TO_MULTIPLE(block_size, length) in crypto_sizes.h. */
* The following creates a bit and shifts along until it finds a TEST_ASSERT( ( ( block_size - 1 ) & block_size ) == 0 );
* match or a mismatch.
*/
int check = 0;
for (size_t index = 1; index > 0; index = index << 1)
{
if (index == block_size)
{
check = 0;
break;
}
if (index > block_size)
{
check = 1;
break;
}
}
TEST_EQUAL( check, 0);
} }
/* END_CASE */ /* END_CASE */