fs: ext4: do not write on filesystem with metadata_csum feature
authorSébastien Szymanski <sebastien.szymanski@armadeus.com>
Fri, 22 Mar 2019 08:33:52 +0000 (09:33 +0100)
committerTom Rini <trini@konsulko.com>
Fri, 22 Mar 2019 16:15:24 +0000 (12:15 -0400)
U-Boot doesn't support metadata_csum feature. Writing to filesystem with
metadata_csum feature makes the filesystem corrupted and unbootable by
Linux:

[    2.527495] EXT4-fs (mmcblk0p2): ext4_check_descriptors: Checksum for group 0 failed (52188!=0)
[    2.537421] EXT4-fs (mmcblk0p2): ext4_check_descriptors: Checksum for group 1 failed (5262!=0)
...
[    2.653308] EXT4-fs (mmcblk0p2): ext4_check_descriptors: Checksum for group 14 failed (42611!=0)
[    2.662179] EXT4-fs (mmcblk0p2): ext4_check_descriptors: Checksum for group 15 failed (21527!=0)
[    2.687920] JBD2: journal checksum error
[    2.691982] EXT4-fs (mmcblk0p2): error loading journal
[    2.698292] VFS: Cannot open root device "mmcblk0p2" or unknown-block(179,2): error -74

Don't write to filesystem with meatadata_csum feature to not corrupt the
filesystem.

Signed-off-by: Sébastien Szymanski <sebastien.szymanski@armadeus.com>
fs/ext4/ext4_write.c
include/ext4fs.h

index a7f543f7df7ddc0ed905734f30e864c90bdd1385..4eb77c327ef32a9bd4a6f98280b3c660d85e12e6 100644 (file)
@@ -864,6 +864,12 @@ int ext4fs_write(const char *fname, unsigned char *buffer,
                printf("error in File System init\n");
                return -1;
        }
+
+       if (le32_to_cpu(fs->sb->feature_ro_compat) & EXT4_FEATURE_RO_COMPAT_METADATA_CSUM) {
+               printf("Unsupported feature metadata_csum found, not writing.\n");
+               return -1;
+       }
+
        inodes_per_block = fs->blksz / fs->inodesz;
        parent_inodeno = ext4fs_get_parent_inode_num(fname, filename, F_FILE);
        if (parent_inodeno == -1)
index bb556391072516a6fee6ffb662ab552e96ad7c9e..24210113411a73f3b727277f59d491b952b55057 100644 (file)
@@ -32,6 +32,7 @@
 #define EXT4_EXTENTS_FL                0x00080000 /* Inode uses extents */
 #define EXT4_EXT_MAGIC                 0xf30a
 #define EXT4_FEATURE_RO_COMPAT_GDT_CSUM        0x0010
+#define EXT4_FEATURE_RO_COMPAT_METADATA_CSUM 0x0400
 #define EXT4_FEATURE_INCOMPAT_EXTENTS  0x0040
 #define EXT4_FEATURE_INCOMPAT_64BIT    0x0080
 #define EXT4_INDIRECT_BLOCKS           12