2022-10-21 09:11:02

by Yue Hu

[permalink] [raw]
Subject: [PATCH v2] erofs: fix general protection fault when reading fragment

From: Yue Hu <[email protected]>

As syzbot reported [1], the fragment feature sb flag is not set, so
packed_inode != NULL needs to be checked in z_erofs_read_fragment().

[1] https://lore.kernel.org/all/[email protected]/

Reported-by: [email protected]
Fixes: 08a0c9ef3e7e ("erofs: support on-disk compressed fragments data")
Signed-off-by: Yue Hu <[email protected]>
---
v2: fix return value to -EFSCURRUPTED (Xiang)

fs/erofs/zdata.c | 3 +++
1 file changed, 3 insertions(+)

diff --git a/fs/erofs/zdata.c b/fs/erofs/zdata.c
index cce56dde135c..55c13cd6934b 100644
--- a/fs/erofs/zdata.c
+++ b/fs/erofs/zdata.c
@@ -659,6 +659,9 @@ static int z_erofs_read_fragment(struct inode *inode, erofs_off_t pos,
u8 *src, *dst;
unsigned int i, cnt;

+ if (!packed_inode)
+ return -EFSCORRUPTED;
+
pos += EROFS_I(inode)->z_fragmentoff;
for (i = 0; i < len; i += cnt) {
cnt = min_t(unsigned int, len - i,
--
2.17.1


2022-11-07 03:07:47

by Gao Xiang

[permalink] [raw]
Subject: Re: [PATCH v2] erofs: fix general protection fault when reading fragment

On Fri, Oct 21, 2022 at 04:53:25PM +0800, Yue Hu wrote:
> From: Yue Hu <[email protected]>
>
> As syzbot reported [1], the fragment feature sb flag is not set, so
> packed_inode != NULL needs to be checked in z_erofs_read_fragment().
>
> [1] https://lore.kernel.org/all/[email protected]/
>
> Reported-by: [email protected]
> Fixes: 08a0c9ef3e7e ("erofs: support on-disk compressed fragments data")
> Signed-off-by: Yue Hu <[email protected]>

Reviewed-by: Gao Xiang <[email protected]>

Thanks,
Gao Xiang

> ---
> v2: fix return value to -EFSCURRUPTED (Xiang)
>
> fs/erofs/zdata.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/fs/erofs/zdata.c b/fs/erofs/zdata.c
> index cce56dde135c..55c13cd6934b 100644
> --- a/fs/erofs/zdata.c
> +++ b/fs/erofs/zdata.c
> @@ -659,6 +659,9 @@ static int z_erofs_read_fragment(struct inode *inode, erofs_off_t pos,
> u8 *src, *dst;
> unsigned int i, cnt;
>
> + if (!packed_inode)
> + return -EFSCORRUPTED;
> +
> pos += EROFS_I(inode)->z_fragmentoff;
> for (i = 0; i < len; i += cnt) {
> cnt = min_t(unsigned int, len - i,
> --
> 2.17.1

2022-11-08 03:47:36

by Chao Yu

[permalink] [raw]
Subject: Re: [PATCH v2] erofs: fix general protection fault when reading fragment

On 2022/10/21 16:53, Yue Hu wrote:
> From: Yue Hu <[email protected]>
>
> As syzbot reported [1], the fragment feature sb flag is not set, so
> packed_inode != NULL needs to be checked in z_erofs_read_fragment().
>
> [1] https://lore.kernel.org/all/[email protected]/
>
> Reported-by: [email protected]
> Fixes: 08a0c9ef3e7e ("erofs: support on-disk compressed fragments data")
> Signed-off-by: Yue Hu <[email protected]>

Reviewed-by: Chao Yu <[email protected]>

Thanks,