tools: kwboot: Allow to use -b without image path as the last getopt() option
authorPali Rohár <pali@kernel.org>
Mon, 7 Feb 2022 09:12:24 +0000 (10:12 +0100)
committerStefan Roese <sr@denx.de>
Thu, 10 Feb 2022 06:12:16 +0000 (07:12 +0100)
Currently it is possible to call "kwboot -b -t /dev/ttyUSB0" but not to
call "kwboot -b /dev/ttyUSB0".

Fix it by not trying to process the last argv[], which is non-getopt()
option (tty path) as the image path for -b.

Fixes: c513fe47dca2 ("tools: kwboot: Allow to use option -b without image path")
Reported-by: Marcel Ziswiler <marcel@ziswiler.com>
Signed-off-by: Pali Rohár <pali@kernel.org>
Reviewed-by: Marcel Ziswiler <marcel@ziswiler.com>
Tested-by: Marcel Ziswiler <marcel@ziswiler.com>
tools/kwboot.c

index 7737188..68c0ef1 100644 (file)
@@ -1770,7 +1770,7 @@ main(int argc, char **argv)
                        bootmsg = kwboot_msg_boot;
                        if (prev_optind == optind)
                                goto usage;
-                       if (argv[optind] && argv[optind][0] != '-')
+                       if (optind < argc - 1 && argv[optind] && argv[optind][0] != '-')
                                imgpath = argv[optind++];
                        break;