CVE-2023-53166 (GCVE-0-2023-53166)
Vulnerability from cvelistv5
Published
2025-09-15 14:03
Modified
2025-09-15 14:03
Severity ?
Summary
In the Linux kernel, the following vulnerability has been resolved: power: supply: bq25890: Fix external_power_changed race bq25890_charger_external_power_changed() dereferences bq->charger, which gets sets in bq25890_power_supply_init() like this: bq->charger = devm_power_supply_register(bq->dev, &bq->desc, &psy_cfg); As soon as devm_power_supply_register() has called device_add() the external_power_changed callback can get called. So there is a window where bq25890_charger_external_power_changed() may get called while bq->charger has not been set yet leading to a NULL pointer dereference. This race hits during boot sometimes on a Lenovo Yoga Book 1 yb1-x90f when the cht_wcove_pwrsrc (extcon) power_supply is done with detecting the connected charger-type which happens to exactly hit the small window: BUG: kernel NULL pointer dereference, address: 0000000000000018 <snip> RIP: 0010:__power_supply_is_supplied_by+0xb/0xb0 <snip> Call Trace: <TASK> __power_supply_get_supplier_property+0x19/0x50 class_for_each_device+0xb1/0xe0 power_supply_get_property_from_supplier+0x2e/0x50 bq25890_charger_external_power_changed+0x38/0x1b0 [bq25890_charger] __power_supply_changed_work+0x30/0x40 class_for_each_device+0xb1/0xe0 power_supply_changed_work+0x5f/0xe0 <snip> Fixing this is easy. The external_power_changed callback gets passed the power_supply which will eventually get stored in bq->charger, so bq25890_charger_external_power_changed() can simply directly use the passed in psy argument which is always valid.
References
416baaa9-dc9f-4396-8d5f-8c081fb06d67https://git.kernel.org/stable/c/029a443b9b6424170f00f6dd5b7682e682cce92e
416baaa9-dc9f-4396-8d5f-8c081fb06d67https://git.kernel.org/stable/c/72c28207c19c2c46fab8ae994aff25e197fb2949
416baaa9-dc9f-4396-8d5f-8c081fb06d67https://git.kernel.org/stable/c/9d20fa1982c35697f3f8c4ae0f12791691ae5958
Impacted products
Vendor Product Version
Linux Linux Version: eab25b4f93aa771728127705eb4b235a3b5aad94
Version: eab25b4f93aa771728127705eb4b235a3b5aad94
Version: eab25b4f93aa771728127705eb4b235a3b5aad94
 Create a notification for this product.
   Linux Linux Version: 5.18
 Create a notification for this product.
Show details on NVD website

To clipboard 

