CVE-2023-52855 usb: dwc2: fix possible NULL pointer dereference caused by driver concurrency

2024-05-2115:31:49

Linux

github.com

linux kernel

usb

null pointer

concurrency

security vulnerability

static analysis

patch

runtime testing

7 High

AI Score

Confidence

Low

0.0004 Low

EPSS

Percentile

13.2%

JSON

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

usb: dwc2: fix possible NULL pointer dereference caused by driver concurrency

In _dwc2_hcd_urb_enqueue(), “urb->hcpriv = NULL” is executed without
holding the lock “hsotg->lock”. In _dwc2_hcd_urb_dequeue():

spin_lock_irqsave(&hsotg-&gt;lock, flags);
...
if (!urb-&gt;hcpriv) {
	dev_dbg(hsotg-&gt;dev, "## urb-&gt;hcpriv is NULL ##\n");
	goto out;
}
rc = dwc2_hcd_urb_dequeue(hsotg, urb-&gt;hcpriv); // Use urb-&gt;hcpriv
...

out:
spin_unlock_irqrestore(&hsotg->lock, flags);

When _dwc2_hcd_urb_enqueue() and _dwc2_hcd_urb_dequeue() are
concurrently executed, the NULL check of “urb->hcpriv” can be executed
before “urb->hcpriv = NULL”. After urb->hcpriv is NULL, it can be used
in the function call to dwc2_hcd_urb_dequeue(), which can cause a NULL
pointer dereference.

This possible bug is found by an experimental static analysis tool
developed by myself. This tool analyzes the locking APIs to extract
function pairs that can be concurrently executed, and then analyzes the
instructions in the paired functions to identify possible concurrency
bugs including data races and atomicity violations. The above possible
bug is reported, when my tool analyzes the source code of Linux 6.5.

To fix this possible bug, “urb->hcpriv = NULL” should be executed with
holding the lock “hsotg->lock”. After using this patch, my tool never
reports the possible bug, with the kernelconfiguration allyesconfig for
x86_64. Because I have no associated hardware, I cannot test the patch
in runtime testing, and just verify it according to the code logic.

CNA Affected

[
  {
    "product": "Linux",
    "vendor": "Linux",
    "defaultStatus": "unaffected",
    "repo": "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git",
    "programFiles": [
      "drivers/usb/dwc2/hcd.c"
    ],
    "versions": [
      {
        "version": "33ad261aa62b",
        "lessThan": "14c9ec34e811",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "33ad261aa62b",
        "lessThan": "fed492aa6493",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "33ad261aa62b",
        "lessThan": "64c47749fc75",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "33ad261aa62b",
        "lessThan": "bdb3dd409630",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "33ad261aa62b",
        "lessThan": "fcaafb574fc8",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "33ad261aa62b",
        "lessThan": "6b21a2272885",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "33ad261aa62b",
        "lessThan": "3e851a77a13c",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "33ad261aa62b",
        "lessThan": "a7bee9598afb",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "33ad261aa62b",
        "lessThan": "ef307bc6ef04",
        "status": "affected",
        "versionType": "git"
      }
    ]
  },
  {
    "product": "Linux",
    "vendor": "Linux",
    "defaultStatus": "affected",
    "repo": "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git",
    "programFiles": [
      "drivers/usb/dwc2/hcd.c"
    ],
    "versions": [
      {
        "version": "4.2",
        "status": "affected"
      },
      {
        "version": "0",
        "lessThan": "4.2",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "4.14.330",
        "lessThanOrEqual": "4.14.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "4.19.299",
        "lessThanOrEqual": "4.19.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "5.4.261",
        "lessThanOrEqual": "5.4.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "5.10.201",
        "lessThanOrEqual": "5.10.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "5.15.139",
        "lessThanOrEqual": "5.15.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "6.1.63",
        "lessThanOrEqual": "6.1.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "6.5.12",
        "lessThanOrEqual": "6.5.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "6.6.2",
        "lessThanOrEqual": "6.6.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "6.7",
        "lessThanOrEqual": "*",
        "status": "unaffected",
        "versionType": "original_commit_for_fix"
      }
    ]
  }
]