Re: [PATCH v4] hpfs: obsolete check=none mount option

[Tetsuo Handa]

On 2026/08/23 0:16, Tetsuo Handa wrote:
> syzbot is reporting use-after-free read problems when a crafted HPFS image
> was mounted with "check=none" option. We should fix code that results in
> use-after-free bugs in the kernel space.
> 
> This patch obsoletes "check=none" option, but this patch does not eliminate
> "check=none" option itself in order not to break existing scripts or
> /etc/fstab configurations that people may have. This patch instead
> eliminates sb_chk != 0 test that becomes redundant. But in order to keep
> this patch minimal, this patch does not fix coding style problems.
> 
> Reported-by: syzbot+fa88eb476e42878f2844@xxxxxxxxxxxxxxxxxxxxxxxxx
> Closes: https://syzkaller.appspot.com/bug?extid=fa88eb476e42878f2844
> Reported-by: syzbot+8debf4b3f7c7391cd8eb@xxxxxxxxxxxxxxxxxxxxxxxxx
> Closes: https://syzkaller.appspot.com/bug?extid=8debf4b3f7c7391cd8eb
> Signed-off-by: Tetsuo Handa <penguin-kernel@xxxxxxxxxxxxxxxxxxx>
> ---
> Resending v4 patch. It seems that sashiko.dev is now running a version
> which includes https://github.com/sashiko-dev/sashiko/issues/320 .
> I guess sashiko.dev can review this time...
> 
>  fs/hpfs/alloc.c |  4 ++--
>  fs/hpfs/anode.c | 13 ++++---------
>  fs/hpfs/dir.c   |  5 ++---
>  fs/hpfs/dnode.c | 19 ++++---------------
>  fs/hpfs/map.c   | 14 +++++++-------
>  fs/hpfs/super.c | 10 +++++-----
>  6 files changed, 24 insertions(+), 41 deletions(-)

sashiko did not find problems with this patch. But sashiko found two pre-existing issues.

https://sashiko.dev/#/patchset/7c77c2f4-a46d-4448-a366-e91e02a56993%40I-love.SAKURA.ne.jp