Use var_Inherit* instead of var_CreateGet*.

[vlc] / src / input / access.c

diff --git a/src/input/access.c b/src/input/access.c
index 98e30d4ccf8e59b0e8bc595ccc8de7fd835b2abc..b8a2f031e3652a1df4503113de9c17744052c3dc 100644 (file)
--- a/src/input/access.c
+++ b/src/input/access.c
@@ -27,13 +27,31 @@
 
 #include "access.h"
 #include <libvlc.h>
+#include <vlc_url.h>
+#include <vlc_modules.h>
+
+/* Decode URL (which has had its scheme stripped earlier) to a file path. */
+static char *get_path(const char *location)
+{
+    char *url, *path;
+
+    /* Prepending "file://" is a bit hackish. But then again, we do not want
+     * to hard-code the list of schemes that use file paths in make_path().
+     */
+    if (asprintf(&url, "file://%s", location) == -1)
+        return NULL;
+
+    path = make_path (url);
+    free (url);
+    return path;
+}
 
 /*****************************************************************************
  * access_New:
  *****************************************************************************/
 access_t *__access_New( vlc_object_t *p_obj, input_thread_t *p_parent_input,
                         const char *psz_access, const char *psz_demux,
-                        const char *psz_path )
+                        const char *psz_location )
 {
     access_t *p_access = vlc_custom_create( p_obj, sizeof (*p_access),
                                             VLC_OBJECT_GENERIC, "access" );
@@ -42,14 +60,20 @@ access_t *__access_New( vlc_object_t *p_obj, input_thread_t *p_parent_input,
         return NULL;
 
     /* */
-    msg_Dbg( p_obj, "creating access '%s' path='%s'",
-             psz_access, psz_path );
 
     p_access->p_input = p_parent_input;
 
-    p_access->psz_path   = strdup( psz_path );
     p_access->psz_access = strdup( psz_access );
+    p_access->psz_location = strdup( psz_location );
+    p_access->psz_filepath = get_path( psz_location );
     p_access->psz_demux  = strdup( psz_demux );
+    if( p_access->psz_access == NULL || p_access->psz_location == NULL
+     || p_access->psz_demux == NULL )
+        goto error;
+
+    msg_Dbg( p_obj, "creating access '%s' location='%s', path='%s'",
+             psz_access, psz_location,
+             p_access->psz_filepath ? p_access->psz_filepath : "(null)" );
 
     p_access->pf_read    = NULL;
     p_access->pf_block   = NULL;
@@ -63,18 +87,18 @@ access_t *__access_New( vlc_object_t *p_obj, input_thread_t *p_parent_input,
     vlc_object_attach( p_access, p_obj );
 
     p_access->p_module = module_need( p_access, "access", psz_access, true );
-
     if( p_access->p_module == NULL )
-    {
-        vlc_object_detach( p_access );
-        free( p_access->psz_access );
-        free( p_access->psz_path );
-        free( p_access->psz_demux );
-        vlc_object_release( p_access );
-        return NULL;
-    }
+        goto error;
 
     return p_access;
+
+error:
+    free( p_access->psz_access );
+    free( p_access->psz_location );
+    free( p_access->psz_filepath );
+    free( p_access->psz_demux );
+    vlc_object_release( p_access );
+    return NULL;
 }
 
 /*****************************************************************************
@@ -83,10 +107,10 @@ access_t *__access_New( vlc_object_t *p_obj, input_thread_t *p_parent_input,
 void access_Delete( access_t *p_access )
 {
     module_unneed( p_access, p_access->p_module );
-    vlc_object_detach( p_access );
 
     free( p_access->psz_access );
-    free( p_access->psz_path );
+    free( p_access->psz_location );
+    free( p_access->psz_filepath );
     free( p_access->psz_demux );
 
     vlc_object_release( p_access );