[vlc-commits] [Git][videolan/vlc][master] 2 commits: url: allow brackets in path part

[Jean-Baptiste Kempf (@jbk)]

Jean-Baptiste Kempf pushed to branch master at VideoLAN / VLC


Commits:
721751d4 by Rémi Denis-Courmont at 2021-09-15T08:09:51+00:00
url: allow brackets in path part

As brackets are separators only within the authority part, thier use
within (or on the edges of) the path does not create any ambiguity.

Fixes #26092.

- - - - -
ff0a2522 by Rémi Denis-Courmont at 2021-09-15T08:09:51+00:00
url: add test case

- - - - -


2 changed files:

- src/test/url.c
- src/text/url.c


Changes:

=====================================
src/test/url.c
=====================================
@@ -293,6 +293,8 @@ int main (void)
                    "www.example.com", 0, "/caf%C3%A9/", NULL);
     test_url_parse("p://h/white%20spaced", "p", NULL, NULL, "h", 0,
                    "/white%20spaced", NULL);
+    test_url_parse("p://h/[hello:world]", "p", NULL, NULL, "h", 0,
+                   "/[hello:world]", NULL);
     /* Relative URIs */
     test_url_parse("//example.com", NULL, NULL, NULL, "example.com", 0,
                    NULL, NULL);


=====================================
src/text/url.c
=====================================
@@ -404,7 +404,7 @@ static bool vlc_uri_host_validate(const char *str)
 
 static bool vlc_uri_path_validate(const char *str)
 {
-    return vlc_uri_component_validate(str, "/@:");
+    return vlc_uri_component_validate(str, "/@:[]");
 }
 
 static int vlc_UrlParseInner(vlc_url_t *restrict url, const char *str)



View it on GitLab: https://code.videolan.org/videolan/vlc/-/compare/a09a26d4b7fa788f057b43f6022ba99bdee9b6ec...ff0a252270401c4a0f09580c692f544430ec8883

-- 
View it on GitLab: https://code.videolan.org/videolan/vlc/-/compare/a09a26d4b7fa788f057b43f6022ba99bdee9b6ec...ff0a252270401c4a0f09580c692f544430ec8883
You're receiving this email because of your account on code.videolan.org.