commit 30afafd5276cfce6e88a33f6fd484b28c02e4f75
author Gilles Peskine <Gilles.Peskine@arm.com> Thu Apr 25 13:47:40 2019 +0200
committer Gilles Peskine <Gilles.Peskine@arm.com> Thu Apr 25 17:42:32 2019 +0200
tree 65ae0e1f443fbdc9a7bf041e7cce1eb5ff549c2b
parent 3495b58fcfea9667f5170d0aab171d2e172f4a57

Fix build errors with MBEDTLS_PSA_CRYPTO_STORAGE_C disabled

library/psa_crypto.c

diff --git a/library/psa_crypto.c b/library/psa_crypto.c
index efec00b..e035ece 100644
--- a/library/psa_crypto.c
+++ b/library/psa_crypto.c
@@ -1307,6 +1307,7 @@
 static psa_status_t psa_finish_key_creation( psa_key_slot_t *slot )
 {
     psa_status_t status = PSA_SUCCESS;
+    (void) slot;
 
 #if defined(MBEDTLS_PSA_CRYPTO_STORAGE_C)
     if( slot->lifetime == PSA_KEY_LIFETIME_PERSISTENT )

library/psa_crypto_slot_management.c

diff --git a/library/psa_crypto_slot_management.c b/library/psa_crypto_slot_management.c
index d8b0a2e..8ee5615 100644
--- a/library/psa_crypto_slot_management.c
+++ b/library/psa_crypto_slot_management.c
@@ -192,7 +192,6 @@
         return( 0 );
     return( 1 );
 }
-#endif /* defined(MBEDTLS_PSA_CRYPTO_STORAGE_C) */
 
 /** Declare a slot as persistent and load it from storage.
  *
@@ -215,7 +214,6 @@
 static psa_status_t psa_internal_make_key_persistent( psa_key_handle_t handle,
                                                       psa_key_file_id_t id )
 {
-#if defined(MBEDTLS_PSA_CRYPTO_STORAGE_C)
     psa_key_slot_t *slot;
     psa_status_t status;
 
@@ -228,13 +226,8 @@
     status = psa_load_persistent_key_into_slot( slot );
 
     return( status );
-
-#else /* MBEDTLS_PSA_CRYPTO_STORAGE_C */
-    (void) handle;
-    (void) id;
-    return( PSA_ERROR_NOT_SUPPORTED );
-#endif /* !MBEDTLS_PSA_CRYPTO_STORAGE_C */
 }
+#endif /* defined(MBEDTLS_PSA_CRYPTO_STORAGE_C) */
 
 psa_status_t psa_validate_persistent_key_parameters(
     psa_key_lifetime_t lifetime,
@@ -242,9 +235,16 @@
 {
     if( lifetime != PSA_KEY_LIFETIME_PERSISTENT )
         return( PSA_ERROR_INVALID_ARGUMENT );
+
+#if defined(MBEDTLS_PSA_CRYPTO_STORAGE_C)
     if( ! psa_is_key_id_valid( id ) )
         return( PSA_ERROR_INVALID_ARGUMENT );
     return( PSA_SUCCESS );
+
+#else /* MBEDTLS_PSA_CRYPTO_STORAGE_C */
+    (void) id;
+    return( PSA_ERROR_NOT_SUPPORTED );
+#endif /* !MBEDTLS_PSA_CRYPTO_STORAGE_C */
 }
 
 static psa_status_t persistent_key_setup( psa_key_lifetime_t lifetime,
@@ -260,6 +260,7 @@
     if( status != PSA_SUCCESS )
         return( status );
 
+#if defined(MBEDTLS_PSA_CRYPTO_STORAGE_C)
     status = psa_internal_allocate_key_slot( handle );
     if( status != PSA_SUCCESS )
         return( status );
@@ -271,6 +272,10 @@
         *handle = 0;
     }
     return( status );
+#else /* defined(MBEDTLS_PSA_CRYPTO_STORAGE_C) */
+    (void) wanted_load_status;
+    return( PSA_ERROR_NOT_SUPPORTED );
+#endif /* !defined(MBEDTLS_PSA_CRYPTO_STORAGE_C) */
 }
 
 psa_status_t psa_open_key( psa_key_lifetime_t lifetime,