summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJason Hrycay <jhrycay@gmail.com>2013-05-31 12:45:11 -0500
committerJaegeuk Kim <jaegeuk.kim@samsung.com>2013-06-03 19:49:09 +0900
commit1e03e38b35b8e72d65fd5d931627bd6ff02926c1 (patch)
treee8da576270efe341f63e1e5bd2cc033631dc84cf
parent83d5d6f66b375f21bee4c2e17178f7c073a66301 (diff)
downloadlinux-1e03e38b35b8e72d65fd5d931627bd6ff02926c1.tar.gz
linux-1e03e38b35b8e72d65fd5d931627bd6ff02926c1.tar.bz2
linux-1e03e38b35b8e72d65fd5d931627bd6ff02926c1.zip
f2fs: handle errors from get_node_page calls
Add check for error pointers returned from get_node_page in order to avoid dereferencing a bad address on the next use. Signed-off-by: Jason Hrycay <jason.hrycay@motorola.com> Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
-rw-r--r--fs/f2fs/xattr.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/f2fs/xattr.c b/fs/f2fs/xattr.c
index 0b02dce31356..ae61f359554f 100644
--- a/fs/f2fs/xattr.c
+++ b/fs/f2fs/xattr.c
@@ -218,6 +218,8 @@ int f2fs_getxattr(struct inode *inode, int name_index, const char *name,
return -ENODATA;
page = get_node_page(sbi, fi->i_xattr_nid);
+ if (IS_ERR(page))
+ return PTR_ERR(page);
base_addr = page_address(page);
list_for_each_xattr(entry, base_addr) {
@@ -268,6 +270,8 @@ ssize_t f2fs_listxattr(struct dentry *dentry, char *buffer, size_t buffer_size)
return 0;
page = get_node_page(sbi, fi->i_xattr_nid);
+ if (IS_ERR(page))
+ return PTR_ERR(page);
base_addr = page_address(page);
list_for_each_xattr(entry, base_addr) {