Whamcloud - gitweb
e2fsck: skip quota update when interrupted
[tools/e2fsprogs.git] / e2fsck / unix.c
index 6471592..9d9ba04 100644 (file)
@@ -77,13 +77,13 @@ static void usage(e2fsck_t ctx)
                "\t\t[-E extended-options] device\n"),
                ctx->program_name);
 
-       fprintf(stderr, _("\nEmergency help:\n"
+       fprintf(stderr, "%s", _("\nEmergency help:\n"
                " -p                   Automatic repair (no questions)\n"
                " -n                   Make no changes to the filesystem\n"
                " -y                   Assume \"yes\" to all questions\n"
                " -c                   Check for bad blocks and add them to the badblock list\n"
                " -f                   Force checking even if filesystem is marked clean\n"));
-       fprintf(stderr, _(""
+       fprintf(stderr, "%s", _(""
                " -v                   Be verbose\n"
                " -b superblock        Use alternative superblock\n"
                " -B blocksize         Force blocksize when looking for superblock\n"
@@ -176,7 +176,7 @@ static void show_stats(e2fsck_t     ctx)
                if (ctx->extent_depth_count[j])
                        break;
        if (++j) {
-               log_out(ctx, _("             Extent depth histogram: "));
+               log_out(ctx, "%s", _("             Extent depth histogram: "));
                for (i=0; i < j; i++) {
                        if (i)
                                fputc('/', stdout);
@@ -249,26 +249,30 @@ static void check_mount(e2fsck_t ctx)
             ((ctx->options & E2F_OPT_FORCE) &&
              (ctx->mount_flags & EXT2_MF_READONLY))) &&
            !(ctx->options & E2F_OPT_WRITECHECK)) {
-               log_out(ctx, _("Warning!  %s is %s.\n"),
-                       ctx->filesystem_name,
-                       ctx->mount_flags & EXT2_MF_MOUNTED ?
-                               "mounted" : "in use");
+               if (ctx->mount_flags & EXT2_MF_MOUNTED)
+                       log_out(ctx, _("Warning!  %s is mounted.\n"),
+                                       ctx->filesystem_name);
+               else
+                       log_out(ctx, _("Warning!  %s is in use.\n"),
+                                       ctx->filesystem_name);
                return;
        }
 
-       log_out(ctx, _("%s is %s.\n"), ctx->filesystem_name,
-               ctx->mount_flags & EXT2_MF_MOUNTED ? "mounted" : "in use");
+       if (ctx->mount_flags & EXT2_MF_MOUNTED)
+               log_out(ctx, _("%s is mounted.\n"), ctx->filesystem_name);
+       else
+               log_out(ctx, _("%s is in use.\n"), ctx->filesystem_name);
        if (!ctx->interactive || ctx->mount_flags & EXT2_MF_BUSY)
                fatal_error(ctx, _("Cannot continue, aborting.\n\n"));
        puts("\007\007\007\007");
-       log_out(ctx, _("\n\nWARNING!!!  "
+       log_out(ctx, "%s", _("\n\nWARNING!!!  "
                       "The filesystem is mounted.   "
                       "If you continue you ***WILL***\n"
                       "cause ***SEVERE*** filesystem damage.\n\n"));
        puts("\007\007\007");
        cont = ask_yn(ctx, _("Do you really want to continue"), 0);
        if (!cont) {
-               printf (_("check aborted.\n"));
+               printf("%s", _("check aborted.\n"));
                exit (0);
        }
        return;
@@ -337,6 +341,9 @@ static void check_if_skip(e2fsck_t ctx)
        int broken_system_clock;
        time_t lastcheck;
 
+       if (ctx->flags & E2F_FLAG_PROBLEMS_FIXED)
+               return;
+
        profile_get_boolean(ctx->profile, "options", "broken_system_clock",
                            0, 0, &broken_system_clock);
        if (ctx->flags & E2F_FLAG_TIME_INSANE)
@@ -388,7 +395,7 @@ static void check_if_skip(e2fsck_t ctx)
        if (reason) {
                log_out(ctx, "%s", ctx->device_name);
                log_out(ctx, reason, reason_arg);
-               log_out(ctx, _(", check forced.\n"));
+               log_out(ctx, "%s", _(", check forced.\n"));
                return;
        }
 
@@ -440,17 +447,18 @@ static void check_if_skip(e2fsck_t ctx)
        if (next_check <= 5) {
                if (next_check == 1) {
                        if (batt)
-                               log_out(ctx, _(" (check deferred; "
-                                              "on battery)"));
+                               log_out(ctx, "%s",
+                                       _(" (check deferred; on battery)"));
                        else
-                               log_out(ctx, _(" (check after next mount)"));
+                               log_out(ctx, "%s",
+                                       _(" (check after next mount)"));
                } else
-                       log_out(ctx, _(" (check in %ld mounts)"), next_check);
+                       log_out(ctx, _(" (check in %ld mounts)"),
+                               next_check);
        }
        log_out(ctx, "\n");
 skip:
-       ext2fs_close(fs);
-       ctx->fs = NULL;
+       ext2fs_close_free(&ctx->fs);
        e2fsck_free_context(ctx);
        exit(FSCK_OK);
 }
@@ -462,7 +470,7 @@ struct percent_tbl {
        int     max_pass;
        int     table[32];
 };
-struct percent_tbl e2fsck_tbl = {
+static struct percent_tbl e2fsck_tbl = {
        5, { 0, 70, 90, 92,  95, 100 }
 };
 static char bar[128], spaces[128];
@@ -481,7 +489,7 @@ static float calc_percent(struct percent_tbl *tbl, int pass, int curr,
                + tbl->table[pass-1]);
 }
 
-extern void e2fsck_clear_progbar(e2fsck_t ctx)
+void e2fsck_clear_progbar(e2fsck_t ctx)
 {
        if (!(ctx->flags & E2F_FLAG_PROG_BAR))
                return;
@@ -663,7 +671,7 @@ static void parse_extended_opts(e2fsck_t ctx, const char *opts)
                        ea_ver = strtoul(arg, &p, 0);
                        if (*p ||
                            ((ea_ver != 1) && (ea_ver != 2))) {
-                               fprintf(stderr,
+                               fprintf(stderr, "%s",
                                        _("Invalid EA version.\n"));
                                extended_usage++;
                                continue;
@@ -747,6 +755,7 @@ static errcode_t PRS(int argc, char *argv[], e2fsck_t *ret_ctx)
                return retval;
 
        *ret_ctx = ctx;
+       e2fsck_global_ctx = ctx;
 
        setvbuf(stdout, NULL, _IONBF, BUFSIZ);
        setvbuf(stderr, NULL, _IONBF, BUFSIZ);
@@ -869,6 +878,8 @@ static errcode_t PRS(int argc, char *argv[], e2fsck_t *ret_ctx)
                case 'L':
                        replace_bad_blocks++;
                case 'l':
+                       if (bad_blocks_file)
+                               free(bad_blocks_file);
                        bad_blocks_file = string_copy(ctx, optarg, 0);
                        break;
                case 'd':
@@ -906,17 +917,17 @@ static errcode_t PRS(int argc, char *argv[], e2fsck_t *ret_ctx)
                usage(ctx);
        if ((ctx->options & E2F_OPT_NO) &&
            (ctx->options & E2F_OPT_COMPRESS_DIRS)) {
-               com_err(ctx->program_name, 0,
+               com_err(ctx->program_name, 0, "%s",
                        _("The -n and -D options are incompatible."));
                fatal_error(ctx, 0);
        }
        if ((ctx->options & E2F_OPT_NO) && cflag) {
-               com_err(ctx->program_name, 0,
+               com_err(ctx->program_name, 0, "%s",
                        _("The -n and -c options are incompatible."));
                fatal_error(ctx, 0);
        }
        if ((ctx->options & E2F_OPT_NO) && bad_blocks_file) {
-               com_err(ctx->program_name, 0,
+               com_err(ctx->program_name, 0, "%s",
                        _("The -n and -l/-L options are incompatible."));
                fatal_error(ctx, 0);
        }
@@ -971,8 +982,8 @@ static errcode_t PRS(int argc, char *argv[], e2fsck_t *ret_ctx)
                close(fd);
        }
        if (cflag && bad_blocks_file) {
-               fprintf(stderr, _("The -c and the -l/-L options may "
-                                 "not be both used at the same time.\n"));
+               fprintf(stderr, "%s", _("The -c and the -l/-L options may not "
+                                       "be both used at the same time.\n"));
                exit(FSCK_USAGE);
        }
 #ifdef HAVE_SIGNAL_H
@@ -986,7 +997,6 @@ static errcode_t PRS(int argc, char *argv[], e2fsck_t *ret_ctx)
 #ifdef SA_RESTART
        sa.sa_flags = SA_RESTART;
 #endif
-       e2fsck_global_ctx = ctx;
        sa.sa_handler = signal_progress_on;
        sigaction(SIGUSR1, &sa, 0);
        sa.sa_handler = signal_progress_off;
@@ -1062,16 +1072,18 @@ static errcode_t try_open_fs(e2fsck_t ctx, int flags, io_manager io_ptr,
                retval = ext2fs_open2(ctx->filesystem_name, ctx->io_options,
                                      flags, 0, 0, io_ptr, ret_fs);
 
-       if (ret_fs)
+       if (retval == 0) {
+               (*ret_fs)->priv_data = ctx;
                e2fsck_set_bitmap_type(*ret_fs, EXT2FS_BMAP64_RBTREE,
                                       "default", NULL);
+       }
        return retval;
 }
 
 static const char *my_ver_string = E2FSPROGS_VERSION;
 static const char *my_ver_date = E2FSPROGS_DATE;
 
-static int e2fsck_check_mmp(ext2_filsys fs, e2fsck_t ctx)
+static errcode_t e2fsck_check_mmp(ext2_filsys fs, e2fsck_t ctx)
 {
        struct mmp_struct *mmp_s;
        unsigned int mmp_check_interval;
@@ -1133,12 +1145,12 @@ check_error:
                        retval = 0;
                }
        } else if (retval == EXT2_ET_MMP_FAILED) {
-               com_err(ctx->program_name, retval,
+               com_err(ctx->program_name, retval, "%s",
                        _("while checking MMP block"));
                dump_mmp_msg(fs->mmp_buf, NULL);
        } else if (retval == EXT2_ET_MMP_FSCK_ON ||
                   retval == EXT2_ET_MMP_UNKNOWN_SEQ) {
-               com_err(ctx->program_name, retval,
+               com_err(ctx->program_name, retval, "%s",
                        _("while checking MMP block"));
                dump_mmp_msg(fs->mmp_buf,
                             _("If you are sure the filesystem is not "
@@ -1163,9 +1175,9 @@ int main (int argc, char *argv[])
        const char      *lib_ver_date;
        int             my_ver, lib_ver;
        e2fsck_t        ctx;
-       blk_t           orig_superblock;
+       blk64_t         orig_superblock;
        struct problem_context pctx;
-       int flags, run_result;
+       int flags, run_result, was_changed;
        int journal_size;
        int sysval, sys_page_size = 4096;
        int old_bitmaps;
@@ -1191,14 +1203,14 @@ int main (int argc, char *argv[])
        my_ver = ext2fs_parse_version_string(my_ver_string);
        lib_ver = ext2fs_get_library_version(0, &lib_ver_date);
        if (my_ver > lib_ver) {
-               fprintf( stderr, _("Error: ext2fs library version "
-                       "out of date!\n"));
+               fprintf( stderr, "%s",
+                        _("Error: ext2fs library version out of date!\n"));
                show_version_only++;
        }
 
        retval = PRS(argc, argv, &ctx);
        if (retval) {
-               com_err("e2fsck", retval,
+               com_err("e2fsck", retval, "%s",
                        _("while trying to initialize program"));
                exit(FSCK_ERROR);
        }
@@ -1262,6 +1274,7 @@ restart:
                        flags &= ~EXT2_FLAG_EXCLUSIVE;
        }
 
+       ctx->openfs_flags = flags;
        retval = try_open_fs(ctx, flags, io_ptr, &fs);
 
        if (!ctx->superblock && !(ctx->options & E2F_OPT_PREEN) &&
@@ -1294,12 +1307,12 @@ restart:
                        orig_superblock = ctx->superblock;
                        get_backup_sb(ctx, fs, ctx->filesystem_name, io_ptr);
                        if (fs)
-                               ext2fs_close(fs);
+                               ext2fs_close_free(&fs);
                        orig_retval = retval;
                        retval = try_open_fs(ctx, flags, io_ptr, &fs);
                        if ((orig_retval == 0) && retval != 0) {
                                if (fs)
-                                       ext2fs_close(fs);
+                                       ext2fs_close_free(&fs);
                                log_out(ctx, _("%s: %s while using the "
                                               "backup blocks"),
                                        ctx->program_name,
@@ -1332,32 +1345,34 @@ failure:
                com_err(ctx->program_name, retval, _("while trying to open %s"),
                        ctx->filesystem_name);
                if (retval == EXT2_ET_REV_TOO_HIGH) {
-                       log_out(ctx, _("The filesystem revision is apparently "
-                              "too high for this version of e2fsck.\n"
-                              "(Or the filesystem superblock "
-                              "is corrupt)\n\n"));
+                       log_out(ctx, "%s",
+                               _("The filesystem revision is apparently "
+                                 "too high for this version of e2fsck.\n"
+                                 "(Or the filesystem superblock "
+                                 "is corrupt)\n\n"));
                        fix_problem(ctx, PR_0_SB_CORRUPT, &pctx);
                } else if (retval == EXT2_ET_SHORT_READ)
-                       log_out(ctx, _("Could this be a zero-length "
-                                      "partition?\n"));
+                       log_out(ctx, "%s",
+                               _("Could this be a zero-length partition?\n"));
                else if ((retval == EPERM) || (retval == EACCES))
                        log_out(ctx, _("You must have %s access to the "
-                              "filesystem or be root\n"),
+                                      "filesystem or be root\n"),
                               (ctx->options & E2F_OPT_READONLY) ?
                               "r/o" : "r/w");
                else if (retval == ENXIO)
-                       log_out(ctx, _("Possibly non-existent or "
-                                      "swap device?\n"));
+                       log_out(ctx, "%s",
+                               _("Possibly non-existent or swap device?\n"));
                else if (retval == EBUSY)
-                       log_out(ctx, _("Filesystem mounted or opened "
-                                "exclusively by another program?\n"));
+                       log_out(ctx, "%s", _("Filesystem mounted or opened "
+                                        "exclusively by another program?\n"));
                else if (retval == ENOENT)
-                       log_out(ctx, _("Possibly non-existent device?\n"));
+                       log_out(ctx, "%s",
+                               _("Possibly non-existent device?\n"));
 #ifdef EROFS
                else if (retval == EROFS)
-                       log_out(ctx, _("Disk write-protected; use the -n "
-                                      "option to do a read-only\n"
-                                      "check of the device.\n"));
+                       log_out(ctx, "%s", _("Disk write-protected; use the "
+                                            "-n option to do a read-only\n"
+                                            "check of the device.\n"));
 #endif
                else
                        fix_problem(ctx, PR_0_SB_CORRUPT, &pctx);
@@ -1391,7 +1406,7 @@ failure:
                 * reopen the filesystem after we get the device size.
                 */
                if (pctx.errcode == EBUSY) {
-                       ext2fs_close(fs);
+                       ext2fs_close_free(&fs);
                        need_restart++;
                        pctx.errcode =
                                ext2fs_get_device_size2(ctx->filesystem_name,
@@ -1411,7 +1426,6 @@ failure:
        }
 
        ctx->fs = fs;
-       fs->priv_data = ctx;
        fs->now = ctx->now;
        sb = fs->super;
 
@@ -1448,8 +1462,7 @@ failure:
                /*
                 * Restart in order to reopen fs but this time start mmp.
                 */
-               ext2fs_close(fs);
-               ctx->fs = NULL;
+               ext2fs_close_free(&ctx->fs);
                flags &= ~EXT2_FLAG_SKIP_MMP;
                goto restart;
        }
@@ -1475,9 +1488,9 @@ failure:
         */
        if (sb->s_feature_incompat & EXT3_FEATURE_INCOMPAT_RECOVER) {
                if (ctx->options & E2F_OPT_READONLY) {
-                       log_out(ctx, _("Warning: skipping journal recovery "
-                                      "because doing a read-only filesystem "
-                                      "check.\n"));
+                       log_out(ctx, "%s",
+                               _("Warning: skipping journal recovery because "
+                                 "doing a read-only filesystem check.\n"));
                        io_channel_flush(ctx->fs->io);
                } else {
                        if (ctx->flags & E2F_FLAG_RESTARTED) {
@@ -1488,7 +1501,8 @@ failure:
                                 * device driver is being bogus.
                                 */
                                com_err(ctx->program_name, 0,
-                                       _("unable to set superblock flags on %s\n"), ctx->device_name);
+                                       _("unable to set superblock flags "
+                                         "on %s\n"), ctx->device_name);
                                fatal_error(ctx, 0);
                        }
                        retval = e2fsck_run_ext3_journal(ctx);
@@ -1498,8 +1512,7 @@ failure:
                                        ctx->device_name);
                                fatal_error(ctx, 0);
                        }
-                       ext2fs_close(ctx->fs);
-                       ctx->fs = 0;
+                       ext2fs_close_free(&ctx->fs);
                        ctx->flags |= E2F_FLAG_RESTARTED;
                        goto restart;
                }
@@ -1594,8 +1607,8 @@ print_unsupp_features:
                log_out(ctx, _("%s: %s while reading bad blocks inode\n"),
                        ctx->program_name, error_message(retval));
                preenhalt(ctx);
-               log_out(ctx, _("This doesn't bode well, "
-                              "but we'll try to go on...\n"));
+               log_out(ctx, "%s", _("This doesn't bode well, "
+                                    "but we'll try to go on...\n"));
        }
 
        /*
@@ -1623,7 +1636,8 @@ print_unsupp_features:
        run_result = e2fsck_run(ctx);
        e2fsck_clear_progbar(ctx);
 
-       if (ctx->flags & E2F_FLAG_JOURNAL_INODE) {
+       if (!ctx->invalid_bitmaps &&
+           (ctx->flags & E2F_FLAG_JOURNAL_INODE)) {
                if (fix_problem(ctx, PR_6_RECREATE_JOURNAL, &pctx)) {
                        if (journal_size < 1024)
                                journal_size = ext2fs_default_journal_size(ext2fs_blocks_count(fs->super));
@@ -1645,15 +1659,23 @@ print_unsupp_features:
                                        "journal\n", error_message(retval));
                                goto no_journal;
                        }
-                       log_out(ctx, _(" Done.\n"));
-                       log_out(ctx, _("\n*** journal has been re-created - "
-                                      "filesystem is now ext3 again ***\n"));
+                       log_out(ctx, "%s", _(" Done.\n"));
+                       log_out(ctx, "%s",
+                               _("\n*** journal has been re-created - "
+                                 "filesystem is now ext3 again ***\n"));
                }
        }
 no_journal:
 
-       if (ctx->qctx) {
+       if (run_result & E2F_FLAG_ABORT) {
+               fatal_error(ctx, _("aborted"));
+       } else if (run_result & E2F_FLAG_CANCEL) {
+               log_out(ctx, _("%s: e2fsck canceled.\n"), ctx->device_name ?
+                       ctx->device_name : ctx->filesystem_name);
+               exit_value |= FSCK_CANCELED;
+       } else if (ctx->qctx && !ctx->invalid_bitmaps) {
                int i, needs_writeout;
+
                for (i = 0; i < MAXQUOTAS; i++) {
                        if (qtype != -1 && qtype != i)
                                continue;
@@ -1669,32 +1691,53 @@ no_journal:
        }
 
        if (run_result == E2F_FLAG_RESTART) {
-               log_out(ctx, _("Restarting e2fsck from the beginning...\n"));
+               log_out(ctx, "%s",
+                       _("Restarting e2fsck from the beginning...\n"));
                retval = e2fsck_reset_context(ctx);
                if (retval) {
-                       com_err(ctx->program_name, retval,
+                       com_err(ctx->program_name, retval, "%s",
                                _("while resetting context"));
                        fatal_error(ctx, 0);
                }
-               ext2fs_close(fs);
+               ext2fs_close_free(&ctx->fs);
                goto restart;
        }
-       if (run_result & E2F_FLAG_CANCEL) {
-               log_out(ctx, _("%s: e2fsck canceled.\n"), ctx->device_name ?
-                       ctx->device_name : ctx->filesystem_name);
-               exit_value |= FSCK_CANCELED;
-       }
-       if (run_result & E2F_FLAG_ABORT)
-               fatal_error(ctx, _("aborted"));
-       if (check_backup_super_block(ctx)) {
-               fs->flags &= ~EXT2_FLAG_MASTER_SB_ONLY;
-               ext2fs_mark_super_dirty(fs);
-       }
 
 #ifdef MTRACE
        mtrace_print("Cleanup");
 #endif
-       if (ext2fs_test_changed(fs)) {
+       was_changed = ext2fs_test_changed(fs);
+       if (!(ctx->flags & E2F_FLAG_RUN_RETURN) &&
+           !(ctx->options & E2F_OPT_READONLY)) {
+               if (ext2fs_test_valid(fs)) {
+                       if (!(sb->s_state & EXT2_VALID_FS))
+                               exit_value |= FSCK_NONDESTRUCT;
+                       sb->s_state = EXT2_VALID_FS;
+                       if (check_backup_super_block(ctx))
+                               fs->flags &= ~EXT2_FLAG_MASTER_SB_ONLY;
+               } else
+                       sb->s_state &= ~EXT2_VALID_FS;
+               if (!(ctx->flags & E2F_FLAG_TIME_INSANE))
+                       sb->s_lastcheck = ctx->now;
+               sb->s_mnt_count = 0;
+               memset(((char *) sb) + EXT4_S_ERR_START, 0, EXT4_S_ERR_LEN);
+               pctx.errcode = ext2fs_set_gdt_csum(ctx->fs);
+               if (pctx.errcode)
+                       fix_problem(ctx, PR_6_SET_BG_CHECKSUM, &pctx);
+               ext2fs_mark_super_dirty(fs);
+       }
+
+       e2fsck_write_bitmaps(ctx);
+       if (fs->flags & EXT2_FLAG_DIRTY) {
+               pctx.errcode = ext2fs_flush(ctx->fs);
+               if (pctx.errcode)
+                       fix_problem(ctx, PR_6_FLUSH_FILESYSTEM, &pctx);
+       }
+       pctx.errcode = io_channel_flush(ctx->fs->io);
+       if (pctx.errcode)
+               fix_problem(ctx, PR_6_IO_FLUSH, &pctx);
+
+       if (was_changed) {
                exit_value |= FSCK_NONDESTRUCT;
                if (!(ctx->options & E2F_OPT_PREEN))
                        log_out(ctx, _("\n%s: ***** FILE SYSTEM WAS "
@@ -1725,41 +1768,12 @@ no_journal:
                    (sb->s_state & EXT2_VALID_FS) &&
                    !(sb->s_state & EXT2_ERROR_FS))
                        exit_value = 0;
-       } else {
+       } else
                show_stats(ctx);
-               if (!(ctx->options & E2F_OPT_READONLY)) {
-                       if (ext2fs_test_valid(fs)) {
-                               if (!(sb->s_state & EXT2_VALID_FS))
-                                       exit_value |= FSCK_NONDESTRUCT;
-                               sb->s_state = EXT2_VALID_FS;
-                       } else
-                               sb->s_state &= ~EXT2_VALID_FS;
-                       sb->s_mnt_count = 0;
-                       if (!(ctx->flags & E2F_FLAG_TIME_INSANE))
-                               sb->s_lastcheck = ctx->now;
-                       memset(((char *) sb) + EXT4_S_ERR_START, 0,
-                              EXT4_S_ERR_LEN);
-                       ext2fs_mark_super_dirty(fs);
-               }
-       }
 
-       if ((run_result & E2F_FLAG_CANCEL) == 0 &&
-           sb->s_feature_ro_compat & EXT4_FEATURE_RO_COMPAT_GDT_CSUM &&
-           !(ctx->options & E2F_OPT_READONLY)) {
-               retval = ext2fs_set_gdt_csum(ctx->fs);
-               if (retval) {
-                       com_err(ctx->program_name, retval,
-                               _("while setting block group checksum info"));
-                       fatal_error(ctx, 0);
-               }
-       }
-
-       e2fsck_write_bitmaps(ctx);
-       io_channel_flush(ctx->fs->io);
        print_resource_track(ctx, NULL, &ctx->global_rtrack, ctx->fs->io);
 
-       ext2fs_close(fs);
-       ctx->fs = NULL;
+       ext2fs_close_free(&ctx->fs);
        free(ctx->journal_name);
 
        e2fsck_free_context(ctx);