diff --git a/archival/gunzip.c b/archival/gunzip.c index 4ab197f09..e379ccf10 100644 --- a/archival/gunzip.c +++ b/archival/gunzip.c @@ -71,90 +71,95 @@ const int gunzip_force = 2; const int gunzip_test = 4; const int gunzip_verbose = 8; -static int gunzip_file (const char *path, int flags) +static int gunzip_file(const char *path, int flags) { - FILE *in_file, *out_file; - struct stat stat_buf; - const char *delete_path = NULL; - char *out_path = NULL; + FILE *in_file, *out_file; + struct stat stat_buf; + const char *delete_path = NULL; + char *out_path = NULL; - if (path == NULL || strcmp (path, "-") == 0) { - in_file = stdin; - flags |= gunzip_to_stdout; - } else { - if ((in_file = wfopen(path, "r")) == NULL) - return -1; - - if (flags & gunzip_verbose) { - fprintf(stderr, "%s:\t", path); - } - - /* set the buffer size */ - setvbuf(in_file, NULL, _IOFBF, 0x8000); - - /* Get the time stamp on the input file. */ - if (stat(path, &stat_buf) < 0) { - error_msg_and_die("Couldn't stat file %s", path); - } - } - - /* Check that the input is sane. */ - if (isatty(fileno(in_file)) && (flags & gunzip_force) == 0) - error_msg_and_die("compressed data not read from terminal. Use -f to force it."); - - /* Set output filename and number */ - if (flags & gunzip_test) { - out_file = xfopen("/dev/null", "w"); /* why does test use filenum 2 ? */ - } else if (flags & gunzip_to_stdout) { - out_file = stdout; - } else { - char *extension; - int length = strlen(path); - - extension = strrchr(path, '.'); - if (extension && strcmp(extension, ".gz") == 0) { - length -= 3; - } else if (extension && strcmp(extension, ".tgz") == 0) { - length -= 4; + if (path == NULL || strcmp(path, "-") == 0) { + in_file = stdin; + flags |= gunzip_to_stdout; } else { - error_msg_and_die("Invalid extension"); + if ((in_file = wfopen(path, "r")) == NULL) { + return -1; + } + if (flags & gunzip_verbose) { + fprintf(stderr, "%s:\t", path); + } + + /* set the buffer size */ + setvbuf(in_file, NULL, _IOFBF, 0x8000); + + /* Get the time stamp on the input file. */ + if (stat(path, &stat_buf) < 0) { + error_msg_and_die("Couldn't stat file %s", path); + } } - out_path = xstrndup(path, length); - - /* Open output file */ - out_file = xfopen(out_path, "w"); - /* Set permissions on the file */ - chmod(out_path, stat_buf.st_mode); - } - - /* do the decompression, and cleanup */ - if (unzip(in_file, out_file) == 0) { - /* Success, remove .gz file */ - if ( !(flags & gunzip_to_stdout )) - delete_path = path; - if (flags & gunzip_verbose) { - fprintf(stderr, "OK\n"); + /* Check that the input is sane. */ + if (isatty(fileno(in_file)) && (flags & gunzip_force) == 0) { + error_msg_and_die + ("compressed data not read from terminal. Use -f to force it."); } - } else { - /* remove failed attempt */ - delete_path = out_path; - } - if (out_file != stdout) - fclose(out_file); - if (in_file != stdin) - fclose(in_file); + /* Set output filename and number */ + if (flags & gunzip_test) { + out_file = xfopen("/dev/null", "w"); /* why does test use filenum 2 ? */ + } else if (flags & gunzip_to_stdout) { + out_file = stdout; + } else { + char *extension; + int length = strlen(path); - if (delete_path && !(flags & gunzip_test)) { - if (unlink(delete_path) < 0) { - error_msg_and_die("Couldn't remove %s", delete_path); + extension = strrchr(path, '.'); + if (extension && strcmp(extension, ".gz") == 0) { + length -= 3; + } else if (extension && strcmp(extension, ".tgz") == 0) { + length -= 4; + } else { + error_msg_and_die("Invalid extension"); + } + out_path = xstrndup(path, length); + + /* Open output file */ + out_file = xfopen(out_path, "w"); + + /* Set permissions on the file */ + chmod(out_path, stat_buf.st_mode); } - } - free(out_path); + /* do the decompression, and cleanup */ + if (unzip(in_file, out_file) == 0) { + /* Success, remove .gz file */ + if (!(flags & gunzip_to_stdout)) { + delete_path = path; + } + if (flags & gunzip_verbose) { + fprintf(stderr, "OK\n"); + } + } else { + /* remove failed attempt */ + delete_path = out_path; + } - return 0; + if (out_file != stdout) { + fclose(out_file); + } + if (in_file != stdin) { + fclose(in_file); + } + + if (delete_path && !(flags & gunzip_test)) { + if (unlink(delete_path) < 0) { + error_msg_and_die("Couldn't remove %s", delete_path); + } + } + + free(out_path); + + return 0; } extern int gunzip_main(int argc, char **argv) @@ -164,8 +169,9 @@ extern int gunzip_main(int argc, char **argv) int status = EXIT_SUCCESS; /* if called as zcat */ - if (strcmp(applet_name, "zcat") == 0) + if (strcmp(applet_name, "zcat") == 0) { flags |= gunzip_to_stdout; + } while ((opt = getopt(argc, argv, "ctfhdqv")) != -1) { switch (opt) { @@ -181,25 +187,27 @@ extern int gunzip_main(int argc, char **argv) case 'v': flags |= gunzip_verbose; break; - case 'd': /* Used to convert gzip to gunzip. */ + case 'd': /* Used to convert gzip to gunzip. */ break; case 'q': error_msg("-q option not supported, ignored"); break; case 'h': default: - show_usage(); /* exit's inside usage */ + show_usage(); /* exit's inside usage */ } } if (optind == argc) { - if (gunzip_file (NULL, flags) < 0) - status = EXIT_FAILURE; + if (gunzip_file(NULL, flags) < 0) { + status = EXIT_FAILURE; + } } else { - for (i = optind; i < argc; i++) { - if (gunzip_file (argv[i], flags) < 0) - status = EXIT_FAILURE; - } + for (i = optind; i < argc; i++) { + if (gunzip_file(argv[i], flags) < 0) { + status = EXIT_FAILURE; + } + } } return status; }