From 6a2efe047d19d86d1a78f7fc4ff1bf256437c267 Mon Sep 17 00:00:00 2001 From: Theodore Ts'o Date: Wed, 8 Mar 2006 18:22:06 -0500 Subject: [PATCH] Fix debugfs's icheck to correctly report the owner of an EA block Hello Ted, here is a minor fixup with debugfs "icheck" finding a block in an EA. It was reporting the inode number as one too low because it was set after the EA check is done. From: Andreas Dilger Signed-off-by: "Theodore Ts'o" --- debugfs/ChangeLog | 5 +++++ debugfs/icheck.c | 4 ++-- 2 files changed, 7 insertions(+), 2 deletions(-) diff --git a/debugfs/ChangeLog b/debugfs/ChangeLog index 3d7193d..f5bc001 100644 --- a/debugfs/ChangeLog +++ b/debugfs/ChangeLog @@ -1,3 +1,8 @@ +2006-03-08 Theodore Ts'o + + * Fix bug in icheck which incorrectly reports the last valid inode + number as the owner for an EA block. + 2005-12-10 Theodore Ts'o * debugfs.c, debugfs.h, logdump.c, ls.c: Clean up various gcc diff --git a/debugfs/icheck.c b/debugfs/icheck.c index 6c4b0a1..e3646de 100644 --- a/debugfs/icheck.c +++ b/debugfs/icheck.c @@ -109,6 +109,8 @@ void do_icheck(int argc, char **argv) if (!inode.i_links_count) goto next; + bw.inode = ino; + if (inode.i_file_acl) { icheck_proc(current_fs, &inode.i_file_acl, 0, 0, 0, &bw); @@ -125,8 +127,6 @@ void do_icheck(int argc, char **argv) if (inode.i_dtime) goto next; - bw.inode = ino; - retval = ext2fs_block_iterate2(current_fs, ino, 0, block_buf, icheck_proc, &bw); if (retval) { -- 1.8.3.1