summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>2012-09-21 06:55:02 +0200
committerRichard Weinberger <richard@nod.at>2012-10-09 22:34:52 +0200
commit91312c53aff4c9863dca6080182091dc5c5824c6 (patch)
treeb1bbd8cec2dca0540584c74b5f7f866b918a03dc
parenthostfs: drop vmtruncate (diff)
downloadlinux-91312c53aff4c9863dca6080182091dc5c5824c6.tar.xz
linux-91312c53aff4c9863dca6080182091dc5c5824c6.zip
hppfs: fix the return value of get_inode()
In case of error, the function get_inode() returns ERR_PTR(). But the users hppfs_lookup() and hppfs_fill_super() use NULL test for check the return value, not IS_ERR(), so we'd better change the return value of get_inode() to NULL instead of ERR_PTR(). dpatch engine is used to generated this patch. (https://github.com/weiyj/dpatch) Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Signed-off-by: Richard Weinberger <richard@nod.at>
-rw-r--r--fs/hppfs/hppfs.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/hppfs/hppfs.c b/fs/hppfs/hppfs.c
index a68cb2092512..78f21f8dc2ec 100644
--- a/fs/hppfs/hppfs.c
+++ b/fs/hppfs/hppfs.c
@@ -674,7 +674,7 @@ static struct inode *get_inode(struct super_block *sb, struct dentry *dentry)
if (!inode) {
dput(dentry);
- return ERR_PTR(-ENOMEM);
+ return NULL;
}
if (S_ISDIR(dentry->d_inode->i_mode)) {