commit d79d5e7ed0cb285e5c4cbb0cb797a9d276de4fac
author David Horstmann <david.horstmann@arm.com> Thu Nov 16 20:26:16 2023 +0000
committer David Horstmann <david.horstmann@arm.com> Wed Nov 22 15:07:31 2023 +0000
tree 3c88639f7035a882962b3431fa31db43ca7ff10f
parent c3be8cb24a7be7bebf7f9c88e5440356fb6c82a7

Check for len == 0 rather than buffer == NULL

This makes the intention clearer

Signed-off-by: David Horstmann <david.horstmann@arm.com>

tests/suites/test_suite_psa_crypto_memory.function

diff --git a/tests/suites/test_suite_psa_crypto_memory.function b/tests/suites/test_suite_psa_crypto_memory.function
index 5938258..83fa6cb 100644
--- a/tests/suites/test_suite_psa_crypto_memory.function
+++ b/tests/suites/test_suite_psa_crypto_memory.function
@@ -94,7 +94,7 @@
     TEST_EQUAL(status, exp_status);
 
     if (exp_status == PSA_SUCCESS) {
-        if (input == NULL) {
+        if (input_len == 0) {
             TEST_ASSERT(input_copy.buffer == NULL);
         } else {
             TEST_ASSERT(input_copy.buffer != input);
@@ -167,7 +167,7 @@
 
     if (exp_status == PSA_SUCCESS) {
         TEST_ASSERT(output_copy.original == output);
-        if (output == NULL) {
+        if (output_len == 0) {
             TEST_ASSERT(output_copy.buffer == NULL);
         }
         TEST_EQUAL(output_copy.len, output_len);
@@ -203,7 +203,7 @@
     output_copy.len = output_len;
     output_copy.original = output;
 
-    if (output_copy.buffer != NULL) {
+    if (output_copy.len != 0) {
         fill_buffer_pattern(output_copy.buffer, output_copy.len);
         memcpy(buffer_copy_for_comparison, output_copy.buffer, output_copy.len);
     }