CVE-2023-54158 (GCVE-0-2023-54158)

Vulnerability from cvelistv5 – Published: 2025-12-24 13:07 – Updated: 2026-01-05 10:34
VLAI?
Title
btrfs: don't free qgroup space unless specified
Summary
In the Linux kernel, the following vulnerability has been resolved: btrfs: don't free qgroup space unless specified Boris noticed in his simple quotas testing that he was getting a leak with Sweet Tea's change to subvol create that stopped doing a transaction commit. This was just a side effect of that change. In the delayed inode code we have an optimization that will free extra reservations if we think we can pack a dir item into an already modified leaf. Previously this wouldn't be triggered in the subvolume create case because we'd commit the transaction, it was still possible but much harder to trigger. It could actually be triggered if we did a mkdir && subvol create with qgroups enabled. This occurs because in btrfs_insert_delayed_dir_index(), which gets called when we're adding the dir item, we do the following: btrfs_block_rsv_release(fs_info, trans->block_rsv, bytes, NULL); if we're able to skip reserving space. The problem here is that trans->block_rsv points at the temporary block rsv for the subvolume create, which has qgroup reservations in the block rsv. This is a problem because btrfs_block_rsv_release() will do the following: if (block_rsv->qgroup_rsv_reserved >= block_rsv->qgroup_rsv_size) { qgroup_to_release = block_rsv->qgroup_rsv_reserved - block_rsv->qgroup_rsv_size; block_rsv->qgroup_rsv_reserved = block_rsv->qgroup_rsv_size; } The temporary block rsv just has ->qgroup_rsv_reserved set, ->qgroup_rsv_size == 0. The optimization in btrfs_insert_delayed_dir_index() sets ->qgroup_rsv_reserved = 0. Then later on when we call btrfs_subvolume_release_metadata() which has btrfs_block_rsv_release(fs_info, rsv, (u64)-1, &qgroup_to_release); btrfs_qgroup_convert_reserved_meta(root, qgroup_to_release); qgroup_to_release is set to 0, and we do not convert the reserved metadata space. The problem here is that the block rsv code has been unconditionally messing with ->qgroup_rsv_reserved, because the main place this is used is delalloc, and any time we call btrfs_block_rsv_release() we do it with qgroup_to_release set, and thus do the proper accounting. The subvolume code is the only other code that uses the qgroup reservation stuff, but it's intermingled with the above optimization, and thus was getting its reservation freed out from underneath it and thus leaking the reserved space. The solution is to simply not mess with the qgroup reservations if we don't have qgroup_to_release set. This works with the existing code as anything that messes with the delalloc reservations always have qgroup_to_release set. This fixes the leak that Boris was observing.
Severity ?
No CVSS data available.
Assigner
References
Impacted products
Vendor Product Version
Linux Linux Affected: ff6bc37eb7f6e7b052e50c13a480e1080b3ec07a , < 1e05bf5e80bb1161b7294c9ce5292b26232ab853 (git)
Affected: ff6bc37eb7f6e7b052e50c13a480e1080b3ec07a , < 148b16cd30b202999ec5b534e3e5d8ab4b766f21 (git)
Affected: ff6bc37eb7f6e7b052e50c13a480e1080b3ec07a , < f264be24146bee2d652010a18ae2517df5856261 (git)
Affected: ff6bc37eb7f6e7b052e50c13a480e1080b3ec07a , < 15e877e5923ec6d6caa5e447dcc4b79a8ff7cc53 (git)
Affected: ff6bc37eb7f6e7b052e50c13a480e1080b3ec07a , < 04ff6bd0317735791ef3e443c7c89f3c0dda548d (git)
Affected: ff6bc37eb7f6e7b052e50c13a480e1080b3ec07a , < 478bd15f46b6e3aae78aac4f3788697f1546eea6 (git)
Affected: ff6bc37eb7f6e7b052e50c13a480e1080b3ec07a , < d246331b78cbef86237f9c22389205bc9b4e1cc1 (git)
Create a notification for this product.
    Linux Linux Affected: 4.17
