Commit 38410f80 authored by Kévin Petit's avatar Kévin Petit Committed by Sandrine Bailleux
Browse files

fip_create: don't succeed if one of the passed files doesn't exist



If one of the files passed to fip_create on the command line doesn't
exist, it will print an error message but produce an incomplete
fip.bin file and report success. This behaviour could potentially
hide errors made in the command line arguments.

This patch addresses the issue by having the tool bail out if one of
the supplied files can't be processed.
Signed-off-by: default avatarKévin Petit <kevin.petit@arm.com>

Fixes ARM-software/tf-issues#279

Change-Id: I1c7d87d09eb4c063005b7969bdaad1d043c29dec
parent 29e32cba
...@@ -543,7 +543,7 @@ static int parse_cmdline(int argc, char **argv, struct option *options, ...@@ -543,7 +543,7 @@ static int parse_cmdline(int argc, char **argv, struct option *options,
if (status != 0) { if (status != 0) {
printf("Failed to process %s\n", printf("Failed to process %s\n",
options[option_index].name); options[option_index].name);
break; return status;
} else { } else {
/* Update package */ /* Update package */
*do_pack = 1; *do_pack = 1;
......
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