mirror of
https://gitlab.freedesktop.org/gstreamer/gstreamer.git
synced 2024-11-22 17:51:16 +00:00
rtsp: gstrtspurl: gst_rtsp_url_get_request_uri: fix incorrect scheme for tls transport methods
gst_rtsp_url_get_request_uri returns rtsp://... url when requested url is rtsps://, this is not in accordance with https://www.rfc-editor.org/rfc/rfc7826.html#section-19.2. This also impedes setting up a rtsps session with a live555 rtsp server. Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3568>
This commit is contained in:
parent
a62b07f3b4
commit
024ef7659d
1 changed files with 13 additions and 3 deletions
|
@ -59,9 +59,11 @@
|
|||
G_DEFINE_BOXED_TYPE (GstRTSPUrl, gst_rtsp_url,
|
||||
(GBoxedCopyFunc) gst_rtsp_url_copy, (GBoxedFreeFunc) gst_rtsp_url_free);
|
||||
|
||||
#define SCHEME_SIZE 7
|
||||
|
||||
static const struct
|
||||
{
|
||||
const char scheme[6];
|
||||
const char scheme[SCHEME_SIZE];
|
||||
GstRTSPLowerTrans transports;
|
||||
} rtsp_schemes_map[] = {
|
||||
{
|
||||
|
@ -326,6 +328,8 @@ gst_rtsp_url_get_request_uri (const GstRTSPUrl * url)
|
|||
const gchar *post_host;
|
||||
const gchar *pre_query;
|
||||
const gchar *query;
|
||||
gchar scheme[SCHEME_SIZE] = "rtsp";
|
||||
guint i;
|
||||
|
||||
g_return_val_if_fail (url != NULL, NULL);
|
||||
|
||||
|
@ -333,12 +337,18 @@ gst_rtsp_url_get_request_uri (const GstRTSPUrl * url)
|
|||
post_host = url->family == GST_RTSP_FAM_INET6 ? "]" : "";
|
||||
pre_query = url->query ? "?" : "";
|
||||
query = url->query ? url->query : "";
|
||||
for (i = 0; i < G_N_ELEMENTS (rtsp_schemes_map); i++) {
|
||||
if (url->transports == rtsp_schemes_map[i].transports) {
|
||||
strcpy (scheme, rtsp_schemes_map[i].scheme);
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
if (url->port != 0) {
|
||||
uri = g_strdup_printf ("rtsp://%s%s%s:%u%s%s%s", pre_host, url->host,
|
||||
uri = g_strdup_printf ("%s://%s%s%s:%u%s%s%s", scheme, pre_host, url->host,
|
||||
post_host, url->port, url->abspath, pre_query, query);
|
||||
} else {
|
||||
uri = g_strdup_printf ("rtsp://%s%s%s%s%s%s", pre_host, url->host,
|
||||
uri = g_strdup_printf ("%s://%s%s%s%s%s%s", scheme, pre_host, url->host,
|
||||
post_host, url->abspath, pre_query, query);
|
||||
}
|
||||
return uri;
|
||||
|
|
Loading…
Reference in a new issue