commit a2971ea62cfdd930ec89ab99f00800550d50398b
author Gilles Peskine <Gilles.Peskine@arm.com> Thu Dec 03 20:36:02 2020 +0100
committer Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com> Mon Dec 07 09:58:47 2020 +0100
tree 2187a0bbc8c423be140f066b28c62f686fe442e8
parent 70edd689a8cc774e6359ebee982e0c8bcd504453

Simplify some buffer comparisons in tests

Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>

tests/suites/test_suite_cipher.function

diff --git a/tests/suites/test_suite_cipher.function b/tests/suites/test_suite_cipher.function
index ffe3284..5e9a1e3 100644
--- a/tests/suites/test_suite_cipher.function
+++ b/tests/suites/test_suite_cipher.function
@@ -1251,10 +1251,7 @@
     else
     {
         TEST_ASSERT( ret == 0 );
-
-        TEST_ASSERT( outlen == clear->len );
-        if( clear->len != 0 )
-            TEST_ASSERT( memcmp( decrypt_buf, clear->x, clear->len ) == 0 );
+        ASSERT_COMPARE( decrypt_buf, outlen, clear->x, clear->len );
     }
 
     /* Free this, but keep cipher_plus_tag for deprecated function with PSA */
@@ -1376,9 +1373,7 @@
     {
         /* authentic message: is the plaintext correct? */
         TEST_ASSERT( ret == 0 );
-
-        TEST_ASSERT( outlen == clear->len );
-        TEST_ASSERT( memcmp( decrypt_buf, clear->x, clear->len ) == 0 );
+        ASSERT_COMPARE(  decrypt_buf, outlen, clear->x, clear->len );
     }
 
     mbedtls_free( decrypt_buf );