diff options
| author | Theodore Ts'o <tytso@mit.edu> | 2007-03-31 18:56:09 -0400 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2007-03-31 18:56:09 -0400 |
| commit | 441ab1e02096e3c968cae6ddfde73140fcad8199 (patch) | |
| tree | cd5afa986bd4ed2de7dfa16a9461f860d54ca18f /e2fsck/ChangeLog | |
| parent | 971fe056305c7f57629d609490701e13298e658b (diff) | |
| download | e2fsprogs-441ab1e02096e3c968cae6ddfde73140fcad8199.tar.gz | |
Fix edge case when i_size doesn't get fixed until second e2fsck run
Don't assume that a special device is bogus just because i_blocks is
non-zero. The i_blocks field could get adjusted later, and if this
happens it will confuse the e2fsck_process_bad_inode() in pass 2. In
practice true garbage inodes will have random non-zero in
i_blocks[4..15], so there's no point doing the check for an illegal
i_blocks value.
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'e2fsck/ChangeLog')
| -rw-r--r-- | e2fsck/ChangeLog | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/e2fsck/ChangeLog b/e2fsck/ChangeLog index 93bb5f54..1ea3b1e7 100644 --- a/e2fsck/ChangeLog +++ b/e2fsck/ChangeLog @@ -1,3 +1,13 @@ +2007-03-31 Theodore Tso <tytso@mit.edu> + + * pass1.c (e2fsck_pass1_check_device_inode): Don't assume that a + special device is bogus just because i_blocks is non-zero. + The i_blocks field could get adjusted later, and if this + happens it will confuse the e2fsck_process_bad_inode() in + pass 2. In practice true garbage inodes will have random + non-zero values in i_blocks[4..15], so there's no point + doing the check for an illegal i_blocks value. + 2007-03-28 Theodore Tso <tytso@mit.edu> * pass1.c (e2fsck_pass1, check_ext_attr), |