]> git.ipfire.org Git - thirdparty/kernel/stable-queue.git/commitdiff
5.4-stable patches
authorGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 11 Mar 2025 14:25:47 +0000 (15:25 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 11 Mar 2025 14:25:47 +0000 (15:25 +0100)
added patches:
perf-cs-etm-add-missing-variable-in-cs_etm__process_queues.patch
udf-fix-use-of-check_add_overflow-with-mixed-type-arguments.patch

queue-5.4/perf-cs-etm-add-missing-variable-in-cs_etm__process_queues.patch [new file with mode: 0644]
queue-5.4/series
queue-5.4/udf-fix-use-of-check_add_overflow-with-mixed-type-arguments.patch [new file with mode: 0644]

diff --git a/queue-5.4/perf-cs-etm-add-missing-variable-in-cs_etm__process_queues.patch b/queue-5.4/perf-cs-etm-add-missing-variable-in-cs_etm__process_queues.patch
new file mode 100644 (file)
index 0000000..45e3204
--- /dev/null
@@ -0,0 +1,37 @@
+From stable+bounces-119388-greg=kroah.com@vger.kernel.org Mon Feb 24 17:20:13 2025
+From: Ben Hutchings <benh@debian.org>
+Date: Mon, 24 Feb 2025 16:57:20 +0100
+Subject: perf cs-etm: Add missing variable in cs_etm__process_queues()
+To: stable@vger.kernel.org
+Cc: James Clark <james.clark@linaro.org>
+Message-ID: <Z7yW4FuPxoZyl7Ga@decadent.org.uk>
+Content-Disposition: inline
+
+From: Ben Hutchings <benh@debian.org>
+
+Commit 5afd032961e8 "perf cs-etm: Don't flush when packet_queue fills
+up" uses i as a loop counter in cs_etm__process_queues().  It was
+backported to the 5.4 and 5.10 stable branches, but the i variable
+doesn't exist there as it was only added in 5.15.
+
+Declare i with the expected type.
+
+Fixes: 1ed167325c32 ("perf cs-etm: Don't flush when packet_queue fills up")
+Fixes: 26db806fa23e ("perf cs-etm: Don't flush when packet_queue fills up")
+Signed-off-by: Ben Hutchings <benh@debian.org>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+---
+ tools/perf/util/cs-etm.c |    2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+--- a/tools/perf/util/cs-etm.c
++++ b/tools/perf/util/cs-etm.c
+@@ -2139,7 +2139,7 @@ static int cs_etm__process_timeless_queu
+ static int cs_etm__process_queues(struct cs_etm_auxtrace *etm)
+ {
+       int ret = 0;
+-      unsigned int cs_queue_nr, queue_nr;
++      unsigned int cs_queue_nr, queue_nr, i;
+       u8 trace_chan_id;
+       u64 timestamp;
+       struct auxtrace_queue *queue;
index d59717fbc2fcbdcacdaac288a5ffe14b1d2df482..8336f11447ac7864361ab565a24c0c96a247d13e 100644 (file)
@@ -1,3 +1,5 @@
+perf-cs-etm-add-missing-variable-in-cs_etm__process_queues.patch
+udf-fix-use-of-check_add_overflow-with-mixed-type-arguments.patch
 afs-fix-directory-format-encoding-struct.patch
 nbd-don-t-allow-reconnect-after-disconnect.patch
 partitions-ldm-remove-the-initial-kernel-doc-notatio.patch
diff --git a/queue-5.4/udf-fix-use-of-check_add_overflow-with-mixed-type-arguments.patch b/queue-5.4/udf-fix-use-of-check_add_overflow-with-mixed-type-arguments.patch
new file mode 100644 (file)
index 0000000..b14f2c9
--- /dev/null
@@ -0,0 +1,43 @@
+From stable+bounces-119387-greg=kroah.com@vger.kernel.org Mon Feb 24 17:02:42 2025
+From: Ben Hutchings <benh@debian.org>
+Date: Mon, 24 Feb 2025 17:00:27 +0100
+Subject: udf: Fix use of check_add_overflow() with mixed type arguments
+To: stable@vger.kernel.org
+Cc: Jan Kara <jack@suse.cz>
+Message-ID: <Z7yXm_Vo1Y0Gjx_X@decadent.org.uk>
+Content-Disposition: inline
+
+From: Ben Hutchings <benh@debian.org>
+
+Commit ebbe26fd54a9 "udf: Avoid excessive partition lengths"
+introduced a use of check_add_overflow() with argument types u32,
+size_t, and u32 *.
+
+This was backported to the 5.x stable branches, where in 64-bit
+configurations it results in a build error (with older compilers) or a
+warning.  Before commit d219d2a9a92e "overflow: Allow mixed type
+arguments", which went into Linux 6.1, mixed type arguments are not
+supported.  That cannot be backported to 5.4 or 5.10 as it would raise
+the minimum compiler version for these kernel versions.
+
+Add a cast to make the argument types compatible.
+
+Fixes: 1497a4484cdb ("udf: Avoid excessive partition lengths")
+Fixes: 551966371e17 ("udf: Avoid excessive partition lengths")
+Signed-off-by: Ben Hutchings <benh@debian.org>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+---
+ fs/udf/super.c |    2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+--- a/fs/udf/super.c
++++ b/fs/udf/super.c
+@@ -1160,7 +1160,7 @@ static int udf_fill_partdesc_info(struct
+               map->s_partition_flags |= UDF_PART_FLAG_UNALLOC_BITMAP;
+               /* Check whether math over bitmap won't overflow. */
+               if (check_add_overflow(map->s_partition_len,
+-                                     sizeof(struct spaceBitmapDesc) << 3,
++                                     (u32)(sizeof(struct spaceBitmapDesc) << 3),
+                                      &sum)) {
+                       udf_err(sb, "Partition %d is too long (%u)\n", p_index,
+                               map->s_partition_len);