]> git.ipfire.org Git - thirdparty/e2fsprogs.git/commitdiff
e2fsck: add option to force encoded filename verification
authorGabriel Krisman Bertazi <krisman@collabora.com>
Thu, 17 Dec 2020 17:35:42 +0000 (18:35 +0100)
committerTheodore Ts'o <tytso@mit.edu>
Thu, 28 Jan 2021 03:18:53 +0000 (22:18 -0500)
This is interesting for !strict filesystems as part of the encoding
update procedure. Once the filesystem is known to not have badly encoded
filenames, the update is trivial, thanks to the stability of assigned
code points in the unicode specification.

Signed-off-by: Gabriel Krisman Bertazi <krisman@collabora.com>
Signed-off-by: Arnaud Ferraris <arnaud.ferraris@collabora.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
e2fsck/e2fsck.h
e2fsck/pass2.c
e2fsck/unix.c

index 32159d2783ee2846715a95a09304aabf9337096c..0e910a92a59b785ba1dfa9627d405beae2447ed0 100644 (file)
@@ -178,6 +178,7 @@ struct resource_track {
 #define E2F_OPT_ICOUNT_FULLMAP 0x20000 /* use an array for inode counts */
 #define E2F_OPT_UNSHARE_BLOCKS  0x40000
 #define E2F_OPT_CLEAR_UNINIT   0x80000 /* Hack to clear the uninit bit */
+#define E2F_OPT_CHECK_ENCODING  0x100000 /* Force verification of encoded filenames */
 
 /*
  * E2fsck flags
index f23a3a9c643367b28a86e034fa92896a7c2d0381..7e961e82fbeabdfde105a56e966a93d959b077c3 100644 (file)
@@ -1049,9 +1049,10 @@ static int check_dir_block(ext2_filsys fs,
        ctx = cd->ctx;
 
        /* We only want filename encoding verification on strict
-        * mode. */
+        * mode or if explicitly requested by user. */
        if (ext2fs_test_inode_bitmap2(ctx->inode_casefold_map, ino) &&
-           (ctx->fs->super->s_encoding_flags & EXT4_ENC_STRICT_MODE_FL))
+           ((ctx->fs->super->s_encoding_flags & EXT4_ENC_STRICT_MODE_FL) ||
+            (ctx->options & E2F_OPT_CHECK_ENCODING)))
                cf_dir = 1;
 
        if (ctx->flags & E2F_FLAG_RUN_RETURN)
index 4fe5fe4cbc711d84619a5b3478fe26f034a9c4c3..08f1863dabc0e4e194069a0b8745fec330b17a0c 100644 (file)
@@ -753,6 +753,9 @@ static void parse_extended_opts(e2fsck_t ctx, const char *opts)
                        ctx->options |= E2F_OPT_UNSHARE_BLOCKS;
                        ctx->options |= E2F_OPT_FORCE;
                        continue;
+               } else if (strcmp(token, "check_encoding") == 0) {
+                       ctx->options |= E2F_OPT_CHECK_ENCODING;
+                       continue;
 #ifdef CONFIG_DEVELOPER_FEATURES
                } else if (strcmp(token, "clear_all_uninit_bits") == 0) {
                        ctx->options |= E2F_OPT_CLEAR_UNINIT;
@@ -784,6 +787,7 @@ static void parse_extended_opts(e2fsck_t ctx, const char *opts)
                fputs("\tbmap2extent\n", stderr);
                fputs("\tunshare_blocks\n", stderr);
                fputs("\tfixes_only\n", stderr);
+               fputs("\tcheck_encoding\n", stderr);
                fputc('\n', stderr);
                exit(1);
        }