From ac2489fa615cab5e4d0362131fb24c6a09ac1276 Mon Sep 17 00:00:00 2001 From: Greg Kroah-Hartman Date: Fri, 9 May 2025 10:57:58 +0200 Subject: [PATCH] 5.15-stable patches added patches: openvswitch-fix-unsafe-attribute-parsing-in-output_userspace.patch --- ...ttribute-parsing-in-output_userspace.patch | 36 +++++++++++++++++++ queue-5.15/series | 1 + 2 files changed, 37 insertions(+) create mode 100644 queue-5.15/openvswitch-fix-unsafe-attribute-parsing-in-output_userspace.patch diff --git a/queue-5.15/openvswitch-fix-unsafe-attribute-parsing-in-output_userspace.patch b/queue-5.15/openvswitch-fix-unsafe-attribute-parsing-in-output_userspace.patch new file mode 100644 index 0000000000..1a42384e08 --- /dev/null +++ b/queue-5.15/openvswitch-fix-unsafe-attribute-parsing-in-output_userspace.patch @@ -0,0 +1,36 @@ +From 6beb6835c1fbb3f676aebb51a5fee6b77fed9308 Mon Sep 17 00:00:00 2001 +From: Eelco Chaudron +Date: Tue, 6 May 2025 16:28:54 +0200 +Subject: openvswitch: Fix unsafe attribute parsing in output_userspace() + +From: Eelco Chaudron + +commit 6beb6835c1fbb3f676aebb51a5fee6b77fed9308 upstream. + +This patch replaces the manual Netlink attribute iteration in +output_userspace() with nla_for_each_nested(), which ensures that only +well-formed attributes are processed. + +Fixes: ccb1352e76cf ("net: Add Open vSwitch kernel components.") +Signed-off-by: Eelco Chaudron +Acked-by: Ilya Maximets +Acked-by: Aaron Conole +Link: https://patch.msgid.link/0bd65949df61591d9171c0dc13e42cea8941da10.1746541734.git.echaudro@redhat.com +Signed-off-by: Jakub Kicinski +Signed-off-by: Greg Kroah-Hartman +--- + net/openvswitch/actions.c | 3 +-- + 1 file changed, 1 insertion(+), 2 deletions(-) + +--- a/net/openvswitch/actions.c ++++ b/net/openvswitch/actions.c +@@ -954,8 +954,7 @@ static int output_userspace(struct datap + upcall.cmd = OVS_PACKET_CMD_ACTION; + upcall.mru = OVS_CB(skb)->mru; + +- for (a = nla_data(attr), rem = nla_len(attr); rem > 0; +- a = nla_next(a, &rem)) { ++ nla_for_each_nested(a, attr, rem) { + switch (nla_type(a)) { + case OVS_USERSPACE_ATTR_USERDATA: + upcall.userdata = a; diff --git a/queue-5.15/series b/queue-5.15/series index d63f905446..e9964c0f5b 100644 --- a/queue-5.15/series +++ b/queue-5.15/series @@ -1,2 +1,3 @@ can-mcan-m_can_class_unregister-fix-order-of-unregistration-calls.patch can-mcp251xfd-mcp251xfd_remove-fix-order-of-unregistration-calls.patch +openvswitch-fix-unsafe-attribute-parsing-in-output_userspace.patch -- 2.47.3