CVE-2024-38633 serial: max3100: Update uart_driver_registered on driver removal

2024-06-2110:18:22

Linux

www.cve.org

vulnerability resolved

linux kernel

max3100 driver removal

kernel oops

serial_core_register_port

uart_driver_registered

error path

uart_register_driver

0.0004 Low

EPSS

Percentile

13.1%

JSON

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

serial: max3100: Update uart_driver_registered on driver removal

The removal of the last MAX3100 device triggers the removal of
the driver. However, code doesn’t update the respective global
variable and after insmod — rmmod — insmod cycle the kernel
oopses:

max3100 spi-PRP0001:01: max3100_probe: adding port 0
BUG: kernel NULL pointer dereference, address: 0000000000000408
…
RIP: 0010:serial_core_register_port+0xa0/0x840
…
max3100_probe+0x1b6/0x280 [max3100]
spi_probe+0x8d/0xb0

Update the actual state so next time UART driver will be registered
again.

Hugo also noticed, that the error path in the probe also affected
by having the variable set, and not cleared. Instead of clearing it
move the assignment after the successfull uart_register_driver() call.

CNA Affected

[
  {
    "product": "Linux",
    "vendor": "Linux",
    "defaultStatus": "unaffected",
    "repo": "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git",
    "programFiles": [
      "drivers/tty/serial/max3100.c"
    ],
    "versions": [
      {
        "version": "7831d56b0a35",
        "lessThan": "21a61a7fbcfd",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "7831d56b0a35",
        "lessThan": "9db4222ed8cd",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "7831d56b0a35",
        "lessThan": "e8e2a4339dec",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "7831d56b0a35",
        "lessThan": "361a92c9038e",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "7831d56b0a35",
        "lessThan": "b6eb7aff23e0",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "7831d56b0a35",
        "lessThan": "e8a10089eddb",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "7831d56b0a35",
        "lessThan": "fa84ca78b048",
        "status": "affected",
        "versionType": "git"
      },
      {
        "version": "7831d56b0a35",
        "lessThan": "712a1fcb38dc",
        "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/tty/serial/max3100.c"
    ],
    "versions": [
      {
        "version": "2.6.30",
        "status": "affected"
      },
      {
        "version": "0",
        "lessThan": "2.6.30",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "4.19.316",
        "lessThanOrEqual": "4.19.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "5.4.278",
        "lessThanOrEqual": "5.4.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "5.10.219",
        "lessThanOrEqual": "5.10.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "5.15.161",
        "lessThanOrEqual": "5.15.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "6.1.93",
        "lessThanOrEqual": "6.1.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "6.6.33",
        "lessThanOrEqual": "6.6.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "6.9.4",
        "lessThanOrEqual": "6.9.*",
        "status": "unaffected",
        "versionType": "custom"
      },
      {
        "version": "6.10-rc1",
        "lessThanOrEqual": "*",
        "status": "unaffected",
        "versionType": "original_commit_for_fix"
      }
    ]
  }
]