]> git.ipfire.org Git - thirdparty/e2fsprogs.git/commitdiff
libext2fs: fix ext2fs_open2() error for meta_bg image file
authorKazuya Mio <k-mio@sx.jp.nec.com>
Sat, 17 Mar 2018 18:56:15 +0000 (14:56 -0400)
committerTheodore Ts'o <tytso@mit.edu>
Sat, 17 Mar 2018 18:56:15 +0000 (14:56 -0400)
dumpe2fs/debugfs can examine the image file by using the -i option.
However, if meta_bg feature is enabled, dumpe2fs/debugfs cannot open
the image file.

$ dumpe2fs -i test.img
dumpe2fs: Attempt to read block from filesystem resulted in short read while trying to open test.img
Couldn't find valid filesystem superblock.

In case of specifying an image file, the location of block group descriptors
is the same as the case of default filesystem regardless of meta_bg feature.
So if EXT2_FLAG_IMAGE_FILE flag is set in ext2fs_open2(),
don't use the meta_bg handling.

Signed-off-by: Kazuya Mio <k-mio@sx.jp.nec.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
lib/ext2fs/openfs.c

index 506c025bef17a54cac15fd27c195b19f5229e20f..385d6e88c4e406f8d41fe0cf41c908f9eb3d3472 100644 (file)
@@ -416,7 +416,8 @@ errcode_t ext2fs_open2(const char *name, const char *io_options,
 #ifdef WORDS_BIGENDIAN
        groups_per_block = EXT2_DESC_PER_BLOCK(fs->super);
 #endif
-       if (ext2fs_has_feature_meta_bg(fs->super)) {
+       if (ext2fs_has_feature_meta_bg(fs->super) &&
+           !(flags & EXT2_FLAG_IMAGE_FILE)) {
                first_meta_bg = fs->super->s_first_meta_bg;
                if (first_meta_bg > fs->desc_blocks)
                        first_meta_bg = fs->desc_blocks;