diff options
3 files changed, 88 insertions, 0 deletions
diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/CVE-2025-47806.patch b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/CVE-2025-47806.patch new file mode 100644 index 0000000000..530d579231 --- /dev/null +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/CVE-2025-47806.patch | |||
@@ -0,0 +1,50 @@ | |||
1 | From da4380c4df0e00f8d0bad569927bfc7ea35ec37d Mon Sep 17 00:00:00 2001 | ||
2 | From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= <sebastian@centricular.com> | ||
3 | Date: Thu, 8 May 2025 12:46:40 +0300 | ||
4 | Subject: [PATCH] subparse: Make sure that subrip time string is not too long | ||
5 | before zero-padding | ||
6 | |||
7 | Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/4419 | ||
8 | Fixes CVE-2025-47806 | ||
9 | |||
10 | Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/9135> | ||
11 | |||
12 | CVE: CVE-2025-47806 | ||
13 | Upstream-Status: Backport [https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/da4380c4df0e00f8d0bad569927bfc7ea35ec37d] | ||
14 | Signed-off-by: Hitendra Prajapati <hprajapati@mvista.com> | ||
15 | --- | ||
16 | gst/subparse/gstsubparse.c | 11 ++++++++++- | ||
17 | 1 file changed, 10 insertions(+), 1 deletion(-) | ||
18 | |||
19 | diff --git a/gst/subparse/gstsubparse.c b/gst/subparse/gstsubparse.c | ||
20 | index 1d8fa51..81a7f65 100644 | ||
21 | --- a/gst/subparse/gstsubparse.c | ||
22 | +++ b/gst/subparse/gstsubparse.c | ||
23 | @@ -850,7 +850,7 @@ parse_subrip_time (const gchar * ts_string, GstClockTime * t) | ||
24 | g_strdelimit (s, " ", '0'); | ||
25 | g_strdelimit (s, ".", ','); | ||
26 | |||
27 | - /* make sure we have exactly three digits after he comma */ | ||
28 | + /* make sure we have exactly three digits after the comma */ | ||
29 | p = strchr (s, ','); | ||
30 | if (p == NULL) { | ||
31 | /* If there isn't a ',' the timestamp is broken */ | ||
32 | @@ -859,6 +859,15 @@ parse_subrip_time (const gchar * ts_string, GstClockTime * t) | ||
33 | return FALSE; | ||
34 | } | ||
35 | |||
36 | + /* Check if the comma is too far into the string to avoid | ||
37 | + * stack overflow when zero-padding the sub-second part. | ||
38 | + * | ||
39 | + * Allow for 3 digits of hours just in case. */ | ||
40 | + if ((p - s) > sizeof ("hhh:mm:ss,")) { | ||
41 | + GST_WARNING ("failed to parse subrip timestamp string '%s'", s); | ||
42 | + return FALSE; | ||
43 | + } | ||
44 | + | ||
45 | ++p; | ||
46 | len = strlen (p); | ||
47 | if (len > 3) { | ||
48 | -- | ||
49 | 2.50.1 | ||
50 | |||
diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/CVE-2025-47808.patch b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/CVE-2025-47808.patch new file mode 100644 index 0000000000..5b9fefc321 --- /dev/null +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/CVE-2025-47808.patch | |||
@@ -0,0 +1,36 @@ | |||
1 | From 6b19f117518a765a25c99d1c4b09f2838a8ed0c9 Mon Sep 17 00:00:00 2001 | ||
2 | From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= <sebastian@centricular.com> | ||
3 | Date: Thu, 8 May 2025 09:04:52 +0300 | ||
4 | Subject: [PATCH] tmplayer: Don't append NULL + 1 to the string buffer when | ||
5 | parsing lines without text | ||
6 | |||
7 | Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/4417 | ||
8 | Fixes CVE-2025-47808 | ||
9 | |||
10 | Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/9132> | ||
11 | |||
12 | CVE: CVE-2025-47808 | ||
13 | Upstream-Status: Backport [https://gitlab.freedesktop.org/gstreamer/gstreamer/-/commit/6b19f117518a765a25c99d1c4b09f2838a8ed0c9] | ||
14 | Signed-off-by: Hitendra Prajapati <hprajapati@mvista.com> | ||
15 | --- | ||
16 | gst/subparse/tmplayerparse.c | 4 +++- | ||
17 | 1 file changed, 3 insertions(+), 1 deletion(-) | ||
18 | |||
19 | diff --git a/gst/subparse/tmplayerparse.c b/gst/subparse/tmplayerparse.c | ||
20 | index 807e332..a9225d3 100644 | ||
21 | --- a/gst/subparse/tmplayerparse.c | ||
22 | +++ b/gst/subparse/tmplayerparse.c | ||
23 | @@ -125,7 +125,9 @@ tmplayer_parse_line (ParserState * state, const gchar * line, guint line_num) | ||
24 | * durations from the start times anyway, so as long as the parser just | ||
25 | * forwards state->start_time by duration after it pushes the line we | ||
26 | * are about to return it will all be good. */ | ||
27 | - g_string_append (state->buf, text_start + 1); | ||
28 | + if (text_start) { | ||
29 | + g_string_append (state->buf, text_start + 1); | ||
30 | + } | ||
31 | } else if (line_num > 0) { | ||
32 | GST_WARNING ("end of subtitle unit but no valid start time?!"); | ||
33 | } | ||
34 | -- | ||
35 | 2.50.1 | ||
36 | |||
diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.20.7.bb b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.20.7.bb index fc9afff628..05d58e83b0 100644 --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.20.7.bb +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.20.7.bb | |||
@@ -20,6 +20,8 @@ SRC_URI = "https://gstreamer.freedesktop.org/src/gst-plugins-base/gst-plugins-ba | |||
20 | file://CVE-2024-47615-1.patch \ | 20 | file://CVE-2024-47615-1.patch \ |
21 | file://CVE-2024-47615-2.patch \ | 21 | file://CVE-2024-47615-2.patch \ |
22 | file://CVE-2024-47835.patch \ | 22 | file://CVE-2024-47835.patch \ |
23 | file://CVE-2025-47806.patch \ | ||
24 | file://CVE-2025-47808.patch \ | ||
23 | " | 25 | " |
24 | SRC_URI[sha256sum] = "fde6696a91875095d82c1012b5777c28ba926047ffce08508e12c1d2c66f0057" | 26 | SRC_URI[sha256sum] = "fde6696a91875095d82c1012b5777c28ba926047ffce08508e12c1d2c66f0057" |
25 | 27 | ||