CVE-2023-52444 f2fs: fix to avoid dirent corruption

2024-02-2216:21:37

Linux

github.com

linux kernel

dirent corruption

f2fs

inumber

cross-directory rename

AI Score

6.7

Confidence

Low

SSVC

Exploitation

none

Automatable

Technical Impact

partial

JSON

In the Linux kernel, the following vulnerability has been resolved:

f2fs: fix to avoid dirent corruption

As Al reported in link[1]:

f2fs_rename()
…
if (old_dir != new_dir && !whiteout)
f2fs_set_link(old_inode, old_dir_entry,
old_dir_page, new_dir);
else
f2fs_put_page(old_dir_page, 0);

You want correct inumber in the “…” link. And cross-directory
rename does move the source to new parent, even if you’d been asked
to leave a whiteout in the old place.

[1] https://lore.kernel.org/all/20231017055040.GN800259@ZenIV/

With below testcase, it may cause dirent corruption, due to it missed
to call f2fs_set_link() to update “…” link to new directory.

mkdir -p dir/foo
renameat2 -w dir/foo bar

[ASSERT] (__chk_dots_dentries:1421) –> Bad inode number[0x4] for ‘…’, parent parent ino is [0x3]
[FSCK] other corrupted bugs [Fail]

CNA Affected

[
  {
    "repo": "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git",
    "vendor": "Linux",
    "product": "Linux",
    "versions": [
      {
        "status": "affected",
        "version": "7e01e7ad746b",
        "lessThan": "02160112e6d4",
        "versionType": "git"
      },
      {
        "status": "affected",
        "version": "7e01e7ad746b",
        "lessThan": "5624a3c1b1eb",
        "versionType": "git"
      },
      {
        "status": "affected",
        "version": "7e01e7ad746b",
        "lessThan": "6f866885e147",
        "versionType": "git"
      },
      {
        "status": "affected",
        "version": "7e01e7ad746b",
        "lessThan": "f100ba617d8b",
        "versionType": "git"
      },
      {
        "status": "affected",
        "version": "7e01e7ad746b",
        "lessThan": "f0145860c20b",
        "versionType": "git"
      },
      {
        "status": "affected",
        "version": "7e01e7ad746b",
        "lessThan": "d3c0b49aaa12",
        "versionType": "git"
      },
      {
        "status": "affected",
        "version": "7e01e7ad746b",
        "lessThan": "2fb4867f4405",
        "versionType": "git"
      },
      {
        "status": "affected",
        "version": "7e01e7ad746b",
        "lessThan": "53edb549565f",
        "versionType": "git"
      }
    ],
    "programFiles": [
      "fs/f2fs/namei.c"
    ],
    "defaultStatus": "unaffected"
  },
  {
    "repo": "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git",
    "vendor": "Linux",
    "product": "Linux",
    "versions": [
      {
        "status": "affected",
        "version": "4.2"
      },
      {
        "status": "unaffected",
        "version": "0",
        "lessThan": "4.2",
        "versionType": "custom"
      },
      {
        "status": "unaffected",
        "version": "4.19.306",
        "versionType": "custom",
        "lessThanOrEqual": "4.19.*"
      },
      {
        "status": "unaffected",
        "version": "5.4.268",
        "versionType": "custom",
        "lessThanOrEqual": "5.4.*"
      },
      {
        "status": "unaffected",
        "version": "5.10.209",
        "versionType": "custom",
        "lessThanOrEqual": "5.10.*"
      },
      {
        "status": "unaffected",
        "version": "5.15.148",
        "versionType": "custom",
        "lessThanOrEqual": "5.15.*"
      },
      {
        "status": "unaffected",
        "version": "6.1.75",
        "versionType": "custom",
        "lessThanOrEqual": "6.1.*"
      },
      {
        "status": "unaffected",
        "version": "6.6.14",
        "versionType": "custom",
        "lessThanOrEqual": "6.6.*"
      },
      {
        "status": "unaffected",
        "version": "6.7.2",
        "versionType": "custom",
        "lessThanOrEqual": "6.7.*"
      },
      {
        "status": "unaffected",
        "version": "6.8",
        "versionType": "original_commit_for_fix",
        "lessThanOrEqual": "*"
      }
    ],
    "programFiles": [
      "fs/f2fs/namei.c"
    ],
    "defaultStatus": "affected"
  }
]