From e73e34fd6fbe2d2d7d6a54d0f459935bc41babe3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= Date: Sun, 26 May 2024 14:02:12 +0300 Subject: [PATCH] rtspsrc: Also handle `rtsps://` and similar URLs as absolute in other places Previously a direct comparison with `rtsp://` was performed, which didn't catch cases like `rtsps://`. Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/3563 Part-of: --- subprojects/gst-plugins-good/gst/rtsp/gstrtspsrc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/subprojects/gst-plugins-good/gst/rtsp/gstrtspsrc.c b/subprojects/gst-plugins-good/gst/rtsp/gstrtspsrc.c index 154b3b40e3..04632df3a1 100644 --- a/subprojects/gst-plugins-good/gst/rtsp/gstrtspsrc.c +++ b/subprojects/gst-plugins-good/gst/rtsp/gstrtspsrc.c @@ -2526,7 +2526,7 @@ gst_rtspsrc_create_stream (GstRTSPSrc * src, GstSDPMessage * sdp, gint idx, * If the control_path starts with a non rtsp: protocol we will most * likely build a URL that the server will fail to understand, this is ok, * we will fail then. */ - if (g_str_has_prefix (control_path, "rtsp://")) + if (gst_uri_is_valid (control_path)) stream->conninfo.location = g_strdup (control_path); else { const gchar *base; @@ -8261,7 +8261,7 @@ gst_rtspsrc_open_from_sdp (GstRTSPSrc * src, GstSDPMessage * sdp, break; /* only take fully qualified urls */ - if (g_str_has_prefix (control, "rtsp://")) + if (gst_uri_is_valid (control)) break; } if (control) {