commit d98059d599dc4745d480b052626718d70bae8ad7
author Ronald Cron <ronald.cron@arm.com> Fri Oct 23 18:00:55 2020 +0200
committer Ronald Cron <ronald.cron@arm.com> Tue Nov 10 18:18:15 2020 +0100
tree 1e956b888424090c62716312b4aa1d62fc459f34
parent 4067d1c1e571b2e0189cfd230671c6c52c6f274f

psa: Fix tests/handling of lifetime incompatible with location

The lifetime of key attributes now encodes whether a key is
volatile/persistent or not AND its location.
Fix PSA code where the fact that the lifetime encodes
the key location was not taken into account properly.

Fix the impacted tests and add two non regression tests.

Signed-off-by: Ronald Cron <ronald.cron@arm.com>

library/psa_crypto.c

diff --git a/library/psa_crypto.c b/library/psa_crypto.c
index 2f01bf2..82e2549 100644
--- a/library/psa_crypto.c
+++ b/library/psa_crypto.c
@@ -1342,7 +1342,7 @@
 #endif /* MBEDTLS_PSA_CRYPTO_SE_C */
 
 #if defined(MBEDTLS_PSA_CRYPTO_STORAGE_C)
-    if( slot->attr.lifetime != PSA_KEY_LIFETIME_VOLATILE )
+    if( ! PSA_KEY_LIFETIME_IS_VOLATILE( slot->attr.lifetime ) )
     {
         status = psa_destroy_persistent_key( slot->attr.id );
         if( overall_status == PSA_SUCCESS )

library/psa_crypto_slot_management.c

diff --git a/library/psa_crypto_slot_management.c b/library/psa_crypto_slot_management.c
index 6f6ba07..7308f6f 100644
--- a/library/psa_crypto_slot_management.c
+++ b/library/psa_crypto_slot_management.c
@@ -348,7 +348,7 @@
     if( status != PSA_SUCCESS )
         return( status );
 
-    if( slot->attr.lifetime == PSA_KEY_LIFETIME_VOLATILE )
+    if( PSA_KEY_LIFETIME_IS_VOLATILE( slot->attr.lifetime ) )
         return PSA_SUCCESS;
 
     return( psa_wipe_key_slot( slot ) );