Patrick Williams | 975a06f | 2022-10-21 14:42:47 -0500 | [diff] [blame^] | 1 | From d9169d380366afc63af5d4bf02791aeb41f47897 Mon Sep 17 00:00:00 2001 |
| 2 | From: Satish Kumar <satish.kumar01@arm.com> |
| 3 | Date: Sun, 12 Dec 2021 10:43:48 +0000 |
| 4 | Subject: [PATCH 08/19] Run psa-arch-test |
| 5 | |
| 6 | Fixes needed to run psa-arch-test |
| 7 | |
| 8 | Upstream-Status: Pending |
| 9 | Signed-off-by: Satish Kumar <satish.kumar01@arm.com> |
| 10 | Signed-off-by: Rui Miguel Silva <rui.silva@linaro.org> |
| 11 | --- |
| 12 | components/service/common/psa_ipc/service_psa_ipc.c | 1 + |
| 13 | .../backend/secure_storage_ipc/secure_storage_ipc.c | 8 -------- |
| 14 | .../service/secure_storage/include/psa/storage_common.h | 4 ++-- |
| 15 | 3 files changed, 3 insertions(+), 10 deletions(-) |
| 16 | |
| 17 | diff --git a/components/service/common/psa_ipc/service_psa_ipc.c b/components/service/common/psa_ipc/service_psa_ipc.c |
| 18 | index 95a07c135f31..5e5815dbc9cf 100644 |
| 19 | --- a/components/service/common/psa_ipc/service_psa_ipc.c |
| 20 | +++ b/components/service/common/psa_ipc/service_psa_ipc.c |
| 21 | @@ -185,6 +185,7 @@ psa_status_t psa_call(struct rpc_caller *caller, psa_handle_t psa_handle, |
| 22 | resp_msg->params.out_vec); |
| 23 | |
| 24 | for (i = 0; i < resp_msg->params.out_len; i++) { |
| 25 | + out_vec[i].len = out_vec_param[i].len; |
| 26 | memcpy(out_vec[i].base, rpc_caller_phys_to_virt(caller, out_vec_param[i].base), |
| 27 | out_vec[i].len); |
| 28 | } |
| 29 | diff --git a/components/service/secure_storage/backend/secure_storage_ipc/secure_storage_ipc.c b/components/service/secure_storage/backend/secure_storage_ipc/secure_storage_ipc.c |
| 30 | index 9b55f77dd395..a1f369db253e 100644 |
| 31 | --- a/components/service/secure_storage/backend/secure_storage_ipc/secure_storage_ipc.c |
| 32 | +++ b/components/service/secure_storage/backend/secure_storage_ipc/secure_storage_ipc.c |
| 33 | @@ -31,10 +31,6 @@ static psa_status_t secure_storage_ipc_set(void *context, uint32_t client_id, |
| 34 | |
| 35 | ipc->client.rpc_status = TS_RPC_CALL_ACCEPTED; |
| 36 | |
| 37 | - /* Validating input parameters */ |
| 38 | - if (p_data == NULL) |
| 39 | - return PSA_ERROR_INVALID_ARGUMENT; |
| 40 | - |
| 41 | psa_status = psa_call(caller, TFM_PROTECTED_STORAGE_SERVICE_HANDLE, |
| 42 | TFM_PS_SET, in_vec, IOVEC_LEN(in_vec), NULL, 0); |
| 43 | if (psa_status < 0) |
| 44 | @@ -96,10 +92,6 @@ static psa_status_t secure_storage_ipc_get_info(void *context, |
| 45 | |
| 46 | (void)client_id; |
| 47 | |
| 48 | - /* Validating input parameters */ |
| 49 | - if (!p_info) |
| 50 | - return PSA_ERROR_INVALID_ARGUMENT; |
| 51 | - |
| 52 | psa_status = psa_call(caller, TFM_PROTECTED_STORAGE_SERVICE_HANDLE, |
| 53 | TFM_PS_GET_INFO, in_vec, |
| 54 | IOVEC_LEN(in_vec), out_vec, IOVEC_LEN(out_vec)); |
| 55 | diff --git a/components/service/secure_storage/include/psa/storage_common.h b/components/service/secure_storage/include/psa/storage_common.h |
| 56 | index 4f6ba2a7d822..1fd6b40dc803 100644 |
| 57 | --- a/components/service/secure_storage/include/psa/storage_common.h |
| 58 | +++ b/components/service/secure_storage/include/psa/storage_common.h |
| 59 | @@ -20,8 +20,8 @@ typedef uint64_t psa_storage_uid_t; |
| 60 | typedef uint32_t psa_storage_create_flags_t; |
| 61 | |
| 62 | struct psa_storage_info_t { |
| 63 | - size_t capacity; |
| 64 | - size_t size; |
| 65 | + uint32_t capacity; |
| 66 | + uint32_t size; |
| 67 | psa_storage_create_flags_t flags; |
| 68 | }; |
| 69 | |
| 70 | -- |
| 71 | 2.38.0 |
| 72 | |