Attachment 150286 Details for Bug 232663 – revised patch to correct symlink byteswapping

revised patch to correct symlink byteswapping

e2fsprogs-1.39-symlink_byteswap.patch (text/x-patch), 1.78 KB, created by Bryn M. Reeves on 2007-03-16 22:45:50 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Bryn M. Reeves

Created: 2007-03-16 22:45:50 UTC

Size: 1.78 KB

patch

obsolete

>diff -Nurp e2fsprogs-1.39.orig/e2fsck/pass2.c e2fsprogs-1.39/e2fsck/pass2.c
>--- e2fsprogs-1.39.orig/e2fsck/pass2.c	2006-03-18 21:34:00.000000000 -0500
>+++ e2fsprogs-1.39/e2fsck/pass2.c	2007-03-16 17:52:56.000000000 -0400
>@@ -1187,22 +1187,6 @@ extern int e2fsck_process_bad_inode(e2fs
> 	    !(fs->super->s_feature_compat & EXT2_FEATURE_COMPAT_EXT_ATTR)) {
> 		if (fix_problem(ctx, PR_2_FILE_ACL_ZERO, &pctx)) {
> 			inode.i_file_acl = 0;
>-#ifdef EXT2FS_ENABLE_SWAPFS
>-			/* 
>-			 * This is a special kludge to deal with long
>-			 * symlinks on big endian systems.  i_blocks
>-			 * had already been decremented earlier in
>-			 * pass 1, but since i_file_acl hadn't yet
>-			 * been cleared, ext2fs_read_inode() assumed
>-			 * that the file was short symlink and would
>-			 * not have byte swapped i_block[0].  Hence,
>-			 * we have to byte-swap it here.
>-			 */
>-			if (LINUX_S_ISLNK(inode.i_mode) &&
>-			    (fs->flags & EXT2_FLAG_SWAP_BYTES) &&
>-			    (inode.i_blocks == fs->blocksize >> 9))
>-				inode.i_block[0] = ext2fs_swab32(inode.i_block[0]);
>-#endif
> 			inode_modified++;
> 		} else
> 			not_fixed++;
>diff -Nurp e2fsprogs-1.39.orig/lib/ext2fs/swapfs.c e2fsprogs-1.39/lib/ext2fs/swapfs.c
>--- e2fsprogs-1.39.orig/lib/ext2fs/swapfs.c	2007-03-16 17:49:42.000000000 -0400
>+++ e2fsprogs-1.39/lib/ext2fs/swapfs.c	2007-03-16 17:52:46.000000000 -0400
>@@ -145,7 +145,8 @@ void ext2fs_swap_inode_full(ext2_filsys 
> 	t->i_flags = ext2fs_swab32(f->i_flags);
> 	t->i_file_acl = ext2fs_swab32(f->i_file_acl);
> 	t->i_dir_acl = ext2fs_swab32(f->i_dir_acl);
>-	if (!islnk || ext2fs_inode_data_blocks(fs, (struct ext2_inode *)t)) {
>+	if (!islnk || ext2fs_inode_data_blocks(fs, 
>+			(struct ext2_inode *)(hostorder) ? f : t)) {
> 		for (i = 0; i < EXT2_N_BLOCKS; i++)
> 			t->i_block[i] = ext2fs_swab32(f->i_block[i]);
> 	} else if (t != f) {

Actions: View

Attachments on bug 232663: 150239 | 150246 | 150286 | 152294 | 152295 | 152296