Commit bed2cc48 authored by ZhangPeng's avatar ZhangPeng Committed by Richard Weinberger
Browse files

hostfs: Fix the NULL vs IS_ERR() bug for __filemap_get_folio() 



The __filemap_get_folio() function returns error pointers.
It never returns NULL. So use IS_ERR() to check it.

Fixes: 1da86618 ("fs: Convert aops->write_begin to take a folio")
Signed-off-by: default avatarZhangPeng <zhangpeng362@huawei.com>
Acked-by: default avatarJohannes Berg <johannes@sipsolutions.net>
Signed-off-by: default avatarRichard Weinberger <richard@nod.at>
parent 2f681ba4

fs/hostfs/hostfs_kern.c

+2 −2
Original line number Diff line number Diff line
@@ -472,8 +472,8 @@ static int hostfs_write_begin(struct file *file, struct address_space *mapping, 


	*foliop = __filemap_get_folio(mapping, index, FGP_WRITEBEGIN,

			mapping_gfp_mask(mapping));

	if (!*foliop)

		return -ENOMEM;

	if (IS_ERR(*foliop))

		return PTR_ERR(*foliop);

	return 0;

}