From fb6627a3fb4fbcb6904716c7a59e5ef9eb5dfafb Mon Sep 17 00:00:00 2001 From: Eric Sandeen Date: Mon, 24 Mar 2008 08:25:40 -0400 Subject: [PATCH] libext2fs: Fix byte-swap logic for i_block[] in inodes with extents ext2fs_swap_inode_full() was incorrectly swapping the i_block array for extents. Signed-off-by: Eric Sandeen Signed-off-by: "Theodore Ts'o" --- lib/ext2fs/swapfs.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/lib/ext2fs/swapfs.c b/lib/ext2fs/swapfs.c index 132f6f3a..33c7ab38 100644 --- a/lib/ext2fs/swapfs.c +++ b/lib/ext2fs/swapfs.c @@ -147,6 +147,7 @@ void ext2fs_swap_inode_full(ext2_filsys fs, struct ext2_inode_large *t, int bufsize) { unsigned i, has_data_blocks, extra_isize, attr_magic; + int has_extents = 0; int islnk = 0; __u32 *eaf, *eat; @@ -172,12 +173,13 @@ void ext2fs_swap_inode_full(ext2_filsys fs, struct ext2_inode_large *t, has_data_blocks = ext2fs_inode_data_blocks(fs, (struct ext2_inode *) t); if (hostorder && (f->i_flags & EXT4_EXTENTS_FL)) - has_data_blocks = 0; + has_extents = 1; t->i_flags = ext2fs_swab32(f->i_flags); - if (hostorder && (t->i_flags & EXT4_EXTENTS_FL)) - has_data_blocks = 0; + if (!hostorder && (t->i_flags & EXT4_EXTENTS_FL)) + has_extents = 1; t->i_dir_acl = ext2fs_swab32(f->i_dir_acl); - if (!islnk || has_data_blocks ) { + /* extent data are swapped on access, not here */ + if (!has_extents && (!islnk || has_data_blocks)) { for (i = 0; i < EXT2_N_BLOCKS; i++) t->i_block[i] = ext2fs_swab32(f->i_block[i]); } else if (t != f) { -- 2.11.0