luc14n0/glib
1
0
Fork 0
mirror of https://gitlab.gnome.org/GNOME/glib.git synced 2025-01-14 00:06:24 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge branch 'uri-double-slash-path' into 'main'

Browse Source 
guri: Fix paths starting with ‘//’ when parsing

See merge request GNOME/glib!4407
This commit is contained in:
Philip Withnall 2024-11-26 11:08:12 +00:00
commit fcc0e4d556
2 changed files with 37 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
glib/guri.c
View File

@ -1535,6 +1535,18 @@ g_uri_parse_relative (GUri *base_uri,
remove_dot_segments (uri->path);
}
/* Fix up the invalid cases from
* https://datatracker.ietf.org/doc/html/rfc3986#section-3, as otherwise
* calling g_uri_to_string() on this URI will fail. These can be caused by
* remove_dot_segments(), e.g. `data:/.//` gets normalised to `data://` whose
* path is invalid given the lack of an authority. */
if (uri->host == NULL && uri->path[0] == '/' && uri->path[1] == '/')
{
char *new_path = g_strconcat ("/.", uri->path, NULL);
g_free (uri->path);
uri->path = g_steal_pointer (&new_path);
}
return g_steal_pointer (&uri);
}

25
glib/tests/uri.c
View File

@ -769,6 +769,18 @@ static const UriAbsoluteTest absolute_tests[] = {
/* Invalid IDN hostname */
{ "http://xn--mixed-\xc3\xbcp/", G_URI_FLAGS_NONE, FALSE, G_URI_ERROR_BAD_HOST,
{ NULL, NULL, NULL, -1, NULL, NULL, NULL } },
/* Paths with double slashes */
{ "data:.///",
G_URI_FLAGS_HAS_PASSWORD | G_URI_FLAGS_ENCODED_PATH | G_URI_FLAGS_ENCODED_QUERY | G_URI_FLAGS_ENCODED_FRAGMENT | G_URI_FLAGS_SCHEME_NORMALIZE,
TRUE, 0,
{ "data", NULL, NULL, -1, "/.//", NULL, NULL }
},
{ "data:/.//",
G_URI_FLAGS_HAS_PASSWORD | G_URI_FLAGS_ENCODED_PATH | G_URI_FLAGS_ENCODED_QUERY | G_URI_FLAGS_ENCODED_FRAGMENT | G_URI_FLAGS_SCHEME_NORMALIZE,
TRUE, 0,
{ "data", NULL, NULL, -1, "/.//", NULL, NULL }
},
};
static void
@ -1743,6 +1755,19 @@ test_uri_join (void)
uri = g_uri_join (G_URI_FLAGS_NONE, "scheme", NULL, "foo:bar._webdav._tcp.local", -1, "", NULL, NULL);
g_assert_cmpstr (uri, ==, "scheme://foo%3Abar._webdav._tcp.local");
g_free (uri);
uri = g_uri_join (G_URI_FLAGS_NONE, "data", NULL, NULL, -1, "/.//", NULL, NULL);
g_assert_cmpstr (uri, ==, "data:/.//");
g_free (uri);
uri = g_uri_join (G_URI_FLAGS_NONE, "data", NULL, NULL, -1, ".///", NULL, NULL);
g_assert_cmpstr (uri, ==, "data:.///");
g_free (uri);
/* From https://url.spec.whatwg.org/#url-serializing */
uri = g_uri_join (G_URI_FLAGS_NONE, "web+demo", NULL, NULL, -1, "/.//not-a-host/", NULL, NULL);
g_assert_cmpstr (uri, ==, "web+demo:/.//not-a-host/");
g_free (uri);
}
static void