diff options
author | Namjae Jeon <namjae.jeon@samsung.com> | 2012-10-09 11:08:56 -0400 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2012-12-13 10:33:23 -0500 |
commit | 2fb7d99d0de3fd8ae869f35ab682581d8455887a (patch) | |
tree | 14bf40139379dd770dff2a89984a5d0bdefa5df1 /fs/udf | |
parent | 25389bb207987b5774182f763b9fb65ff08761c8 (diff) |
udf: fix memory leak while allocating blocks during write
Need to brelse the buffer_head stored in cur_epos and next_epos.
Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: Ashish Sangwan <a.sangwan@samsung.com>
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/udf')
-rw-r--r-- | fs/udf/inode.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/udf/inode.c b/fs/udf/inode.c index df88b957ccf0..2b7759371ff6 100644 --- a/fs/udf/inode.c +++ b/fs/udf/inode.c | |||
@@ -765,6 +765,8 @@ static sector_t inode_getblk(struct inode *inode, sector_t block, | |||
765 | goal, err); | 765 | goal, err); |
766 | if (!newblocknum) { | 766 | if (!newblocknum) { |
767 | brelse(prev_epos.bh); | 767 | brelse(prev_epos.bh); |
768 | brelse(cur_epos.bh); | ||
769 | brelse(next_epos.bh); | ||
768 | *err = -ENOSPC; | 770 | *err = -ENOSPC; |
769 | return 0; | 771 | return 0; |
770 | } | 772 | } |
@@ -795,6 +797,8 @@ static sector_t inode_getblk(struct inode *inode, sector_t block, | |||
795 | udf_update_extents(inode, laarr, startnum, endnum, &prev_epos); | 797 | udf_update_extents(inode, laarr, startnum, endnum, &prev_epos); |
796 | 798 | ||
797 | brelse(prev_epos.bh); | 799 | brelse(prev_epos.bh); |
800 | brelse(cur_epos.bh); | ||
801 | brelse(next_epos.bh); | ||
798 | 802 | ||
799 | newblock = udf_get_pblock(inode->i_sb, newblocknum, | 803 | newblock = udf_get_pblock(inode->i_sb, newblocknum, |
800 | iinfo->i_location.partitionReferenceNum, 0); | 804 | iinfo->i_location.partitionReferenceNum, 0); |