Bug 2347770 (CVE-2022-49086) - CVE-2022-49086 kernel: net: openvswitch: fix leak of nested actions
Summary: CVE-2022-49086 kernel: net: openvswitch: fix leak of nested actions
Keywords:
Status: NEW
Alias: CVE-2022-49086
Product: Security Response
Classification: Other
Component: vulnerability
Version: unspecified
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Product Security DevOps Team
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2025-02-26 03:06 UTC by OSIDB Bzimport
Modified: 2025-02-26 16:10 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Embargoed:


Attachments (Terms of Use)

Description OSIDB Bzimport 2025-02-26 03:06:09 UTC
In the Linux kernel, the following vulnerability has been resolved:

net: openvswitch: fix leak of nested actions

While parsing user-provided actions, openvswitch module may dynamically
allocate memory and store pointers in the internal copy of the actions.
So this memory has to be freed while destroying the actions.

Currently there are only two such actions: ct() and set().  However,
there are many actions that can hold nested lists of actions and
ovs_nla_free_flow_actions() just jumps over them leaking the memory.

For example, removal of the flow with the following actions will lead
to a leak of the memory allocated by nf_ct_tmpl_alloc():

  actions:clone(ct(commit),0)

Non-freed set() action may also leak the 'dst' structure for the
tunnel info including device references.

Under certain conditions with a high rate of flow rotation that may
cause significant memory leak problem (2MB per second in reporter's
case).  The problem is also hard to mitigate, because the user doesn't
have direct control over the datapath flows generated by OVS.

Fix that by iterating over all the nested actions and freeing
everything that needs to be freed recursively.

New build time assertion should protect us from this problem if new
actions will be added in the future.

Unfortunately, openvswitch module doesn't use NLA_F_NESTED, so all
attributes has to be explicitly checked.  sample() and clone() actions
are mixing extra attributes into the user-provided action list.  That
prevents some code generalization too.

Comment 1 Avinash Hanwate 2025-02-26 11:42:51 UTC
Upstream advisory:
https://lore.kernel.org/linux-cve-announce/2025022657-CVE-2022-49086-a289@gregkh/T

Comment 4 Avinash Hanwate 2025-02-26 16:09:13 UTC
Upstream advisory:
https://lore.kernel.org/linux-cve-announce/2025022657-CVE-2022-49086-a289@gregkh/T


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