Bug 2266421 (CVE-2021-46912) - CVE-2021-46912 kernel: namespace leak into all other net namespaces
Summary: CVE-2021-46912 kernel: namespace leak into all other net namespaces
Keywords:
Status: NEW
Alias: CVE-2021-46912
Product: Security Response
Classification: Other
Component: vulnerability
Version: unspecified
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Product Security
QA Contact:
URL:
Whiteboard:
Depends On: 2266426
Blocks: 2266369
TreeView+ depends on / blocked
 
Reported: 2024-02-27 19:16 UTC by Rohit Keshri
Modified: 2024-03-27 19:23 UTC (History)
49 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
A flaw was found in the network sub-component in the Linux Kernel. The tcp_allowed_congestion_control is global and writable, and writing to it in any net namespace will leak into all other net namespaces.
Clone Of:
Environment:
Last Closed:
Embargoed:


Attachments (Terms of Use)

Description Rohit Keshri 2024-02-27 19:16:56 UTC
In the Linux kernel, the following vulnerability has been resolved:

net: Make tcp_allowed_congestion_control readonly in non-init netns

Currently, tcp_allowed_congestion_control is global and writable;
writing to it in any net namespace will leak into all other net
namespaces.

tcp_available_congestion_control and tcp_allowed_congestion_control are
the only sysctls in ipv4_net_table (the per-netns sysctl table) with a
NULL data pointer; their handlers (proc_tcp_available_congestion_control
and proc_allowed_congestion_control) have no other way of referencing a
struct net. Thus, they operate globally.

Because ipv4_net_table does not use designated initializers, there is no
easy way to fix up this one "bad" table entry. However, the data pointer
updating logic shouldn't be applied to NULL pointers anyway, so we
instead force these entries to be read-only.

These sysctls used to exist in ipv4_table (init-net only), but they were
moved to the per-net ipv4_net_table, presumably without realizing that
tcp_allowed_congestion_control was writable and thus introduced a leak.

Because the intent of that commit was only to know (i.e. read) "which
congestion algorithms are available or allowed", this read-only solution
should be sufficient.

The logic added in recent commit
31c4d2f160eb: ("net: Ensure net namespace isolation of sysctls")
does not and cannot check for NULL data pointers, because
other table entries (e.g. /proc/sys/net/netfilter/nf_log/) have
.data=NULL but use other methods (.extra2) to access the struct net.

https://git.kernel.org/stable/c/1ccdf1bed140820240e383ba0accc474ffc7f006
https://git.kernel.org/stable/c/35d7491e2f77ce480097cabcaf93ed409e916e12
https://git.kernel.org/stable/c/97684f0970f6e112926de631fdd98d9693c7e5c1

Comment 1 Rohit Keshri 2024-02-27 19:36:23 UTC
Created kernel tracking bugs for this issue:

Affects: fedora-all [bug 2266426]

Comment 3 Justin M. Forbes 2024-02-27 23:04:24 UTC
This was fixed for Fedora with the 5.11.16 stable kernel updates.


Note You need to log in before you can comment on or make changes to this bug.