secure_storage: its: improve return codes

Some minor improvements.

Signed-off-by: Tomi Fontanilles <[email protected]>

Author: tomi-font

subsys/secure_storage/src/its/store/settings.c

@@ -120,7 +120,7 @@ psa_status_t secure_storage_its_store_remove(secure_storage_its_uid_t uid)
 	secure_storage_its_store_settings_get_name(uid, name);
 
 	ret = settings_delete(name);
-
 	LOG_DBG("%s %s. (%d)", ret ? "Failed to delete" : "Deleted", name, ret);
+
 	return ret ? PSA_ERROR_STORAGE_FAILURE : PSA_SUCCESS;
 }

subsys/secure_storage/src/its/store/zms.c

@@ -108,15 +108,15 @@ psa_status_t secure_storage_its_store_get(secure_storage_its_uid_t uid, size_t d
 
 psa_status_t secure_storage_its_store_remove(secure_storage_its_uid_t uid)
 {
-	int zms_ret;
+	int ret;
 	const uint32_t zms_id = zms_id_from(uid);
 
 	if (has_forbidden_bits_set(uid)) {
 		return PSA_ERROR_INVALID_ARGUMENT;
 	}
 
-	zms_ret = zms_delete(&s_zms, zms_id);
-	LOG_DBG("%s 0x%x. (%d)", zms_ret ? "Failed to delete" : "Deleted", zms_id, zms_ret);
-	BUILD_ASSERT(PSA_SUCCESS == 0);
-	return zms_ret;
+	ret = zms_delete(&s_zms, zms_id);
+	LOG_DBG("%s 0x%x. (%d)", ret ? "Failed to delete" : "Deleted", zms_id, ret);
+
+	return ret ? PSA_ERROR_STORAGE_FAILURE : PSA_SUCCESS;
 }

subsys/secure_storage/src/its/transform/aead.c

@@ -31,7 +31,7 @@ static psa_status_t psa_aead_crypt(psa_key_usage_t operation, secure_storage_its
 	psa_set_key_lifetime(&key_attributes, PSA_KEY_LIFETIME_VOLATILE);
 	psa_set_key_type(&key_attributes, key_type);
 	psa_set_key_algorithm(&key_attributes, alg);
-	psa_set_key_bits(&key_attributes, sizeof(key) * 8);
+	psa_set_key_bits(&key_attributes, PSA_BYTES_TO_BITS(sizeof(key)));
 
 	/* Avoid calling psa_aead_*crypt() because that would require importing keys into
 	 * PSA Crypto. This gets called from PSA Crypto for storing persistent keys so,
@@ -113,7 +113,7 @@ psa_status_t secure_storage_its_transform_from_store(
 		psa_storage_create_flags_t *create_flags)
 {
 	if (stored_data_len < STORED_ENTRY_LEN(0)) {
-		return PSA_ERROR_STORAGE_FAILURE;
+		return PSA_ERROR_DATA_CORRUPT;
 	}
 
 	psa_status_t ret;

subsys/secure_storage/src/its/transform/aead_get.c

@@ -75,6 +75,7 @@ psa_status_t secure_storage_its_transform_aead_get_key(
 	if (hwinfo_ret != 0) {
 		hwinfo_ret = hwinfo_get_device_id(data.device_id, sizeof(data.device_id));
 		if (hwinfo_ret <= 0) {
+			LOG_DBG("Failed to retrieve the device ID. (%zd)", hwinfo_ret);
 			return PSA_ERROR_HARDWARE_FAILURE;
 		}
 		if (hwinfo_ret < sizeof(data.device_id)) {