summaryrefslogtreecommitdiff
path: root/tools/fit_common.c
diff options
context:
space:
mode:
authorJordan Hand <jordanhand22@gmail.com>2019-03-05 14:47:56 -0800
committerTom Rini <trini@konsulko.com>2019-03-08 11:31:44 -0500
commitd32aa3cae44e618048ff7f378577d44f9b6d6dcc (patch)
treefc4cfbe78b9629281ee4a8c1ccead9204afa9224 /tools/fit_common.c
parent280fafff165428bc69db221faaccaf4edfc32d9d (diff)
fdt: Fix FIT header verification in mkimage and conduct same checks as bootm
FIT header verification in mkimage was treating a return code as a boolean, which meant that failures in validating the fit were seen as successes. Additionally, mkimage was checking all formats to find a header which passes validation, rather than using the image type specified to mkimage. checkpatch.pl checks for lines ending with '(' and alignment matching open parentheses are ignored to keep with existing coding style. Signed-off-by: Jordan Hand <jorhand@microsoft.com>
Diffstat (limited to 'tools/fit_common.c')
-rw-r--r--tools/fit_common.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/tools/fit_common.c b/tools/fit_common.c
index d96085eaad..9506390214 100644
--- a/tools/fit_common.c
+++ b/tools/fit_common.c
@@ -26,7 +26,10 @@
int fit_verify_header(unsigned char *ptr, int image_size,
struct image_tool_params *params)
{
- return fdt_check_header(ptr);
+ if (fdt_check_header(ptr) != EXIT_SUCCESS || !fit_check_format(ptr))
+ return EXIT_FAILURE;
+
+ return EXIT_SUCCESS;
}
int fit_check_image_types(uint8_t type)