Unverified Commit 100992b5 authored by Soby Mathew's avatar Soby Mathew Committed by GitHub
Browse files

Merge pull request #1484 from nathan-menhorn/tee-validate-header-603

Update optee_utils.c to fix ARM-software/tf-issues#603
parents d853d3b2 b554e768
...@@ -25,7 +25,8 @@ typedef struct optee_image { ...@@ -25,7 +25,8 @@ typedef struct optee_image {
#define OPTEE_PAGER_IMAGE_ID 0 #define OPTEE_PAGER_IMAGE_ID 0
#define OPTEE_PAGED_IMAGE_ID 1 #define OPTEE_PAGED_IMAGE_ID 1
#define OPTEE_MAX_IMAGE_NUM 2
#define OPTEE_MAX_NUM_IMAGES 2u
#define TEE_MAGIC_NUM_OPTEE 0x4554504f #define TEE_MAGIC_NUM_OPTEE 0x4554504f
/* /*
...@@ -53,14 +54,20 @@ typedef struct optee_header { ...@@ -53,14 +54,20 @@ typedef struct optee_header {
******************************************************************************/ ******************************************************************************/
static inline int tee_validate_header(optee_header_t *header) static inline int tee_validate_header(optee_header_t *header)
{ {
int valid = 0;
if ((header->magic == TEE_MAGIC_NUM_OPTEE) && if ((header->magic == TEE_MAGIC_NUM_OPTEE) &&
(header->version == 2) && (header->version == 2u) &&
(header->nb_images <= OPTEE_MAX_IMAGE_NUM)) { (header->nb_images > 0u) &&
return 1; (header->nb_images <= OPTEE_MAX_NUM_IMAGES)) {
valid = 1;
} }
else {
WARN("Not a known TEE, use default loading options.\n"); WARN("Not a known TEE, use default loading options.\n");
return 0; }
return valid;
} }
/******************************************************************************* /*******************************************************************************
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment