Re: [PATCH] hfsplus: fix uninit-value in copy_name

From: Kees Cook
Date: Fri Jul 05 2024 - 17:21:21 EST


On Tue, 21 May 2024 13:21:46 +0800, Edward Adam Davis wrote:
> [syzbot reported]
> BUG: KMSAN: uninit-value in sized_strscpy+0xc4/0x160
> sized_strscpy+0xc4/0x160
> copy_name+0x2af/0x320 fs/hfsplus/xattr.c:411
> hfsplus_listxattr+0x11e9/0x1a50 fs/hfsplus/xattr.c:750
> vfs_listxattr fs/xattr.c:493 [inline]
> listxattr+0x1f3/0x6b0 fs/xattr.c:840
> path_listxattr fs/xattr.c:864 [inline]
> __do_sys_listxattr fs/xattr.c:876 [inline]
> __se_sys_listxattr fs/xattr.c:873 [inline]
> __x64_sys_listxattr+0x16b/0x2f0 fs/xattr.c:873
> x64_sys_call+0x2ba0/0x3b50 arch/x86/include/generated/asm/syscalls_64.h:195
> do_syscall_x64 arch/x86/entry/common.c:52 [inline]
> do_syscall_64+0xcf/0x1e0 arch/x86/entry/common.c:83
> entry_SYSCALL_64_after_hwframe+0x77/0x7f
>
> [...]

I've taken some security-related hfsplus stuff before, so:

Applied to for-next/hardening, thanks!

[1/1] hfsplus: fix uninit-value in copy_name
https://git.kernel.org/kees/c/974a00974829

Take care,

--
Kees Cook