From 68224d108d1e5359a111cb2be0404f6556cb90dd Mon Sep 17 00:00:00 2001 From: Karel Zak Date: Tue, 16 Apr 2019 13:47:17 +0200 Subject: 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 --- disk-utils/mkfs.bfs.c | 10 ++-------- disk-utils/mkfs.c | 10 ++-------- 2 files changed, 4 insertions(+), 16 deletions(-) (limited to 'disk-utils') diff --git a/disk-utils/mkfs.bfs.c b/disk-utils/mkfs.bfs.c index 5bf8fdfd8..77f253f1e 100644 --- a/disk-utils/mkfs.bfs.c +++ b/disk-utils/mkfs.bfs.c @@ -91,12 +91,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 *device, *volume, *fsname; @@ -134,7 +128,7 @@ int main(int argc, char **argv) errtryhelp(EXIT_FAILURE); } if (argc == 2 && !strcmp(argv[1], "-V")) - print_version(); + print_version(EXIT_SUCCESS); volume = fsname = " "; /* is there a default? */ inodes = 0; @@ -169,7 +163,7 @@ int main(int argc, char **argv) break; case VERSION_OPTION: - print_version(); + print_version(EXIT_SUCCESS); case 'h': usage(); default: 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; -- cgit v1.2.3-55-g7522