cuberite/polarssl
1
0
Fork 0
mirror of https://github.com/cuberite/polarssl.git synced 2025-09-15 18:18:12 -04:00
Code Issues Packages Projects Releases Wiki Activity

Fix unintentional direct use of memcpy()

Browse Source 
Change psa_crypto_output_copy_free() to use psa_crypto_copy_output()
rather than calling memcpy directly as was erroneously done previously.

Signed-off-by: David Horstmann <david.horstmann@arm.com>
This commit is contained in:
David Horstmann 2023-11-15 15:57:32 +00:00
parent ba65bd79e8
commit 761761ff64
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
library/psa_crypto.c
View File

@ -5617,6 +5617,8 @@ psa_status_t psa_crypto_output_copy_alloc(uint8_t *output, size_t output_len,
psa_status_t psa_crypto_output_copy_free(psa_crypto_output_copy_t *output_copy) psa_status_t psa_crypto_output_copy_free(psa_crypto_output_copy_t *output_copy)
{ {
psa_status_t status;
if (output_copy->buffer == NULL) { if (output_copy->buffer == NULL) {
output_copy->len = 0; output_copy->len = 0;
return PSA_SUCCESS; return PSA_SUCCESS;
@ -5626,7 +5628,11 @@ psa_status_t psa_crypto_output_copy_free(psa_crypto_output_copy_t *output_copy)
return PSA_ERROR_CORRUPTION_DETECTED; return PSA_ERROR_CORRUPTION_DETECTED;
} }
memcpy(output_copy->original, output_copy->buffer, output_copy->len); status = psa_crypto_copy_output(output_copy->buffer, output_copy->len,
output_copy->original, output_copy->len);
if (status != PSA_SUCCESS) {
return status;
}
mbedtls_free(output_copy->buffer); mbedtls_free(output_copy->buffer);
output_copy->buffer = NULL; output_copy->buffer = NULL;