Unaffected: 0 , < 4.17 (semver)
Unaffected: 5.4.243 , ≤ 5.4.* (semver)
Unaffected: 5.10.180 , ≤ 5.10.* (semver)
Unaffected: 5.15.112 , ≤ 5.15.* (semver)
Unaffected: 6.1.29 , ≤ 6.1.* (semver)
Unaffected: 6.2.16 , ≤ 6.2.* (semver)
Unaffected: 6.3.3 , ≤ 6.3.* (semver)
Unaffected: 6.4 , ≤ * (original_commit_for_fix)
Create a notification for this product.
Show details on NVD website
To clipboard 

{
  "containers": {
    "cna": {
      "affected": [
        {
          "defaultStatus": "unaffected",
          "product": "Linux",
          "programFiles": [
            "fs/btrfs/block-rsv.c"
          ],
          "repo": "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git",
          "vendor": "Linux",
          "versions": [
            {
              "lessThan": "1e05bf5e80bb1161b7294c9ce5292b26232ab853",
              "status": "affected",
              "version": "ff6bc37eb7f6e7b052e50c13a480e1080b3ec07a",
              "versionType": "git"
            },
            {
              "lessThan": "148b16cd30b202999ec5b534e3e5d8ab4b766f21",
              "status": "affected",
              "version": "ff6bc37eb7f6e7b052e50c13a480e1080b3ec07a",
              "versionType": "git"
            },
            {
              "lessThan": "f264be24146bee2d652010a18ae2517df5856261",
              "status": "affected",
              "version": "ff6bc37eb7f6e7b052e50c13a480e1080b3ec07a",
              "versionType": "git"
            },
            {
              "lessThan": "15e877e5923ec6d6caa5e447dcc4b79a8ff7cc53",
              "status": "affected",
              "version": "ff6bc37eb7f6e7b052e50c13a480e1080b3ec07a",
              "versionType": "git"
            },
            {
              "lessThan": "04ff6bd0317735791ef3e443c7c89f3c0dda548d",
              "status": "affected",
              "version": "ff6bc37eb7f6e7b052e50c13a480e1080b3ec07a",
              "versionType": "git"
            },
            {
              "lessThan": "478bd15f46b6e3aae78aac4f3788697f1546eea6",
              "status": "affected",
              "version": "ff6bc37eb7f6e7b052e50c13a480e1080b3ec07a",
              "versionType": "git"
            },
            {
              "lessThan": "d246331b78cbef86237f9c22389205bc9b4e1cc1",
              "status": "affected",
              "version": "ff6bc37eb7f6e7b052e50c13a480e1080b3ec07a",
              "versionType": "git"
            }
          ]
        },
        {
          "defaultStatus": "affected",
          "product": "Linux",
          "programFiles": [
            "fs/btrfs/block-rsv.c"
          ],
          "repo": "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git",
          "vendor": "Linux",
          "versions": [
            {
              "status": "affected",
              "version": "4.17"
            },
            {
              "lessThan": "4.17",
              "status": "unaffected",
              "version": "0",
              "versionType": "semver"
            },
            {
              "lessThanOrEqual": "5.4.*",
              "status": "unaffected",
              "version": "5.4.243",
              "versionType": "semver"
            },
            {
              "lessThanOrEqual": "5.10.*",
              "status": "unaffected",
              "version": "5.10.180",
              "versionType": "semver"
            },
            {
              "lessThanOrEqual": "5.15.*",
              "status": "unaffected",
              "version": "5.15.112",
              "versionType": "semver"
            },
            {
              "lessThanOrEqual": "6.1.*",
              "status": "unaffected",
              "version": "6.1.29",
              "versionType": "semver"
            },
            {
              "lessThanOrEqual": "6.2.*",
              "status": "unaffected",
              "version": "6.2.16",
              "versionType": "semver"
            },
            {
              "lessThanOrEqual": "6.3.*",
              "status": "unaffected",
              "version": "6.3.3",
              "versionType": "semver"
            },
            {
              "lessThanOrEqual": "*",
              "status": "unaffected",
              "version": "6.4",
              "versionType": "original_commit_for_fix"
            }
          ]
        }
      ],
      "cpeApplicability": [
        {
          "nodes": [
            {
              "cpeMatch": [
                {
                  "criteria": "cpe:2.3:o:linux:linux_kernel:*:*:*:*:*:*:*:*",
                  "versionEndExcluding": "5.4.243",
                  "versionStartIncluding": "4.17",
                  "vulnerable": true
                },
                {
                  "criteria": "cpe:2.3:o:linux:linux_kernel:*:*:*:*:*:*:*:*",
                  "versionEndExcluding": "5.10.180",
                  "versionStartIncluding": "4.17",
                  "vulnerable": true
                },
                {
                  "criteria": "cpe:2.3:o:linux:linux_kernel:*:*:*:*:*:*:*:*",
                  "versionEndExcluding": "5.15.112",
                  "versionStartIncluding": "4.17",
                  "vulnerable": true
                },
                {
                  "criteria": "cpe:2.3:o:linux:linux_kernel:*:*:*:*:*:*:*:*",
                  "versionEndExcluding": "6.1.29",
                  "versionStartIncluding": "4.17",
                  "vulnerable": true
                },
                {
                  "criteria": "cpe:2.3:o:linux:linux_kernel:*:*:*:*:*:*:*:*",
                  "versionEndExcluding": "6.2.16",
                  "versionStartIncluding": "4.17",
                  "vulnerable": true
                },
                {
                  "criteria": "cpe:2.3:o:linux:linux_kernel:*:*:*:*:*:*:*:*",
                  "versionEndExcluding": "6.3.3",
                  "versionStartIncluding": "4.17",
                  "vulnerable": true
                },
                {
                  "criteria": "cpe:2.3:o:linux:linux_kernel:*:*:*:*:*:*:*:*",
                  "versionEndExcluding": "6.4",
                  "versionStartIncluding": "4.17",
                  "vulnerable": true
                }
              ],
              "negate": false,
              "operator": "OR"
            }
          ]
        }
      ],
      "descriptions": [
        {
          "lang": "en",
          "value": "In the Linux kernel, the following vulnerability has been resolved:\n\nbtrfs: don\u0027t free qgroup space unless specified\n\nBoris noticed in his simple quotas testing that he was getting a leak\nwith Sweet Tea\u0027s change to subvol create that stopped doing a\ntransaction commit.  This was just a side effect of that change.\n\nIn the delayed inode code we have an optimization that will free extra\nreservations if we think we can pack a dir item into an already modified\nleaf.  Previously this wouldn\u0027t be triggered in the subvolume create\ncase because we\u0027d commit the transaction, it was still possible but\nmuch harder to trigger.  It could actually be triggered if we did a\nmkdir \u0026\u0026 subvol create with qgroups enabled.\n\nThis occurs because in btrfs_insert_delayed_dir_index(), which gets\ncalled when we\u0027re adding the dir item, we do the following:\n\n  btrfs_block_rsv_release(fs_info, trans-\u003eblock_rsv, bytes, NULL);\n\nif we\u0027re able to skip reserving space.\n\nThe problem here is that trans-\u003eblock_rsv points at the temporary block\nrsv for the subvolume create, which has qgroup reservations in the block\nrsv.\n\nThis is a problem because btrfs_block_rsv_release() will do the\nfollowing:\n\n  if (block_rsv-\u003eqgroup_rsv_reserved \u003e= block_rsv-\u003eqgroup_rsv_size) {\n\t  qgroup_to_release = block_rsv-\u003eqgroup_rsv_reserved -\n\t\t  block_rsv-\u003eqgroup_rsv_size;\n\t  block_rsv-\u003eqgroup_rsv_reserved = block_rsv-\u003eqgroup_rsv_size;\n  }\n\nThe temporary block rsv just has -\u003eqgroup_rsv_reserved set,\n-\u003eqgroup_rsv_size == 0.  The optimization in\nbtrfs_insert_delayed_dir_index() sets -\u003eqgroup_rsv_reserved = 0.  Then\nlater on when we call btrfs_subvolume_release_metadata() which has\n\n  btrfs_block_rsv_release(fs_info, rsv, (u64)-1, \u0026qgroup_to_release);\n  btrfs_qgroup_convert_reserved_meta(root, qgroup_to_release);\n\nqgroup_to_release is set to 0, and we do not convert the reserved\nmetadata space.\n\nThe problem here is that the block rsv code has been unconditionally\nmessing with -\u003eqgroup_rsv_reserved, because the main place this is used\nis delalloc, and any time we call btrfs_block_rsv_release() we do it\nwith qgroup_to_release set, and thus do the proper accounting.\n\nThe subvolume code is the only other code that uses the qgroup\nreservation stuff, but it\u0027s intermingled with the above optimization,\nand thus was getting its reservation freed out from underneath it and\nthus leaking the reserved space.\n\nThe solution is to simply not mess with the qgroup reservations if we\ndon\u0027t have qgroup_to_release set.  This works with the existing code as\nanything that messes with the delalloc reservations always have\nqgroup_to_release set.  This fixes the leak that Boris was observing."
        }
      ],
      "providerMetadata": {
        "dateUpdated": "2026-01-05T10:34:05.642Z",
        "orgId": "416baaa9-dc9f-4396-8d5f-8c081fb06d67",
        "shortName": "Linux"
      },
      "references": [
        {
          "url": "https://git.kernel.org/stable/c/1e05bf5e80bb1161b7294c9ce5292b26232ab853"
        },
        {
          "url": "https://git.kernel.org/stable/c/148b16cd30b202999ec5b534e3e5d8ab4b766f21"
        },
        {
          "url": "https://git.kernel.org/stable/c/f264be24146bee2d652010a18ae2517df5856261"
        },
        {
          "url": "https://git.kernel.org/stable/c/15e877e5923ec6d6caa5e447dcc4b79a8ff7cc53"
        },
        {
          "url": "https://git.kernel.org/stable/c/04ff6bd0317735791ef3e443c7c89f3c0dda548d"
        },
        {
          "url": "https://git.kernel.org/stable/c/478bd15f46b6e3aae78aac4f3788697f1546eea6"
        },
        {
          "url": "https://git.kernel.org/stable/c/d246331b78cbef86237f9c22389205bc9b4e1cc1"
        }
      ],
      "title": "btrfs: don\u0027t free qgroup space unless specified",
      "x_generator": {
        "engine": "bippy-1.2.0"
      }
    }
  },
  "cveMetadata": {
    "assignerOrgId": "416baaa9-dc9f-4396-8d5f-8c081fb06d67",
    "assignerShortName": "Linux",
    "cveId": "CVE-2023-54158",
    "datePublished": "2025-12-24T13:07:07.438Z",
    "dateReserved": "2025-12-24T13:02:52.530Z",
    "dateUpdated": "2026-01-05T10:34:05.642Z",
    "state": "PUBLISHED"
  },
  "dataType": "CVE_RECORD",
  "dataVersion": "5.2",
  "vulnerability-lookup:meta": {
    "nvd": "{\"cve\":{\"id\":\"CVE-2023-54158\",\"sourceIdentifier\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\",\"published\":\"2025-12-24T13:16:17.850\",\"lastModified\":\"2025-12-29T15:58:13.147\",\"vulnStatus\":\"Awaiting Analysis\",\"cveTags\":[],\"descriptions\":[{\"lang\":\"en\",\"value\":\"In the Linux kernel, the following vulnerability has been resolved:\\n\\nbtrfs: don\u0027t free qgroup space unless specified\\n\\nBoris noticed in his simple quotas testing that he was getting a leak\\nwith Sweet Tea\u0027s change to subvol create that stopped doing a\\ntransaction commit.  This was just a side effect of that change.\\n\\nIn the delayed inode code we have an optimization that will free extra\\nreservations if we think we can pack a dir item into an already modified\\nleaf.  Previously this wouldn\u0027t be triggered in the subvolume create\\ncase because we\u0027d commit the transaction, it was still possible but\\nmuch harder to trigger.  It could actually be triggered if we did a\\nmkdir \u0026\u0026 subvol create with qgroups enabled.\\n\\nThis occurs because in btrfs_insert_delayed_dir_index(), which gets\\ncalled when we\u0027re adding the dir item, we do the following:\\n\\n  btrfs_block_rsv_release(fs_info, trans-\u003eblock_rsv, bytes, NULL);\\n\\nif we\u0027re able to skip reserving space.\\n\\nThe problem here is that trans-\u003eblock_rsv points at the temporary block\\nrsv for the subvolume create, which has qgroup reservations in the block\\nrsv.\\n\\nThis is a problem because btrfs_block_rsv_release() will do the\\nfollowing:\\n\\n  if (block_rsv-\u003eqgroup_rsv_reserved \u003e= block_rsv-\u003eqgroup_rsv_size) {\\n\\t  qgroup_to_release = block_rsv-\u003eqgroup_rsv_reserved -\\n\\t\\t  block_rsv-\u003eqgroup_rsv_size;\\n\\t  block_rsv-\u003eqgroup_rsv_reserved = block_rsv-\u003eqgroup_rsv_size;\\n  }\\n\\nThe temporary block rsv just has -\u003eqgroup_rsv_reserved set,\\n-\u003eqgroup_rsv_size == 0.  The optimization in\\nbtrfs_insert_delayed_dir_index() sets -\u003eqgroup_rsv_reserved = 0.  Then\\nlater on when we call btrfs_subvolume_release_metadata() which has\\n\\n  btrfs_block_rsv_release(fs_info, rsv, (u64)-1, \u0026qgroup_to_release);\\n  btrfs_qgroup_convert_reserved_meta(root, qgroup_to_release);\\n\\nqgroup_to_release is set to 0, and we do not convert the reserved\\nmetadata space.\\n\\nThe problem here is that the block rsv code has been unconditionally\\nmessing with -\u003eqgroup_rsv_reserved, because the main place this is used\\nis delalloc, and any time we call btrfs_block_rsv_release() we do it\\nwith qgroup_to_release set, and thus do the proper accounting.\\n\\nThe subvolume code is the only other code that uses the qgroup\\nreservation stuff, but it\u0027s intermingled with the above optimization,\\nand thus was getting its reservation freed out from underneath it and\\nthus leaking the reserved space.\\n\\nThe solution is to simply not mess with the qgroup reservations if we\\ndon\u0027t have qgroup_to_release set.  This works with the existing code as\\nanything that messes with the delalloc reservations always have\\nqgroup_to_release set.  This fixes the leak that Boris was observing.\"}],\"metrics\":{},\"references\":[{\"url\":\"https://git.kernel.org/stable/c/04ff6bd0317735791ef3e443c7c89f3c0dda548d\",\"source\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\"},{\"url\":\"https://git.kernel.org/stable/c/148b16cd30b202999ec5b534e3e5d8ab4b766f21\",\"source\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\"},{\"url\":\"https://git.kernel.org/stable/c/15e877e5923ec6d6caa5e447dcc4b79a8ff7cc53\",\"source\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\"},{\"url\":\"https://git.kernel.org/stable/c/1e05bf5e80bb1161b7294c9ce5292b26232ab853\",\"source\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\"},{\"url\":\"https://git.kernel.org/stable/c/478bd15f46b6e3aae78aac4f3788697f1546eea6\",\"source\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\"},{\"url\":\"https://git.kernel.org/stable/c/d246331b78cbef86237f9c22389205bc9b4e1cc1\",\"source\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\"},{\"url\":\"https://git.kernel.org/stable/c/f264be24146bee2d652010a18ae2517df5856261\",\"source\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\"}]}}"
  }
}


Log in or create an account to share your comment.




Tags
Taxonomy of the tags.




Sightings

Author Source Type Date

Nomenclature

  • Seen: The vulnerability was mentioned, discussed, or observed by the user.
  • Confirmed: The vulnerability has been validated from an analyst's perspective.
  • Published Proof of Concept: A public proof of concept is available for this vulnerability.
  • Exploited: The vulnerability was observed as exploited by the user who reported the sighting.
  • Patched: The vulnerability was observed as successfully patched by the user who reported the sighting.
  • Not exploited: The vulnerability was not observed as exploited by the user who reported the sighting.
  • Not confirmed: The user expressed doubt about the validity of the vulnerability.
  • Not patched: The vulnerability was not observed as successfully patched by the user who reported the sighting.



Detection rules are retrieved from Rulezet.