{
  "containers": {
    "cna": {
      "affected": [
        {
          "defaultStatus": "unaffected",
          "product": "Linux",
          "programFiles": [
            "drivers/power/supply/bq25890_charger.c"
          ],
          "repo": "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git",
          "vendor": "Linux",
          "versions": [
            {
              "lessThan": "72c28207c19c2c46fab8ae994aff25e197fb2949",
              "status": "affected",
              "version": "eab25b4f93aa771728127705eb4b235a3b5aad94",
              "versionType": "git"
            },
            {
              "lessThan": "9d20fa1982c35697f3f8c4ae0f12791691ae5958",
              "status": "affected",
              "version": "eab25b4f93aa771728127705eb4b235a3b5aad94",
              "versionType": "git"
            },
            {
              "lessThan": "029a443b9b6424170f00f6dd5b7682e682cce92e",
              "status": "affected",
              "version": "eab25b4f93aa771728127705eb4b235a3b5aad94",
              "versionType": "git"
            }
          ]
        },
        {
          "defaultStatus": "affected",
          "product": "Linux",
          "programFiles": [
            "drivers/power/supply/bq25890_charger.c"
          ],
          "repo": "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git",
          "vendor": "Linux",
          "versions": [
            {
              "status": "affected",
              "version": "5.18"
            },
            {
              "lessThan": "5.18",
              "status": "unaffected",
              "version": "0",
              "versionType": "semver"
            },
            {
              "lessThanOrEqual": "6.1.*",
              "status": "unaffected",
              "version": "6.1.31",
              "versionType": "semver"
            },
            {
              "lessThanOrEqual": "6.3.*",
              "status": "unaffected",
              "version": "6.3.5",
              "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": "6.1.31",
                  "versionStartIncluding": "5.18",
                  "vulnerable": true
                },
                {
                  "criteria": "cpe:2.3:o:linux:linux_kernel:*:*:*:*:*:*:*:*",
                  "versionEndExcluding": "6.3.5",
                  "versionStartIncluding": "5.18",
                  "vulnerable": true
                },
                {
                  "criteria": "cpe:2.3:o:linux:linux_kernel:*:*:*:*:*:*:*:*",
                  "versionEndExcluding": "6.4",
                  "versionStartIncluding": "5.18",
                  "vulnerable": true
                }
              ],
              "negate": false,
              "operator": "OR"
            }
          ]
        }
      ],
      "descriptions": [
        {
          "lang": "en",
          "value": "In the Linux kernel, the following vulnerability has been resolved:\n\npower: supply: bq25890: Fix external_power_changed race\n\nbq25890_charger_external_power_changed() dereferences bq-\u003echarger,\nwhich gets sets in bq25890_power_supply_init() like this:\n\n  bq-\u003echarger = devm_power_supply_register(bq-\u003edev, \u0026bq-\u003edesc, \u0026psy_cfg);\n\nAs soon as devm_power_supply_register() has called device_add()\nthe external_power_changed callback can get called. So there is a window\nwhere bq25890_charger_external_power_changed() may get called while\nbq-\u003echarger has not been set yet leading to a NULL pointer dereference.\n\nThis race hits during boot sometimes on a Lenovo Yoga Book 1 yb1-x90f\nwhen the cht_wcove_pwrsrc (extcon) power_supply is done with detecting\nthe connected charger-type which happens to exactly hit the small window:\n\n  BUG: kernel NULL pointer dereference, address: 0000000000000018\n  \u003csnip\u003e\n  RIP: 0010:__power_supply_is_supplied_by+0xb/0xb0\n  \u003csnip\u003e\n  Call Trace:\n   \u003cTASK\u003e\n   __power_supply_get_supplier_property+0x19/0x50\n   class_for_each_device+0xb1/0xe0\n   power_supply_get_property_from_supplier+0x2e/0x50\n   bq25890_charger_external_power_changed+0x38/0x1b0 [bq25890_charger]\n   __power_supply_changed_work+0x30/0x40\n   class_for_each_device+0xb1/0xe0\n   power_supply_changed_work+0x5f/0xe0\n  \u003csnip\u003e\n\nFixing this is easy. The external_power_changed callback gets passed\nthe power_supply which will eventually get stored in bq-\u003echarger,\nso bq25890_charger_external_power_changed() can simply directly use\nthe passed in psy argument which is always valid."
        }
      ],
      "providerMetadata": {
        "dateUpdated": "2025-09-15T14:03:55.131Z",
        "orgId": "416baaa9-dc9f-4396-8d5f-8c081fb06d67",
        "shortName": "Linux"
      },
      "references": [
        {
          "url": "https://git.kernel.org/stable/c/72c28207c19c2c46fab8ae994aff25e197fb2949"
        },
        {
          "url": "https://git.kernel.org/stable/c/9d20fa1982c35697f3f8c4ae0f12791691ae5958"
        },
        {
          "url": "https://git.kernel.org/stable/c/029a443b9b6424170f00f6dd5b7682e682cce92e"
        }
      ],
      "title": "power: supply: bq25890: Fix external_power_changed race",
      "x_generator": {
        "engine": "bippy-1.2.0"
      }
    }
  },
  "cveMetadata": {
    "assignerOrgId": "416baaa9-dc9f-4396-8d5f-8c081fb06d67",
    "assignerShortName": "Linux",
    "cveId": "CVE-2023-53166",
    "datePublished": "2025-09-15T14:03:55.131Z",
    "dateReserved": "2025-09-15T13:59:19.063Z",
    "dateUpdated": "2025-09-15T14:03:55.131Z",
    "state": "PUBLISHED"
  },
  "dataType": "CVE_RECORD",
  "dataVersion": "5.1",
  "vulnerability-lookup:meta": {
    "nvd": "{\"cve\":{\"id\":\"CVE-2023-53166\",\"sourceIdentifier\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\",\"published\":\"2025-09-15T14:15:38.343\",\"lastModified\":\"2025-09-15T15:22:27.090\",\"vulnStatus\":\"Awaiting Analysis\",\"cveTags\":[],\"descriptions\":[{\"lang\":\"en\",\"value\":\"In the Linux kernel, the following vulnerability has been resolved:\\n\\npower: supply: bq25890: Fix external_power_changed race\\n\\nbq25890_charger_external_power_changed() dereferences bq-\u003echarger,\\nwhich gets sets in bq25890_power_supply_init() like this:\\n\\n  bq-\u003echarger = devm_power_supply_register(bq-\u003edev, \u0026bq-\u003edesc, \u0026psy_cfg);\\n\\nAs soon as devm_power_supply_register() has called device_add()\\nthe external_power_changed callback can get called. So there is a window\\nwhere bq25890_charger_external_power_changed() may get called while\\nbq-\u003echarger has not been set yet leading to a NULL pointer dereference.\\n\\nThis race hits during boot sometimes on a Lenovo Yoga Book 1 yb1-x90f\\nwhen the cht_wcove_pwrsrc (extcon) power_supply is done with detecting\\nthe connected charger-type which happens to exactly hit the small window:\\n\\n  BUG: kernel NULL pointer dereference, address: 0000000000000018\\n  \u003csnip\u003e\\n  RIP: 0010:__power_supply_is_supplied_by+0xb/0xb0\\n  \u003csnip\u003e\\n  Call Trace:\\n   \u003cTASK\u003e\\n   __power_supply_get_supplier_property+0x19/0x50\\n   class_for_each_device+0xb1/0xe0\\n   power_supply_get_property_from_supplier+0x2e/0x50\\n   bq25890_charger_external_power_changed+0x38/0x1b0 [bq25890_charger]\\n   __power_supply_changed_work+0x30/0x40\\n   class_for_each_device+0xb1/0xe0\\n   power_supply_changed_work+0x5f/0xe0\\n  \u003csnip\u003e\\n\\nFixing this is easy. The external_power_changed callback gets passed\\nthe power_supply which will eventually get stored in bq-\u003echarger,\\nso bq25890_charger_external_power_changed() can simply directly use\\nthe passed in psy argument which is always valid.\"}],\"metrics\":{},\"references\":[{\"url\":\"https://git.kernel.org/stable/c/029a443b9b6424170f00f6dd5b7682e682cce92e\",\"source\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\"},{\"url\":\"https://git.kernel.org/stable/c/72c28207c19c2c46fab8ae994aff25e197fb2949\",\"source\":\"416baaa9-dc9f-4396-8d5f-8c081fb06d67\"},{\"url\":\"https://git.kernel.org/stable/c/9d20fa1982c35697f3f8c4ae0f12791691ae5958\",\"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 seen somewhere by the user.
  • Confirmed: The vulnerability is confirmed from an analyst perspective.
  • Exploited: This vulnerability was exploited and seen by the user reporting the sighting.
  • Patched: This vulnerability was successfully patched by the user reporting the sighting.
  • Not exploited: This vulnerability was not exploited or seen by the user reporting the sighting.
  • Not confirmed: The user expresses doubt about the veracity of the vulnerability.
  • Not patched: This vulnerability was not successfully patched by the user reporting the sighting.