diff options
| author | Theodore Ts'o <tytso@mit.edu> | 2006-03-08 18:22:06 -0500 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2006-03-08 18:22:06 -0500 |
| commit | 6a2efe047d19d86d1a78f7fc4ff1bf256437c267 (patch) | |
| tree | 2370f540d804b0c05db140fc49910ce669805e3c | |
| parent | d2d0d79861a4744aaf1051d8eb81a15d6f5e4b8f (diff) | |
| download | e2fsprogs-6a2efe047d19d86d1a78f7fc4ff1bf256437c267.tar.gz | |
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 <adilger@clusterfs.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
| -rw-r--r-- | debugfs/ChangeLog | 5 | ||||
| -rw-r--r-- | debugfs/icheck.c | 4 |
2 files changed, 7 insertions, 2 deletions
diff --git a/debugfs/ChangeLog b/debugfs/ChangeLog index 3d7193d0..f5bc0012 100644 --- a/debugfs/ChangeLog +++ b/debugfs/ChangeLog @@ -1,3 +1,8 @@ +2006-03-08 Theodore Ts'o <tytso@mit.edu> + + * 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 <tytso@mit.edu> * debugfs.c, debugfs.h, logdump.c, ls.c: Clean up various gcc diff --git a/debugfs/icheck.c b/debugfs/icheck.c index 6c4b0a11..e3646def 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) { |