diff options
author | Yunlei He <heyunlei@huawei.com> | 2016-12-19 07:10:48 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-12 00:41:51 -0500 |
commit | ec160ad2acaad4a7de6eab7338ef5e3b2b19f907 (patch) | |
tree | 71b9646f2248e486079a613e977fd5d1b17fd30d /fs | |
parent | d78f93384da1e21825144fdfac5dc0bc18db21bd (diff) |
f2fs: fix a problem of using memory after free
commit 7855eba4d6102f811b6dd142d6c749f53b591fa3 upstream.
This patch fix a problem of using memory after free
in function __try_merge_extent_node.
Fixes: 0f825ee6e873 ("f2fs: add new interfaces for extent tree")
Signed-off-by: Yunlei He <heyunlei@huawei.com>
Reviewed-by: Chao Yu <yuchao0@huawei.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/f2fs/extent_cache.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/f2fs/extent_cache.c b/fs/f2fs/extent_cache.c index 2b06d4fcd954..7b32ce979fe1 100644 --- a/fs/f2fs/extent_cache.c +++ b/fs/f2fs/extent_cache.c | |||
@@ -352,11 +352,12 @@ static struct extent_node *__try_merge_extent_node(struct inode *inode, | |||
352 | } | 352 | } |
353 | 353 | ||
354 | if (next_ex && __is_front_mergeable(ei, &next_ex->ei)) { | 354 | if (next_ex && __is_front_mergeable(ei, &next_ex->ei)) { |
355 | if (en) | ||
356 | __release_extent_node(sbi, et, prev_ex); | ||
357 | next_ex->ei.fofs = ei->fofs; | 355 | next_ex->ei.fofs = ei->fofs; |
358 | next_ex->ei.blk = ei->blk; | 356 | next_ex->ei.blk = ei->blk; |
359 | next_ex->ei.len += ei->len; | 357 | next_ex->ei.len += ei->len; |
358 | if (en) | ||
359 | __release_extent_node(sbi, et, prev_ex); | ||
360 | |||
360 | en = next_ex; | 361 | en = next_ex; |
361 | } | 362 | } |
362 | 363 | ||