summaryrefslogtreecommitdiffstats
path: root/fs/f2fs
diff options
context:
space:
mode:
authorNamjae Jeon <namjae.jeon@samsung.com>2012-12-22 12:09:43 +0900
committerJaegeuk Kim <jaegeuk.kim@samsung.com>2012-12-28 11:26:13 +0900
commit06025f4df88e9e41f4ebcf6b4c3df30661332bc9 (patch)
tree9d5fab4bb021444db75538a70095675af227dded /fs/f2fs
parent029cd28c1f739bbfc5105035696d5f1f4e45d161 (diff)
downloadop-kernel-dev-06025f4df88e9e41f4ebcf6b4c3df30661332bc9.zip
op-kernel-dev-06025f4df88e9e41f4ebcf6b4c3df30661332bc9.tar.gz
f2fs: handle error from f2fs_iget_nowait
In case f2fs_iget_nowait returns error, it results in truncate_hole being called with 'error' value as inode pointer. There is no check in truncate_hole for valid inode, so it could result in crash due "invalid access to memory". Avoid this by handling error condition properly. Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com> Signed-off-by: Amit Sahrawat <a.sahrawat@samsung.com> Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
Diffstat (limited to 'fs/f2fs')
-rw-r--r--fs/f2fs/recovery.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/fs/f2fs/recovery.c b/fs/f2fs/recovery.c
index b07e9b6..207e2c8 100644
--- a/fs/f2fs/recovery.c
+++ b/fs/f2fs/recovery.c
@@ -228,6 +228,9 @@ static void check_index_in_prev_nodes(struct f2fs_sb_info *sbi,
/* Deallocate previous index in the node page */
inode = f2fs_iget_nowait(sbi->sb, ino);
+ if (IS_ERR(inode))
+ return;
+
truncate_hole(inode, bidx, bidx + 1);
iput(inode);
}
OpenPOWER on IntegriCloud