mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2025-06-05 23:18:52 +00:00
Code Issues 1 Projects Releases Wiki Activity

adaptivedemux2: fix critical when using an unsupported URI

Browse source 
adaptivedemux2 only supports http(s), trying to use it with, say,
file:// was raising a CRITICAL in libsoup.

Fix #2476

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/4421>
This commit is contained in:
Guillaume Desmottes 2023-04-11 17:54:23 +02:00 committed by GStreamer Marge Bot
parent f4aeac45e5
commit a921e40228
1 changed files with 9 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
subprojects/gst-plugins-good/ext/adaptivedemux2/gstadaptivedemux.c
View file

@ -953,6 +953,15 @@ handle_incoming_manifest (GstAdaptiveDemux * demux)
GST_DEBUG_OBJECT (demux, "Fetched manifest at URI: %s (base: %s)", GST_DEBUG_OBJECT (demux, "Fetched manifest at URI: %s (base: %s)",
demux->manifest_uri, GST_STR_NULL (demux->manifest_base_uri)); demux->manifest_uri, GST_STR_NULL (demux->manifest_base_uri));
if (!g_str_has_prefix (demux->manifest_uri, "http://")
&& !g_str_has_prefix (demux->manifest_uri, "https://")) {
GST_ELEMENT_ERROR (demux, STREAM, DEMUX,
(_("Invalid manifest URI")),
("Manifest URI needs to use either http:// or https://"));
ret = FALSE;
goto unlock_out;
}
} else { } else {
GST_WARNING_OBJECT (demux, "Upstream URI query failed."); GST_WARNING_OBJECT (demux, "Upstream URI query failed.");
} }