diff options
author | Davidlohr Bueso | 2012-04-27 13:23:42 +0200 |
---|---|---|
committer | Karel Zak | 2012-05-02 09:33:24 +0200 |
commit | 9f6c866f656b43f8a190f526a697c0e803cfb249 (patch) | |
tree | af7210cb59a77eb9acdd9911940a990dcb462d98 /fdisk/fdisk.c | |
parent | findmnt: restrict within-device matching (diff) | |
download | kernel-qcow2-util-linux-9f6c866f656b43f8a190f526a697c0e803cfb249.tar.gz kernel-qcow2-util-linux-9f6c866f656b43f8a190f526a697c0e803cfb249.tar.xz kernel-qcow2-util-linux-9f6c866f656b43f8a190f526a697c0e803cfb249.zip |
fdisk: remove stack jumping
This patch eliminates the long/setjmp code. The current logic does not do
anything with stack jumps as it only exits from
print_partition_table_from_option() when a fatal error occurs.
Signed-off-by: Davidlohr Bueso <dave@gnu.org>
Diffstat (limited to 'fdisk/fdisk.c')
-rw-r--r-- | fdisk/fdisk.c | 17 |
1 files changed, 2 insertions, 15 deletions
diff --git a/fdisk/fdisk.c b/fdisk/fdisk.c index 4054b4d28..886f011bf 100644 --- a/fdisk/fdisk.c +++ b/fdisk/fdisk.c @@ -14,7 +14,6 @@ #include <string.h> #include <fcntl.h> #include <ctype.h> -#include <setjmp.h> #include <errno.h> #include <getopt.h> #include <sys/stat.h> @@ -245,8 +244,6 @@ int has_topology; enum labeltype disklabel; /* Current disklabel */ -jmp_buf listingbuf; - static void __attribute__ ((__noreturn__)) usage(FILE *out) { fprintf(out, _("Usage:\n" @@ -266,13 +263,8 @@ static void __attribute__ ((__noreturn__)) usage(FILE *out) exit(out == stderr ? EXIT_FAILURE : EXIT_SUCCESS); } -void fatal(enum failure why) { - - if (listing) { - close(fd); - longjmp(listingbuf, 1); - } - +void fatal(enum failure why) +{ switch (why) { case unable_to_open: err(EXIT_FAILURE, _("unable to open %s"), disk_device); @@ -2708,8 +2700,6 @@ print_partition_table_from_option(char *device) int gb; disk_device = device; - if (setjmp(listingbuf)) - return; gpt_warning(device); gb = get_boot(1); if (gb < 0) { /* no DOS signature */ @@ -2950,9 +2940,6 @@ main(int argc, char **argv) { nowarn = 1; if (argc > optind) { int k; - /* avoid gcc warning: - variable `k' might be clobbered by `longjmp' */ - dummy(&k); listing = 1; for (k = optind; k < argc; k++) print_partition_table_from_option(argv[k]); |