Bug 1224606 (CVE-2024-35827) - VUL-0: CVE-2024-35827: kernel: io_uring/net: fix overflow check in io_recvmsg_mshot_prep()
Summary: VUL-0: CVE-2024-35827: kernel: io_uring/net: fix overflow check in io_recvmsg...
Status: IN_PROGRESS
Alias: CVE-2024-35827
Product: SUSE Security Incidents
Classification: Novell Products
Component: Incidents (show other bugs)
Version: unspecified
Hardware: Other Other
: P3 - Medium : Normal
Target Milestone: ---
Assignee: Security Team bot
QA Contact: Security Team bot
URL: https://smash.suse.de/issue/406447/
Whiteboard: CVSSv3.1:SUSE:CVE-2024-35827:5.5:(AV:...
Keywords:
Depends on:
Blocks:
 
Reported: 2024-05-20 15:30 UTC by SMASH SMASH
Modified: 2024-07-08 14:53 UTC (History)
3 users (show)

See Also:
Found By: Security Response Team
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description SMASH SMASH 2024-05-20 15:30:14 UTC
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.

References:
http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2024-35827
https://git.kernel.org/pub/scm/linux/security/vulns.git/plain/cve/published/2024/CVE-2024-35827.mbox
https://git.kernel.org/stable/c/868ec868616438df487b9e2baa5a99f8662cc47c
https://git.kernel.org/stable/c/59a534690ecc3af72c6ab121aeac1237a4adae66
https://git.kernel.org/stable/c/0c8c74bb59e7d77554016efc34c2d10376985e5e
https://git.kernel.org/stable/c/b6563ad0d599110bd5cf8f56c47d279c3ed796fe
https://git.kernel.org/stable/c/8ede3db5061bb1fe28e2c9683329aafa89d2b1b4
https://www.cve.org/CVERecord?id=CVE-2024-35827
https://bugzilla.redhat.com/show_bug.cgi?id=2281183
Comment 1 Joey Lee 2024-05-21 08:45:25 UTC
https://www.suse.com/security/cve/CVE-2024-35827.html
cvss 5.5
Comment 4 Gabriel Krisman Bertazi 2024-06-14 15:20:27 UTC
backport applied to all branches. reassigning to security team.