[coreboot-gerrit] [XS] Change in coreboot[main]: ifdtool: Exit with failure on unrecognized flags

[Hsuan-ting Chen (Code Review)]

Hsuan-ting Chen has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/84260?usp=email )

Change subject: ifdtool: Exit with failure on unrecognized flags
......................................................................

ifdtool: Exit with failure on unrecognized flags

ifdtool will exit with success while encountering an unrecognized flag.
For example, -g is a newly introduced flag, when we want to call it with
an older version of ifdtool, we will get the return value 0 and cause
confusion.

This patch change the exit status for unrecognized flags and doesn't
change the exit status for -h and -?.

BUG=b:362983041
BRANCH=none
TEST=futility update --servo --image /var/tmp/image.bin --quirks
     unlock_csme on the servo host with old ifdtool

Signed-off-by: Hsuan Ting Chen 
Change-Id: I046ad7ec790cda41a98a1de5cd730d32f65a9067
---
M util/ifdtool/ifdtool.c
1 file changed, 4 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/60/84260/1

diff --git a/util/ifdtool/ifdtool.c b/util/ifdtool/ifdtool.c
index 32b2081..5993bfc 100644
--- a/util/ifdtool/ifdtool.c
+++ b/util/ifdtool/ifdtool.c
@@ -2514,10 +2514,13 @@
    		break;
    	case 'h':
    	case '?':
-		default:
    		print_usage(argv[0]);
    		exit(EXIT_SUCCESS);
    		break;
+		default:
+			print_usage(argv[0]);
+			exit(EXIT_FAILURE);
+			break;
    	}
    }