<html><head></head><body>URLs are already fixed up when they get to this point. Fixing up URLs twice is a recipe for disaster. We've tried.<br><br>-1<br><br><div class="gmail_quote">Le 9 octobre 2020 09:56:58 GMT+03:00, Thomas Guillem <thomas@gllm.fr> a écrit :<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<pre class="k9mail">This will allow VLC to open unvalid URLs (using a space instead of the<br>%20 encoding, for example). Every other browsers or players are doing<br>this fixup.<br><br>Fixes #3435<hr> modules/access/http/access.c | 12 ++++++++++--<br> 1 file changed, 10 insertions(+), 2 deletions(-)<br><br>diff --git a/modules/access/http/access.c b/modules/access/http/access.c<br>index a3805d0d426..123daaf51d3 100644<br>--- a/modules/access/http/access.c<br>+++ b/modules/access/http/access.c<br>@@ -171,19 +171,27 @@ static int Open(vlc_object_t *obj)<br>     struct vlc_url_t crd_url;<br>     char *psz_realm = NULL;<br> <br>-    vlc_UrlParse(&crd_url, access->psz_url);<br>+    char *url_fixup = vlc_uri_fixup(access->psz_url);<br>+    if (url_fixup == NULL)<br>+        goto error;<br>+<br>+    vlc_UrlParse(&crd_url, url_fixup);<br>     vlc_credential_init(&crd, &crd_url);<br> <br>     sys->manager = vlc_http_mgr_create(obj, jar);<br>     if (sys->manager == NULL)<br>+    {<br>+        free(url_fixup);<br>         goto error;<br>+    }<br> <br>     char *ua = var_InheritString(obj, "http-user-agent");<br>     char *referer = var_InheritString(obj, "http-referrer");<br>     bool live = var_InheritBool(obj, "http-continuous");<br> <br>     sys->resource = (live ? vlc_http_live_create : vlc_http_file_create)(<br>-        sys->manager, access->psz_url, ua, referer);<br>+        sys->manager, url_fixup, ua, referer);<br>+    free(url_fixup);<br>     free(referer);<br>     free(ua);<br> </pre></blockquote></div><br>-- <br>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté.</body></html>