michaelh/zephyr
1
0
Fork 0
Code Releases Activity

modules: openthread: platform: optimize stack usage in crypto_psa

Browse source 
A PSA crypto operation object can be initialized in multiple
ways according to the documentation. For example,
1. Using a dedicated psa_xxx_operation_init() function that
   returns an initialized object.
2. Using memset() to zero out the operation object.

For some PSA crypto driver implementations, using the first
method causes an excessive stack usage if the operation
object is large and psa_xxx_operation_init() is not inlined.
Instead, it is better to stick to memset() for this purpose.

Signed-off-by: Damian Krolik <damian.krolik@nordicsemi.no>
This commit is contained in:
Damian Krolik 2025-02-21 10:09:21 +01:00 • committed by Carles Cufí
commit ab09c522b5
1 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

4
modules/openthread/platform/crypto_psa.c
View file

@ -226,7 +226,7 @@ otError otPlatCryptoHmacSha256Init(otCryptoContext *aContext)
}
operation = aContext->mContext;
*operation = psa_mac_operation_init();
memset(operation, 0, sizeof(*operation));
return OT_ERROR_NONE;
}
@ -347,7 +347,7 @@ otError otPlatCryptoSha256Init(otCryptoContext *aContext)
}
operation = aContext->mContext;
*operation = psa_hash_operation_init();
memset(operation, 0, sizeof(*operation));
return OT_ERROR_NONE;
}