diff options
| author | Jaegeuk Kim <jaegeuk@kernel.org> | 2014-11-17 16:06:55 -0800 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2014-11-18 17:02:47 -0800 |
| commit | 6d20aff83c24e9a7f15fffe4be36ed33d24465da (patch) | |
| tree | cce1d809eca221a24fdb40e5f9ff5bd4e8370634 /fs/f2fs | |
| parent | 92dffd01790a5219d234fc83c3ba854f4490b7f4 (diff) | |
| download | op-kernel-dev-6d20aff83c24e9a7f15fffe4be36ed33d24465da.zip op-kernel-dev-6d20aff83c24e9a7f15fffe4be36ed33d24465da.tar.gz | |
f2fs: fix to call put_page at the error handling routine
The locked page should be released before returning the function.
Reviewed-by: Chao Yu <chao2.yu@samsung.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'fs/f2fs')
| -rw-r--r-- | fs/f2fs/inline.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/f2fs/inline.c b/fs/f2fs/inline.c index 053d114..f26fb87 100644 --- a/fs/f2fs/inline.c +++ b/fs/f2fs/inline.c @@ -141,8 +141,8 @@ int f2fs_convert_inline_inode(struct inode *inode) ipage = get_node_page(sbi, inode->i_ino); if (IS_ERR(ipage)) { - f2fs_unlock_op(sbi); - return PTR_ERR(ipage); + err = PTR_ERR(ipage); + goto out; } set_new_dnode(&dn, inode, ipage, ipage, 0); @@ -151,7 +151,7 @@ int f2fs_convert_inline_inode(struct inode *inode) err = f2fs_convert_inline_page(&dn, page); f2fs_put_dnode(&dn); - +out: f2fs_unlock_op(sbi); f2fs_put_page(page, 1); |
