diff options
| author | Daniel Wagner <daniel.wagner@siemens.com> | 2017-06-12 12:50:54 +0200 |
|---|---|---|
| committer | David Oberhollenzer <david.oberhollenzer@sigma-star.at> | 2017-06-14 11:00:30 +0200 |
| commit | 76e27a79eddb6279bbbc0ec840d66b753a9ba0f3 (patch) | |
| tree | 50650ace637991c5845b15a7800e67f7cdece62b /ubi-utils/ubiformat.c | |
| parent | b8e785a96f907022466bd383ec6afaa201e5f759 (diff) | |
ubi-utils: Return error code if command line option is unknown
The tools in question will quit with an exit code 0 if the command
line option was not recognized. By returning an error code a calling
script has the possibility to distinguish between a real success and
an invalid invocation.
We need to return -1 instead of EXIT_FAILURE to be consistent with the
other exit code places.
Signed-off-by: Daniel Wagner <daniel.wagner@siemens.com>
Signed-off-by: David Oberhollenzer <david.oberhollenzer@sigma-star.at>
Diffstat (limited to 'ubi-utils/ubiformat.c')
| -rw-r--r-- | ubi-utils/ubiformat.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/ubi-utils/ubiformat.c b/ubi-utils/ubiformat.c index 896fe20..ef0378a 100644 --- a/ubi-utils/ubiformat.c +++ b/ubi-utils/ubiformat.c @@ -209,11 +209,15 @@ static int parse_opt(int argc, char * const argv[]) exit(EXIT_SUCCESS); case 'h': - case '?': printf("%s\n\n", doc); printf("%s\n\n", usage); printf("%s\n", optionsstr); exit(EXIT_SUCCESS); + case '?': + printf("%s\n\n", doc); + printf("%s\n\n", usage); + printf("%s\n", optionsstr); + return -1; case ':': return errmsg("parameter is missing"); |