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 /ubifs-utils | |
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 'ubifs-utils')
-rw-r--r-- | ubifs-utils/mkfs.ubifs/mkfs.ubifs.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/ubifs-utils/mkfs.ubifs/mkfs.ubifs.c b/ubifs-utils/mkfs.ubifs/mkfs.ubifs.c index a60052d..9e69a4f 100644 --- a/ubifs-utils/mkfs.ubifs/mkfs.ubifs.c +++ b/ubifs-utils/mkfs.ubifs/mkfs.ubifs.c @@ -548,15 +548,17 @@ static int get_options(int argc, char**argv) yes = 1; break; case 'h': + printf("%s", helptext); + exit(EXIT_SUCCESS); case '?': printf("%s", helptext); - exit(0); + exit(-1); case 'v': verbose = 1; break; case 'V': common_print_version(); - exit(0); + exit(EXIT_SUCCESS); case 'g': debug_level = strtol(optarg, &endp, 0); if (*endp != '\0' || endp == optarg || |