summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristian Kellner <gicmo@gnome.org>2011-05-08 12:17:38 +0200
committerChristian Kellner <gicmo@gnome.org>2011-05-08 12:18:12 +0200
commita2a2fe70666618062148f5cbf0ca5e7f1eedfa0a (patch)
tree183d7407c05b213bc9fa1fa88641b6889450b997
parent927d91cfcda34c88fd5137f6c118818903a1f02d (diff)
downloadgvfs-a2a2fe70666618062148f5cbf0ca5e7f1eedfa0a.tar.gz
dav: correctly handle usernames supplied in the url
When comparing URIs in dav_uri_match() do not take username or password into account because on a redirect the server is likely to not include the username in the url supplied in the Location header. Therefore redirection will fail and thus mounting. Should fix bug 586248. Patch based on a version from Henrik Holst.
-rw-r--r--daemon/gvfsbackenddav.c19
1 files changed, 9 insertions, 10 deletions
diff --git a/daemon/gvfsbackenddav.c b/daemon/gvfsbackenddav.c
index d63059fc..9e75ded2 100644
--- a/daemon/gvfsbackenddav.c
+++ b/daemon/gvfsbackenddav.c
@@ -250,16 +250,15 @@ path_equal (const char *a, const char *b, gboolean relax)
static gboolean
dav_uri_match (SoupURI *a, SoupURI *b, gboolean relax)
{
- if (a->scheme != b->scheme ||
- a->port != b->port ||
- ! str_equal (a->user, b->user, FALSE) ||
- ! str_equal (a->password, b->password, FALSE) ||
- ! str_equal (a->host, b->host, TRUE) ||
- ! path_equal (a->path, b->path, relax) ||
- ! str_equal (a->query, b->query, FALSE) ||
- ! str_equal (a->fragment, b->fragment, FALSE))
- return FALSE;
- return TRUE;
+ gboolean diff;
+
+ diff = a->scheme != b->scheme || a->port != b->port ||
+ ! str_equal (a->host, b->host, TRUE) ||
+ ! path_equal (a->path, b->path, relax) ||
+ ! str_equal (a->query, b->query, FALSE) ||
+ ! str_equal (a->fragment, b->fragment, FALSE);
+
+ return !diff;
}
static gboolean