diff options
Diffstat (limited to 'meta/recipes-multimedia/gstreamer/gstreamer1.0/0005-allocator-Avoid-integer-overflow-when-allocating-sys.patch')
-rw-r--r-- | meta/recipes-multimedia/gstreamer/gstreamer1.0/0005-allocator-Avoid-integer-overflow-when-allocating-sys.patch | 56 |
1 files changed, 56 insertions, 0 deletions
diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0/0005-allocator-Avoid-integer-overflow-when-allocating-sys.patch b/meta/recipes-multimedia/gstreamer/gstreamer1.0/0005-allocator-Avoid-integer-overflow-when-allocating-sys.patch new file mode 100644 index 0000000000..5d8575711a --- /dev/null +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0/0005-allocator-Avoid-integer-overflow-when-allocating-sys.patch | |||
@@ -0,0 +1,56 @@ | |||
1 | From f1cdc6f24340f6cce4cc7020628002f5c70dd6c7 Mon Sep 17 00:00:00 2001 | ||
2 | From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= <sebastian@centricular.com> | ||
3 | Date: Thu, 26 Sep 2024 22:07:22 +0300 | ||
4 | Subject: [PATCH] allocator: Avoid integer overflow when allocating sysmem | ||
5 | |||
6 | Thanks to Antonio Morales for finding and reporting the issue. | ||
7 | |||
8 | Fixes GHSL-2024-166 | ||
9 | Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/3851 | ||
10 | |||
11 | Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/8032> | ||
12 | |||
13 | CVE: CVE-2024-47606 | ||
14 | Upstream-Status: Backport [https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/f1cdc6f24340f6cce4cc7020628002f5c70dd6c7] | ||
15 | Signed-off-by: Peter Marko <peter.marko@siemens.com> | ||
16 | --- | ||
17 | gst/gstallocator.c | 14 ++++++++++++++ | ||
18 | 1 file changed, 14 insertions(+) | ||
19 | |||
20 | diff --git a/gst/gstallocator.c b/gst/gstallocator.c | ||
21 | index 996f5dc946..198cfe9523 100644 | ||
22 | --- a/gst/gstallocator.c | ||
23 | +++ b/gst/gstallocator.c | ||
24 | @@ -430,8 +430,20 @@ _sysmem_new_block (GstMemoryFlags flags, | ||
25 | /* ensure configured alignment */ | ||
26 | align |= gst_memory_alignment; | ||
27 | /* allocate more to compensate for alignment */ | ||
28 | + if (align > G_MAXSIZE || maxsize > G_MAXSIZE - align) { | ||
29 | + GST_CAT_WARNING (GST_CAT_MEMORY, | ||
30 | + "Allocating %" G_GSIZE_FORMAT " bytes with alignment %" G_GSIZE_FORMAT | ||
31 | + "x overflows", maxsize, align); | ||
32 | + return NULL; | ||
33 | + } | ||
34 | maxsize += align; | ||
35 | /* alloc header and data in one block */ | ||
36 | + if (maxsize > G_MAXSIZE - sizeof (GstMemorySystem)) { | ||
37 | + GST_CAT_WARNING (GST_CAT_MEMORY, | ||
38 | + "Allocating %" G_GSIZE_FORMAT " bytes with alignment %" G_GSIZE_FORMAT | ||
39 | + "x overflows", maxsize, align); | ||
40 | + return NULL; | ||
41 | + } | ||
42 | slice_size = sizeof (GstMemorySystem) + maxsize; | ||
43 | |||
44 | mem = g_slice_alloc (slice_size); | ||
45 | @@ -481,6 +493,8 @@ _sysmem_copy (GstMemorySystem * mem, gssize offset, gsize size) | ||
46 | size = mem->mem.size > offset ? mem->mem.size - offset : 0; | ||
47 | |||
48 | copy = _sysmem_new_block (0, size, mem->mem.align, 0, size); | ||
49 | + if (!copy) | ||
50 | + return NULL; | ||
51 | GST_CAT_DEBUG (GST_CAT_PERFORMANCE, | ||
52 | "memcpy %" G_GSIZE_FORMAT " memory %p -> %p", size, mem, copy); | ||
53 | memcpy (copy->data, mem->data + mem->mem.offset + offset, size); | ||
54 | -- | ||
55 | 2.30.2 | ||
56 | |||