cbfstool: Check for excessive arguments

Change-Id: I66de6a33b43c284198c0a0a97c5c6a10f9b96e02
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.coreboot.org/16019
Tested-by: build bot (Jenkins)
Reviewed-by: Omar Pakker
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
This commit is contained in:
Nico Huber 2016-08-01 23:18:29 +02:00
parent 9ade717571
commit 5407310e64
1 changed files with 7 additions and 1 deletions

View File

@ -1296,8 +1296,14 @@ int main(int argc, char **argv)
c = getopt_long(argc, argv, commands[i].optstring, c = getopt_long(argc, argv, commands[i].optstring,
long_options, &option_index); long_options, &option_index);
if (c == -1) if (c == -1) {
if (optind < argc) {
ERROR("%s: excessive argument -- '%s'"
"\n", argv[0], argv[optind]);
return 1;
}
break; break;
}
/* filter out illegal long options */ /* filter out illegal long options */
if (strchr(commands[i].optstring, c) == NULL) { if (strchr(commands[i].optstring, c) == NULL) {