diff options
| -rw-r--r-- | meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base/CVE-2025-47806.patch | 50 | ||||
| -rw-r--r-- | meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.22.12.bb | 1 |
2 files changed, 51 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..632a5fb38e --- /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 4ea4ec6..035068d 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_1.22.12.bb b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.22.12.bb index 44ecdc0b55..bfc6bb65ef 100644 --- a/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.22.12.bb +++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0-plugins-base_1.22.12.bb | |||
| @@ -20,6 +20,7 @@ SRC_URI = "https://gstreamer.freedesktop.org/src/gst-plugins-base/gst-plugins-ba | |||
| 20 | file://0011-discoverer-Don-t-print-channel-layout-for-more-than-.patch \ | 20 | file://0011-discoverer-Don-t-print-channel-layout-for-more-than-.patch \ |
| 21 | file://0012-subparse-Check-for-NULL-return-of-strchr-when-parsin.patch \ | 21 | file://0012-subparse-Check-for-NULL-return-of-strchr-when-parsin.patch \ |
| 22 | file://CVE-2025-47808.patch \ | 22 | file://CVE-2025-47808.patch \ |
| 23 | file://CVE-2025-47806.patch \ | ||
| 23 | " | 24 | " |
| 24 | SRC_URI[sha256sum] = "73cfadc3a6ffe77ed974cfd6fb391c605e4531f48db21dd6b9f42b8cb69bd8c1" | 25 | SRC_URI[sha256sum] = "73cfadc3a6ffe77ed974cfd6fb391c605e4531f48db21dd6b9f42b8cb69bd8c1" |
| 25 | 26 | ||
