CVE-2024-35827 io_uring/net: fix overflow check in io_recvmsg_mshot_prep()

2024-05-1713:41:09

Linux

github.com

linux kernel

vulnerability

io_uring/net

fix

overflow check

integer underflow

check_add_overflow

size_t

unsigned long

AI Score

6.9

Confidence

High

SSVC

Exploitation

none

Automatable

Technical Impact

partial

JSON

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

io_uring/net: fix overflow check in io_recvmsg_mshot_prep()

The “controllen” variable is type size_t (unsigned long). Casting it
to int could lead to an integer underflow.

The check_add_overflow() function considers the type of the destination
which is type int. If we add two positive values and the result cannot
fit in an integer then that’s counted as an overflow.

However, if we cast “controllen” to an int and it turns negative, then
negative values can fit into an int type so there is no overflow.

Good: 100 + (unsigned long)-4 = 96 <– overflow
Bad: 100 + (int)-4 = 96 <– no overflow

I deleted the cast of the sizeof() as well. That’s not a bug but the
cast is unnecessary.

CNA Affected

[
  {
    "repo": "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git",
    "vendor": "Linux",
    "product": "Linux",
    "versions": [
      {
        "status": "affected",
        "version": "9b0fc3c054ff",
        "lessThan": "868ec8686164",
        "versionType": "git"
      },
      {
        "status": "affected",
        "version": "9b0fc3c054ff",
        "lessThan": "59a534690ecc",
        "versionType": "git"
      },
      {
        "status": "affected",
        "version": "9b0fc3c054ff",
        "lessThan": "0c8c74bb59e7",
        "versionType": "git"
      },
      {
        "status": "affected",
        "version": "9b0fc3c054ff",
        "lessThan": "b6563ad0d599",
        "versionType": "git"
      },
      {
        "status": "affected",
        "version": "9b0fc3c054ff",
        "lessThan": "8ede3db5061b",
        "versionType": "git"
      }
    ],
    "programFiles": [
      "io_uring/net.c"
    ],
    "defaultStatus": "unaffected"
  },
  {
    "repo": "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git",
    "vendor": "Linux",
    "product": "Linux",
    "versions": [
      {
        "status": "affected",
        "version": "6.0"
      },
      {
        "status": "unaffected",
        "version": "0",
        "lessThan": "6.0",
        "versionType": "custom"
      },
      {
        "status": "unaffected",
        "version": "6.1.83",
        "versionType": "custom",
        "lessThanOrEqual": "6.1.*"
      },
      {
        "status": "unaffected",
        "version": "6.6.23",
        "versionType": "custom",
        "lessThanOrEqual": "6.6.*"
      },
      {
        "status": "unaffected",
        "version": "6.7.11",
        "versionType": "custom",
        "lessThanOrEqual": "6.7.*"
      },
      {
        "status": "unaffected",
        "version": "6.8.2",
        "versionType": "custom",
        "lessThanOrEqual": "6.8.*"
      },
      {
        "status": "unaffected",
        "version": "6.9",
        "versionType": "original_commit_for_fix",
        "lessThanOrEqual": "*"
      }
    ],
    "programFiles": [
      "io_uring/net.c"
    ],
    "defaultStatus": "affected"
  }
]