summaryrefslogtreecommitdiffstats
path: root/fs/ext2
diff options
context:
space:
mode:
authorWang Shilong <wangsl-fnst@cn.fujitsu.com>2013-01-12 01:19:32 -0800
committerJan Kara <jack@suse.cz>2013-01-21 11:19:56 +0100
commit8d8759eb488f9e88fa5f976c4fd7ed205661c872 (patch)
tree3c33cd06749c2c1caba82b3fcab64789c9f3aab0 /fs/ext2
parentf56426ae4d4414c9c996567710dceecbdfc39acc (diff)
downloadlinux-8d8759eb488f9e88fa5f976c4fd7ed205661c872.tar.gz
linux-8d8759eb488f9e88fa5f976c4fd7ed205661c872.tar.bz2
linux-8d8759eb488f9e88fa5f976c4fd7ed205661c872.zip
Ext2: free memory allocated and forget buffer head when io error happens
Add a necessary check when an io error happens. If io error happens,free the memory allocated and forget buffer head. Signed-off-by: Wang Shilong <wangsl-fnst@cn.fujitsu.com> Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/ext2')
-rw-r--r--fs/ext2/inode.c12
1 files changed, 12 insertions, 0 deletions
diff --git a/fs/ext2/inode.c b/fs/ext2/inode.c
index 6363ac66fafa..c3881e56662e 100644
--- a/fs/ext2/inode.c
+++ b/fs/ext2/inode.c
@@ -495,6 +495,10 @@ static int ext2_alloc_branch(struct inode *inode,
* parent to disk.
*/
bh = sb_getblk(inode->i_sb, new_blocks[n-1]);
+ if (unlikely(!bh)) {
+ err = -ENOMEM;
+ goto failed;
+ }
branch[n].bh = bh;
lock_buffer(bh);
memset(bh->b_data, 0, blocksize);
@@ -523,6 +527,14 @@ static int ext2_alloc_branch(struct inode *inode,
}
*blks = num;
return err;
+
+failed:
+ for (i = 1; i < n; i++)
+ bforget(branch[i].bh);
+ for (i = 0; i < indirect_blks; i++)
+ ext2_free_blocks(inode, new_blocks[i], 1);
+ ext2_free_blocks(inode, new_blocks[i], num);
+ return err;
}
/**