diff options
author | Samuel Holland <samuel@sholland.org> | 2019-10-14 20:28:37 -0500 |
---|---|---|
committer | Nico Huber <nico.h@gmx.de> | 2019-10-16 08:31:19 +0000 |
commit | dc9025c14d7d7614ee036b9e5c77a22ecb1aa0a3 (patch) | |
tree | 0da1310a28d7d5f2b6ef733c2accdcb319df718b | |
parent | 62b6a8394a6d09476ea71b99e079dbe954851d4e (diff) | |
download | coreboot-dc9025c14d7d7614ee036b9e5c77a22ecb1aa0a3.tar.xz |
util/ifdtool: Fix argument parsing for layout/validate
When `mode_validate` was added, a second copy of `mode_layout` was
accidentally added to the multiple-mode-argument check instead. This
prevents `-f` from working. Fix the check to reference the correct
variable.
Change-Id: Ibac6f090550ff63ec9158355b0450da204a300a7
Signed-off-by: Samuel Holland <samuel@sholland.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/36049
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
-rw-r--r-- | util/ifdtool/ifdtool.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/util/ifdtool/ifdtool.c b/util/ifdtool/ifdtool.c index 0e83c760d8..24e20e19ab 100644 --- a/util/ifdtool/ifdtool.c +++ b/util/ifdtool/ifdtool.c @@ -1659,7 +1659,7 @@ int main(int argc, char *argv[]) if ((mode_dump + mode_layout + mode_extract + mode_inject + mode_newlayout + (mode_spifreq | mode_em100 | mode_unlocked | - mode_locked) + mode_altmedisable + mode_layout) > 1) { + mode_locked) + mode_altmedisable + mode_validate) > 1) { fprintf(stderr, "You may not specify more than one mode.\n\n"); print_usage(argv[0]); exit(EXIT_FAILURE); |