commit 0071b873a3609885597e20be5ec6f428ceb22b95
author Moran Peker <moran.peker@arm.com> Sun Apr 22 20:16:58 2018 +0300
committer itayzafrir <itay.zafrir@arm.com> Wed Sep 05 12:14:28 2018 +0300
tree 0998bba7bb38f3c2eaba3215a32fff093e804ec2
parent 4c80d8331a1cabbad3f61f7d1b16eadbadd3ad2c

add missing parameter output_size on psa_cipher_finish

include/psa/crypto.h

diff --git a/include/psa/crypto.h b/include/psa/crypto.h
index b1c1abb..089484f 100644
--- a/include/psa/crypto.h
+++ b/include/psa/crypto.h
@@ -1064,6 +1064,7 @@
 
 psa_status_t psa_cipher_finish(psa_cipher_operation_t *operation,
                                uint8_t *output,
+                               size_t output_size,
                                size_t *output_length);
 
 psa_status_t psa_cipher_abort(psa_cipher_operation_t *operation);

library/psa_crypto.c

diff --git a/library/psa_crypto.c b/library/psa_crypto.c
index bb74c26..fbc5949 100644
--- a/library/psa_crypto.c
+++ b/library/psa_crypto.c
@@ -1461,6 +1461,7 @@
 
 psa_status_t psa_cipher_finish(psa_cipher_operation_t *operation,
                                uint8_t *output,
+                               size_t output_size,
                                size_t *output_length)
 {
     int ret = MBEDTLS_ERR_CIPHER_FEATURE_UNAVAILABLE;

tests/suites/test_suite_psa_crypto.function

diff --git a/tests/suites/test_suite_psa_crypto.function b/tests/suites/test_suite_psa_crypto.function
index eb217f9..bc46ad2 100644
--- a/tests/suites/test_suite_psa_crypto.function
+++ b/tests/suites/test_suite_psa_crypto.function
@@ -574,7 +574,7 @@
                                     output, output_size, 
                                     &output_length) == PSA_SUCCESS );
 	TEST_ASSERT( psa_cipher_finish( &operation, output + output_length,
-                                    &output_length) == PSA_SUCCESS );
+                                    output_size, &output_length) == PSA_SUCCESS );
 
     TEST_ASSERT( psa_cipher_abort( &operation ) == PSA_SUCCESS );
 
@@ -637,7 +637,7 @@
                                     output, output_size, 
                                     &output_length) == PSA_SUCCESS );
 	TEST_ASSERT( psa_cipher_finish( &operation, output + output_length,
-                                    &output_length) == PSA_SUCCESS );
+                                    output_size, &output_length) == PSA_SUCCESS );
 
     TEST_ASSERT( psa_cipher_abort( &operation ) == PSA_SUCCESS );
 
@@ -700,7 +700,7 @@
                                     output1, output1_size, 
                                     &output1_length) == PSA_SUCCESS );
 	TEST_ASSERT( psa_cipher_finish( &operation1, output1 + output1_length,
-                                    &tmp_output_length) == PSA_SUCCESS );
+                                    output1_size, &tmp_output_length) == PSA_SUCCESS );
 									
     output1_length += tmp_output_length;
 
@@ -715,7 +715,7 @@
                                     output2, output2_size, &output2_length) == PSA_SUCCESS );
     tmp_output_length = 0;
 	TEST_ASSERT( psa_cipher_finish( &operation2, output2 + output2_length,
-                                    &tmp_output_length) == PSA_SUCCESS );  
+                                    output2_size, &tmp_output_length) == PSA_SUCCESS );  
     
 	output2_length += tmp_output_length;