add invalid argument incase receiving uid==0

pull/9862/head
Netanel Gonen 2019-02-26 18:27:48 +02:00
parent d2eb2e4eb5
commit 3e89a05e0b
2 changed files with 16 additions and 0 deletions

View File

@ -191,6 +191,9 @@ psa_status_t psa_storage_set_impl(KVStore *kvstore, int32_t pid, psa_storage_uid
return PSA_ERROR_NOT_SUPPORTED; return PSA_ERROR_NOT_SUPPORTED;
} }
if (uid == 0) {
return PSA_ERROR_INVALID_ARGUMENT;
}
// Generate KVStore key // Generate KVStore key
char kv_key[PSA_STORAGE_FILE_NAME_MAX] = {'\0'}; char kv_key[PSA_STORAGE_FILE_NAME_MAX] = {'\0'};
generate_fn(kv_key, PSA_STORAGE_FILE_NAME_MAX, uid, pid); generate_fn(kv_key, PSA_STORAGE_FILE_NAME_MAX, uid, pid);
@ -208,6 +211,10 @@ psa_status_t psa_storage_set_impl(KVStore *kvstore, int32_t pid, psa_storage_uid
psa_status_t psa_storage_get_impl(KVStore *kvstore, int32_t pid, psa_storage_uid_t uid, psa_status_t psa_storage_get_impl(KVStore *kvstore, int32_t pid, psa_storage_uid_t uid,
uint32_t data_offset, uint32_t data_length, void *p_data) uint32_t data_offset, uint32_t data_length, void *p_data)
{ {
if (uid == 0) {
return PSA_ERROR_INVALID_ARGUMENT;
}
// Generate KVStore key // Generate KVStore key
char kv_key[PSA_STORAGE_FILE_NAME_MAX] = {'\0'}; char kv_key[PSA_STORAGE_FILE_NAME_MAX] = {'\0'};
generate_fn(kv_key, PSA_STORAGE_FILE_NAME_MAX, uid, pid); generate_fn(kv_key, PSA_STORAGE_FILE_NAME_MAX, uid, pid);
@ -242,6 +249,11 @@ psa_status_t psa_storage_get_impl(KVStore *kvstore, int32_t pid, psa_storage_uid
psa_status_t psa_storage_get_info_impl(KVStore *kvstore, int32_t pid, psa_storage_uid_t uid, psa_status_t psa_storage_get_info_impl(KVStore *kvstore, int32_t pid, psa_storage_uid_t uid,
struct psa_storage_info_t *p_info) struct psa_storage_info_t *p_info)
{ {
if (uid == 0) {
return PSA_ERROR_INVALID_ARGUMENT;
}
// Generate KVStore key // Generate KVStore key
char kv_key[PSA_STORAGE_FILE_NAME_MAX] = {'\0'}; char kv_key[PSA_STORAGE_FILE_NAME_MAX] = {'\0'};
generate_fn(kv_key, PSA_STORAGE_FILE_NAME_MAX, uid, pid); generate_fn(kv_key, PSA_STORAGE_FILE_NAME_MAX, uid, pid);
@ -262,6 +274,10 @@ psa_status_t psa_storage_get_info_impl(KVStore *kvstore, int32_t pid, psa_storag
psa_status_t psa_storage_remove_impl(KVStore *kvstore, int32_t pid, psa_storage_uid_t uid) psa_status_t psa_storage_remove_impl(KVStore *kvstore, int32_t pid, psa_storage_uid_t uid)
{ {
if (uid == 0) {
return PSA_ERROR_INVALID_ARGUMENT;
}
// Generate KVStore key // Generate KVStore key
char kv_key[PSA_STORAGE_FILE_NAME_MAX] = {'\0'}; char kv_key[PSA_STORAGE_FILE_NAME_MAX] = {'\0'};
generate_fn(kv_key, PSA_STORAGE_FILE_NAME_MAX, uid, pid); generate_fn(kv_key, PSA_STORAGE_FILE_NAME_MAX, uid, pid);

View File