lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Message-ID: <2024052020-CVE-2024-35960-2eaa@gregkh>
Date: Mon, 20 May 2024 11:42:25 +0200
From: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To: linux-cve-announce@...r.kernel.org
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Subject: CVE-2024-35960: net/mlx5: Properly link new fs rules into the tree

Description
===========

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

net/mlx5: Properly link new fs rules into the tree

Previously, add_rule_fg would only add newly created rules from the
handle into the tree when they had a refcount of 1. On the other hand,
create_flow_handle tries hard to find and reference already existing
identical rules instead of creating new ones.

These two behaviors can result in a situation where create_flow_handle
1) creates a new rule and references it, then
2) in a subsequent step during the same handle creation references it
   again,
resulting in a rule with a refcount of 2 that is not linked into the
tree, will have a NULL parent and root and will result in a crash when
the flow group is deleted because del_sw_hw_rule, invoked on rule
deletion, assumes node->parent is != NULL.

This happened in the wild, due to another bug related to incorrect
handling of duplicate pkt_reformat ids, which lead to the code in
create_flow_handle incorrectly referencing a just-added rule in the same
flow handle, resulting in the problem described above. Full details are
at [1].

This patch changes add_rule_fg to add new rules without parents into
the tree, properly initializing them and avoiding the crash. This makes
it more consistent with how rules are added to an FTE in
create_flow_handle.

The Linux kernel CVE team has assigned CVE-2024-35960 to this issue.


Affected and fixed versions
===========================

	Issue introduced in 4.10 with commit 74491de93712 and fixed in 4.19.313 with commit de0139719cdd
	Issue introduced in 4.10 with commit 74491de93712 and fixed in 5.4.275 with commit 1263b0b26077
	Issue introduced in 4.10 with commit 74491de93712 and fixed in 5.10.216 with commit 3d90ca9145f6
	Issue introduced in 4.10 with commit 74491de93712 and fixed in 5.15.156 with commit 7aaee12b804c
	Issue introduced in 4.10 with commit 74491de93712 and fixed in 6.1.87 with commit 2e8dc5cffc84
	Issue introduced in 4.10 with commit 74491de93712 and fixed in 6.6.28 with commit 5cf5337ef701
	Issue introduced in 4.10 with commit 74491de93712 and fixed in 6.8.7 with commit adf67a03af39
	Issue introduced in 4.10 with commit 74491de93712 and fixed in 6.9 with commit 7c6782ad4911

Please see https://www.kernel.org for a full list of currently supported
kernel versions by the kernel community.

Unaffected versions might change over time as fixes are backported to
older supported kernel versions.  The official CVE entry at
	https://cve.org/CVERecord/?id=CVE-2024-35960
will be updated if fixes are backported, please check that for the most
up to date information about this issue.


Affected files
==============

The file(s) affected by this issue are:
	drivers/net/ethernet/mellanox/mlx5/core/fs_core.c


Mitigation
==========

The Linux kernel CVE team recommends that you update to the latest
stable kernel version for this, and many other bugfixes.  Individual
changes are never tested alone, but rather are part of a larger kernel
release.  Cherry-picking individual commits is not recommended or
supported by the Linux kernel community at all.  If however, updating to
the latest release is impossible, the individual changes to resolve this
issue can be found at these commits:
	https://git.kernel.org/stable/c/de0139719cdda82806a47580ca0df06fc85e0bd2
	https://git.kernel.org/stable/c/1263b0b26077b1183c3c45a0a2479573a351d423
	https://git.kernel.org/stable/c/3d90ca9145f6b97b38d0c2b6b30f6ca6af9c1801
	https://git.kernel.org/stable/c/7aaee12b804c5e0374e7b132b6ec2158ff33dd64
	https://git.kernel.org/stable/c/2e8dc5cffc844dacfa79f056dea88002312f253f
	https://git.kernel.org/stable/c/5cf5337ef701830f173b4eec00a4f984adeb57a0
	https://git.kernel.org/stable/c/adf67a03af39095f05d82050f15813d6f700159d
	https://git.kernel.org/stable/c/7c6782ad4911cbee874e85630226ed389ff2e453

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