summaryrefslogtreecommitdiffstats
path: root/fs/ext2/ialloc.c
diff options
context:
space:
mode:
authorJan Kara <jack@suse.cz>2011-12-09 00:08:58 +0100
committerJan Kara <jack@suse.cz>2012-01-09 13:52:07 +0100
commitef6919c283257155def420bd247140e9fd2e9843 (patch)
tree49c929bcb5b8be103a4ea8fe9964d0e02a0ec740 /fs/ext2/ialloc.c
parent1415dd8705394399d59a3df1ab48d149e1e41e77 (diff)
downloadop-kernel-dev-ef6919c283257155def420bd247140e9fd2e9843.zip
op-kernel-dev-ef6919c283257155def420bd247140e9fd2e9843.tar.gz
ext2: Fix error handling on inode bitmap corruption
When insert_inode_locked() fails in ext2_new_inode() it most likely means inode bitmap got corrupted and we allocated again inode which is already in use. Also doing unlock_new_inode() during error recovery is wrong since the inode does not have I_NEW set. Fix the problem by informing about filesystem error and jumping to fail: (instead of fail_drop:) which doesn't call unlock_new_inode(). Reviewed-by: Eric Sandeen <sandeen@redhat.com> Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/ext2/ialloc.c')
-rw-r--r--fs/ext2/ialloc.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/fs/ext2/ialloc.c b/fs/ext2/ialloc.c
index c4e81df..78502c1 100644
--- a/fs/ext2/ialloc.c
+++ b/fs/ext2/ialloc.c
@@ -573,8 +573,11 @@ got:
inode->i_generation = sbi->s_next_generation++;
spin_unlock(&sbi->s_next_gen_lock);
if (insert_inode_locked(inode) < 0) {
- err = -EINVAL;
- goto fail_drop;
+ ext2_error(sb, "ext2_new_inode",
+ "inode number already in use - inode=%lu",
+ (unsigned long) ino);
+ err = -EIO;
+ goto fail;
}
dquot_initialize(inode);
OpenPOWER on IntegriCloud