diff options
author | Karel Zak | 2019-04-16 13:47:17 +0200 |
---|---|---|
committer | Karel Zak | 2019-04-16 13:47:17 +0200 |
commit | 68224d108d1e5359a111cb2be0404f6556cb90dd (patch) | |
tree | 4d956d4d580dd6ceac9e7a23469a914780d115ac /disk-utils/mkfs.c | |
parent | include/closestream: add close_stdout_atexit() (diff) | |
download | kernel-qcow2-util-linux-68224d108d1e5359a111cb2be0404f6556cb90dd.tar.gz kernel-qcow2-util-linux-68224d108d1e5359a111cb2be0404f6556cb90dd.tar.xz kernel-qcow2-util-linux-68224d108d1e5359a111cb2be0404f6556cb90dd.zip |
include/c: add print_version() macro
Let's consolidate the version printing code. It also seems better to
use exit() after --version, because it's handled in different way by
ASAN.
It's strange, but ASAN reports leaks after return in main(). Note that
we do not use free-before-exit.
Signed-off-by: Karel Zak <kzak@redhat.com>
Diffstat (limited to 'disk-utils/mkfs.c')
-rw-r--r-- | disk-utils/mkfs.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/disk-utils/mkfs.c b/disk-utils/mkfs.c index 5bcd9613c..fbd575e7e 100644 --- a/disk-utils/mkfs.c +++ b/disk-utils/mkfs.c @@ -61,12 +61,6 @@ static void __attribute__((__noreturn__)) usage(void) exit(EXIT_SUCCESS); } -static void __attribute__ ((__noreturn__)) print_version(void) -{ - printf(UTIL_LINUX_VERSION); - exit(EXIT_SUCCESS); -} - int main(int argc, char **argv) { char *progname; /* name of executable to be called */ @@ -89,7 +83,7 @@ int main(int argc, char **argv) atexit(close_stdout); if (argc == 2 && !strcmp(argv[1], "-V")) - print_version(); + print_version(EXIT_SUCCESS); /* Check commandline options. */ opterr = 0; @@ -106,7 +100,7 @@ int main(int argc, char **argv) case 'h': usage(); case VERSION_OPTION: - print_version(); + print_version(EXIT_SUCCESS); default: optind--; more = 1; |