usb: gadget: goku_udc: avoid NULL deref of dev->driver in INT_USBRESET log
- Published
- Last modified
- Where the record stands: published, rejected by its publisher, or superseded by another id.What do these mean? →
- published
- Whether CISA lists it in its Known Exploited Vulnerabilities catalog: proof that attackers are using it.What do these mean? →
- Not listed in CISA KEV
- Fix
- Fixed in 5.10.271, 5.15.222, 6.1.189, 6.6.158, 6.12.111, and 1 more
- Whether working attack code is public (Exploit-DB, Metasploit, Nuclei or GitHub). Public code makes attacks easier.What do these mean? →
- None on Exploit-DB, Metasploit or GitHub
Summary
In the Linux kernel, the following vulnerability has been resolved: usb: gadget: goku_udc: avoid NULL deref of dev->driver in INT_USBRESET log goku_irq() handles a number of bus events under a single ep0 path. It already guards the gadget driver suspend/resume callbacks against a NULL ->driver: if (dev->gadget.speed != USB_SPEED_UNKNOWN && dev->driver && dev->driver->resume) { spin_unlock(&dev->lock); dev->driver->resume(&dev->gadget); ... } but the very next branch unconditionally dereferences dev->driver when an INT_USBRESET arrives: if (stat & INT_USBRESET) { ACK(INT_USBRESET); INFO(dev, "USB reset done, gadget %s\n", dev->driver->driver.name); } If the controller raises INT_USBRESET before any gadget driver has been bound (or after one has been unbound), dev->driver is NULL and the printk dereferences NULL. smatch flags the inconsistency: drivers/usb/gadget/udc/goku_udc.c:1618 goku_irq() error: we previously assumed 'dev->driver' could be null (see line 1607) Fall back to a placeholder when the gadget driver is not bound. No functional change while a gadget driver is bound.
Timeline
- CVE published
- Record last updated
Affected versions
- Linux Linux
From 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 · fixed in d7b33afbf9b54b61cb423af9663df54eae11d3b1; From 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 · fixed in ea2901a4a90afc662e4d023542d91f10ab10fd3c; From 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 · fixed in 999ff3bf665f2abfb6af519ef3d6d42b061d3c15; From 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 · fixed in a018d4195114ca2cf79858cc45c98581dee4795b; From 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 · fixed in 10d7fc3008d5b7ec03af8e08927f1d090014c2af; From 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 · fixed in 28d78783a871cdb7a637dcb72452ff7d303430ae; From 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 · fixed in 5bf5e3fba9bc7dfd69701521dbe9809f8ccbdb02
- Linux Linux
2.6.12
- KernelLinux
From 2.6.12 · fixed in 5.10.271; From 5.11.0 · fixed in 5.15.222; From 5.16.0 · fixed in 6.1.189; From 6.2.0 · fixed in 6.6.158; From 6.7.0 · fixed in 6.12.111; From 6.13.0 · fixed in 6.18.53
References 7
git.kernel.org
- stable/c/d7b33afbf9b54b61cb423af9663df54eae11d3b1
- stable/c/ea2901a4a90afc662e4d023542d91f10ab10fd3c
- stable/c/999ff3bf665f2abfb6af519ef3d6d42b061d3c15
- stable/c/a018d4195114ca2cf79858cc45c98581dee4795b
- stable/c/10d7fc3008d5b7ec03af8e08927f1d090014c2af
- stable/c/28d78783a871cdb7a637dcb72452ff7d303430ae
- stable/c/5bf5e3fba9bc7dfd69701521dbe9809f8ccbdb02