]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
jfs: add check read-only before truncation in jfs_truncate_nolock()
authorVasiliy Kovalev <kovalev@altlinux.org>
Tue, 24 Dec 2024 14:49:14 +0000 (17:49 +0300)
committerDave Kleikamp <dave.kleikamp@oracle.com>
Wed, 19 Feb 2025 21:29:08 +0000 (15:29 -0600)
commitb5799dd77054c1ec49b0088b006c9908e256843b
treeb6267faa45233d9d5c43a16c1ecd7d6660191549
parent0176e69743ecc02961f2ae1ea42439cd2bf9ed58
jfs: add check read-only before truncation in jfs_truncate_nolock()

Added a check for "read-only" mode in the `jfs_truncate_nolock`
function to avoid errors related to writing to a read-only
filesystem.

Call stack:

block_write_begin() {
  jfs_write_failed() {
    jfs_truncate() {
      jfs_truncate_nolock() {
        txEnd() {
          ...
          log = JFS_SBI(tblk->sb)->log;
          // (log == NULL)

If the `isReadOnly(ip)` condition is triggered in
`jfs_truncate_nolock`, the function execution will stop, and no
further data modification will occur. Instead, the `xtTruncate`
function will be called with the "COMMIT_WMAP" flag, preventing
modifications in "read-only" mode.

Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
Reported-by: syzbot+4e89b5368baba8324e07@syzkaller.appspotmail.com
Link: https://syzkaller.appspot.com/bug?extid=4e89b5368baba8324e07
Signed-off-by: Vasiliy Kovalev <kovalev@altlinux.org>
Signed-off-by: Dave Kleikamp <dave.kleikamp@oracle.com>
fs/jfs/inode.c