Description
In the Linux kernel, the following vulnerability has been resolved: hfsplus: fix uninit-value by validating catalog record size Syzbot reported a KMSAN uninit-value issue in hfsplus_strcasecmp(). The root cause is that hfs_brec_read() doesn't validate that the on-disk record size matches the expected size for the record type being read. When mounting a corrupted filesystem, hfs_brec_read() may read less data than expected. For example, when reading a catalog thread record, the debug output showed: HFSPLUS_BREC_READ: rec_len=520, fd->entrylength=26 HFSPLUS_BREC_READ: WARNING - entrylength (26) < rec_len (520) - PARTIAL READ! hfs_brec_read() only validates that entrylength is not greater than the buffer size, but doesn't check if it's less than expected. It successfully reads 26 bytes into a 520-byte structure and returns success, leaving 494 bytes uninitialized. This uninitialized data in tmp.thread.nodeName then gets copied by hfsplus_cat_build_key_uni() and used by hfsplus_strcasecmp(), triggering the KMSAN warning when the uninitialized bytes are used as array indices in case_fold(). Fix by introducing hfsplus_brec_read_cat() wrapper that: 1. Calls hfs_brec_read() to read the data 2. Validates the record size based on the type field: - Fixed size for folder and file records - Variable size for thread records (depends on string length) 3. Returns -EIO if size doesn't match expected For thread records, check against HFSPLUS_MIN_THREAD_SZ before reading nodeName.length to avoid reading uninitialized data at call sites that don't zero-initialize the entry structure. Also initialize the tmp variable in hfsplus_find_cat() as defensive programming to ensure no uninitialized data even if validation is bypassed.
Product status
1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 (git) before 61a790974ff7e533acbceca06c7d02f22bf96d4d
1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 (git) before c91bbd6193c70a02c50c22e0fb1f60c3c5bd053a
1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 (git) before a420904450962a562ad053a41a53a27755021b48
1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 (git) before 93e8d613f1a01b6637f387cc93f184cf7fb881d6
1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 (git) before b6b592275aeff184aa82fcf6abccd833fb71b393
2.6.12
Any version before 2.6.12
6.6.140 (semver)
6.12.88 (semver)
6.18.30 (semver)
7.0.7 (semver)
7.1-rc1 (original_commit_for_fix)
References
git.kernel.org/...c/61a790974ff7e533acbceca06c7d02f22bf96d4d
git.kernel.org/...c/c91bbd6193c70a02c50c22e0fb1f60c3c5bd053a
git.kernel.org/...c/a420904450962a562ad053a41a53a27755021b48
git.kernel.org/...c/93e8d613f1a01b6637f387cc93f184cf7fb881d6
git.kernel.org/...c/b6b592275aeff184aa82fcf6abccd833fb71b393