From c9a4a6d6f531277070830c45694361644c673b91 Mon Sep 17 00:00:00 2001
From: Michael Smith <msmith@xiph.org>
Date: Thu, 20 Nov 2008 21:05:14 +0000
Subject: [PATCH] plugins/elements/gstfilesrc.c: Use g_filename_from_uri() for
 URI parsing in filesrc rather than rolling out own sligh...

Original commit message from CVS:
* plugins/elements/gstfilesrc.c:
Use g_filename_from_uri() for URI parsing in filesrc rather than rolling
out own slightly incorrect version. Fixes use of some paths on
win32.
---
 ChangeLog                     |  7 +++++
 plugins/elements/gstfilesrc.c | 52 +++++++++++++++++------------------
 2 files changed, 32 insertions(+), 27 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 91bec7c959..369a9e59d1 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2008-11-20  Michael Smith <msmith@songbirdnest.com>
+
+	* plugins/elements/gstfilesrc.c:
+	  Use g_filename_from_uri() for URI parsing in filesrc rather than rolling
+	  out own slightly incorrect version. Fixes use of some paths on
+	  win32.
+
 2008-11-20  Michael Smith <msmith@songbirdnest.com>
 
 	* gst/gstregistrybinary.c:
diff --git a/plugins/elements/gstfilesrc.c b/plugins/elements/gstfilesrc.c
index f890fb95d4..6d63152f5f 100644
--- a/plugins/elements/gstfilesrc.c
+++ b/plugins/elements/gstfilesrc.c
@@ -1100,46 +1100,44 @@ gst_file_src_uri_get_uri (GstURIHandler * handler)
 static gboolean
 gst_file_src_uri_set_uri (GstURIHandler * handler, const gchar * uri)
 {
-  gchar *protocol, *location;
+  gchar *location, *hostname = NULL;
   gboolean ret;
   GstFileSrc *src = GST_FILE_SRC (handler);
 
-  protocol = gst_uri_get_protocol (uri);
-  if (strcmp (protocol, "file") != 0) {
-    g_free (protocol);
-    return FALSE;
-  }
-  g_free (protocol);
-
-  /* allow file://localhost/foo/bar by stripping localhost but fail
-   * for every other hostname */
-  if (g_str_has_prefix (uri, "file://localhost/")) {
-    char *tmp;
-
-    /* 16 == strlen ("file://localhost") */
-    tmp = g_strconcat ("file://", uri + 16, NULL);
-    /* we use gst_uri_get_location() although we already have the
-     * "location" with uri + 16 because it provides unescaping */
-    location = gst_uri_get_location (tmp);
-    g_free (tmp);
-  } else if (strcmp (uri, "file://") == 0) {
+  if (strcmp (uri, "file://") == 0) {
     /* Special case for "file://" as this is used by some applications
      *  to test with gst_element_make_from_uri if there's an element
      *  that supports the URI protocol. */
     gst_file_src_set_location (src, NULL);
     return TRUE;
-  } else {
-    location = gst_uri_get_location (uri);
-    GST_LOG_OBJECT (src, "Location '%s' found from uri '%s'", location, uri);
   }
 
-  if (!location)
-    return FALSE;
-  if (!g_path_is_absolute (location)) {
-    g_free (location);
+  location = g_filename_from_uri (uri, &hostname, NULL);
+
+  if (!location) {
+    GST_WARNING_OBJECT (src, "Invalid URI '%s' for filesrc", uri);
     return FALSE;
   }
 
+  if (hostname) {
+    if (!strcmp (hostname, "localhost")) {
+      /* Only 'localhost' is permitted */
+      GST_WARNING_OBJECT (src, "Invalid hostname '%s' for filesrc", hostname);
+      g_free (hostname);
+      return FALSE;
+    }
+    g_free (hostname);
+  }
+#ifdef G_OS_WIN32
+  /* Unfortunately, g_filename_from_uri() doesn't handle some UNC paths
+   * correctly on windows, it leaves them with an extra backslash
+   * at the start if they're of the mozilla-style file://///host/path/file 
+   * form. Correct this.
+   */
+  if (location[0] == '\\' && location[1] == '\\' && location[2] == '\\')
+    g_memmove (location, location + 1, strlen (location + 1) + 1);
+#endif
+
   ret = gst_file_src_set_location (src, location);
   g_free (location);