This does not work for the following testcases, + lines are invalid:
+test("/foo/bar", "..");
+test("/foo/bar/baz", "..");
- /foo/bar .. =>
/
- /foo/bar/baz .. =>
/foo/
+ /foo/bar .. =>
//foo/bar/baz
+ /foo/bar/baz .. =>
/foo//foo/bar/baz
This one would be also nice if it worked:
+test("http://server/foo/bar;, "a/b/../c/d/../e../..f/.../other/url/");
- http://server/foo/bar
a/b/../c/d/../e../..f/.../other/url/.. =>
http://server/foo/a/c/e../..f/.../other/
+ http://server/foo/bar
a/b/../c/d/../e../..f/.../other/url/.. =>
http://server/foo/a/c/e../..f/.../other
On Mon, Jul 27, 2020 at 3:08 PM Steven Liu wrote:
> Signed-off-by: Steven Liu
> ---
> libavformat/tests/url.c | 14 ++
> tests/ref/fate/url | 12
> 2 files changed, 26 insertions(+)
>
> diff --git a/libavformat/tests/url.c b/libavformat/tests/url.c
> index 1d961a1b43..0de511caf9 100644
> --- a/libavformat/tests/url.c
> +++ b/libavformat/tests/url.c
> @@ -24,6 +24,8 @@
> static void test(const char *base, const char *rel)
> {
> char buf[200], buf2[200];
> +memset(buf, 0, 200);
> +memset(buf2, 0, 200);
> ff_make_absolute_url(buf, sizeof(buf), base, rel);
> printf("%50s %-20s => %s\n", base, rel, buf);
> if (base) {
> @@ -66,7 +68,19 @@ int main(void)
> test("http://server/foo/bar?param=value/with/slashes;, "/baz");
> test("http://server/foo/bar?param;, "?someparam");
> test("http://server/foo/bar;, "//other/url");
> +test("http://server/foo/bar;, "../other/url");
> +test("http://server/foo/bar;, "other/url");
> test("http://server/foo/bar;, "../../../../../other/url");
> +test("http://server/foo/bar;, "../../../../../other/url/test..mp3");
> +test("http://server/foo/bar;, "../../../../../other/url/test..");
> +test("http://server/foo/bar;, "../../../../../other/url/test/...");
> +test("http://server/foo/bar;,
> "../../../../../other/url/.../test/out");
> +test("http://server/foo/bar;,
> "../../../../../other/url/.../../test/out");
> +test("http://server/foo/bar;,
> "../../../../../other/url/.../..test/out");
> +test("http://server/foo/bar;, "../../../../../other/url/..");
> +test("http://server/foo/bar;, "../../../../../other/url/..mp3");
> +test("http://server/foo/bar;, "../../../../../other/url/..test/mp3");
> +test("http://server/foo/bar;, "../../../../../other/url/test../mp3");
> test("http://server/foo/bar;, "/../../../../../other/url");
> test("http://server/foo/bar;, "/test/../../../../../other/url");
> test("http://server/foo/bar;, "/test/../../test/../../../other/url");
> diff --git a/tests/ref/fate/url b/tests/ref/fate/url
> index 533ba2cb1e..35eee25f4a 100644
> --- a/tests/ref/fate/url
> +++ b/tests/ref/fate/url
> @@ -13,7 +13,19 @@ Testing ff_make_absolute_url:
> http://server/foo/bar?param=value/with/slashes /baz
> => http://server/baz
> http://server/foo/bar?param ?someparam
> => http://server/foo/bar?someparam
> http://server/foo/bar //other/url
> => http://other/url
> + http://server/foo/bar ../other/url =>
> http://server/other/url
> + http://server/foo/bar other/url =>
> http://server/foo/other/url
> http://server/foo/bar
> ../../../../../other/url => http://server/other/url
> + http://server/foo/bar
> ../../../../../other/url/test..mp3 => http://server/other/url/test..mp3
> + http://server/foo/bar
> ../../../../../other/url/test.. => http://server/other/url/test..
> + http://server/foo/bar
> ../../../../../other/url/test/... => http://server/other/url/test/...
> + http://server/foo/bar
> ../../../../../other/url/.../test/out =>
> http://server/other/url/.../test/out
> + http://server/foo/bar
> ../../../../../other/url/.../../test/out =>
> http://server/other/url/test/out
> + http://server/foo/bar
> ../../../../../other/url/.../..test/out =>
> http://server/other/url/.../..test/out
> + http://server/foo/bar
> ../../../../../other/url/.. => http://server/other
> + http://server/foo/bar
> ../../../../../other/url/..mp3 => http://server/other/url/..mp3
> + http://server/foo/bar
> ../../../../../other/url/..test/mp3 => http://server/other/url/..test/mp3
> + http://server/foo/bar
> ../../../../../other/url/test../mp3 => http://server/other/url/test../mp3
>
