]> git.sesse.net Git - vlc/blobdiff - modules/demux/playlist/xspf.c
Input access locking, part 3 (final).
[vlc] / modules / demux / playlist / xspf.c
index 34ae342a83379538e6e12e9ef5e091aa0a066a04..c33a4197ae40a613b36131d69f45aa8a01b29113 100644 (file)
@@ -1,5 +1,11 @@
-/******************************************************************************
- * Copyright (C) 2006 Daniel Stränger <vlc at schmaller dot de>
+/*******************************************************************************
+ * xspf.c : XSPF playlist import functions
+ *******************************************************************************
+ * Copyright (C) 2006 the VideoLAN team
+ * $Id$
+ *
+ * Authors: Daniel Stränger <vlc at schmaller dot de>
+ *          Yoann Peronneau <yoann@videolan.org>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  */
 
 #include <vlc/vlc.h>
-#include <vlc/input.h>
-#include <vlc/intf.h>
+#include <vlc_demux.h>
 
 #include "playlist.h"
 #include "vlc_xml.h"
 #include "vlc_strings.h"
+#include "vlc_url.h"
 #include "xspf.h"
 
+struct demux_sys_t
+{
+    playlist_item_t *p_item_in_category;
+    input_item_t **pp_tracklist;
+    int i_tracklist_entries;
+    int i_identifier;
+    char * psz_base;
+};
+
+static int Control( demux_t *, int, va_list );
+static int Demux( demux_t * );
+
 /**
  * \brief XSPF submodule initialization function
  */
-int E_(xspf_import_Activate)( vlc_object_t *p_this )
+int E_(Import_xspf)( vlc_object_t *p_this )
 {
-    demux_t *p_demux = (demux_t *)p_this;
-    char    *psz_ext;
-
-    psz_ext = strrchr ( p_demux->psz_path, '.' );
-
-    if( ( psz_ext && !strcasecmp( psz_ext, ".xspf") ) ||
-        ( p_demux->psz_demux && !strcmp(p_demux->psz_demux, "xspf-open") ) )
-    {
-        ;
-    }
-    else
-    {
-        return VLC_EGENERIC;
-    }
-    msg_Dbg( p_demux, "using xspf playlist import");
-
-    p_demux->pf_control = xspf_import_Control;
-    p_demux->pf_demux = xspf_import_Demux;
-
+    DEMUX_BY_EXTENSION_OR_FORCED_MSG( ".xspf", "xspf-open",
+                                      "using XSPF playlist reader" );
     return VLC_SUCCESS;
 }
 
+void E_(Close_xspf)( vlc_object_t *p_this )
+{
+    demux_t *p_demux = (demux_t *)p_this;
+    FREENULL( p_demux->p_sys->pp_tracklist );
+    FREENULL( p_demux->p_sys->psz_base );
+    free( p_demux->p_sys );
+}
+
 /**
  * \brief demuxer function for XSPF parsing
  */
-int xspf_import_Demux( demux_t *p_demux )
+int Demux( demux_t *p_demux )
 {
-    playlist_t *p_playlist = NULL;
-    playlist_item_t *p_current = NULL;
-
-    vlc_bool_t b_play;
     int i_ret = VLC_SUCCESS;
-
     xml_t *p_xml = NULL;
     xml_reader_t *p_xml_reader = NULL;
     char *psz_name = NULL;
+    INIT_PLAYLIST_STUFF;
+    p_demux->p_sys->pp_tracklist = NULL;
+    p_demux->p_sys->i_tracklist_entries = 0;
+    p_demux->p_sys->i_identifier = 0;
+    p_demux->p_sys->psz_base = NULL;
 
     /* create new xml parser from stream */
     p_xml = xml_Create( p_demux );
@@ -82,72 +91,43 @@ int xspf_import_Demux( demux_t *p_demux )
             i_ret = VLC_EGENERIC;
     }
 
-    /* start with parsing the root node */
-    if ( i_ret == VLC_SUCCESS )
-        if ( xml_ReaderRead( p_xml_reader ) != 1 )
-        {
-            msg_Err( p_demux, "can't read xml stream" );
-            i_ret = VLC_EGENERIC;
-        }
-    /* checking root nody type */
-    if ( i_ret == VLC_SUCCESS )
-        if( xml_ReaderNodeType( p_xml_reader ) != XML_READER_STARTELEM )
-        {
-            msg_Err( p_demux, "invalid root node type: %i", xml_ReaderNodeType( p_xml_reader ) );
-            i_ret = VLC_EGENERIC;
-        }
-    /* checking root node name */
-    if ( i_ret == VLC_SUCCESS )
-        psz_name = xml_ReaderName( p_xml_reader );
-    if ( !psz_name || strcmp( psz_name, "playlist" ) )
-    {
-        msg_Err( p_demux, "invalid root node name: %s", psz_name );
-        i_ret = VLC_EGENERIC;
-    }
-    FREE_NAME();
-
-    /* get the playlist ... */
-    if ( i_ret == VLC_SUCCESS )
+    /* locating the root node */
+    if( i_ret == VLC_SUCCESS )
     {
-        p_playlist = (playlist_t *) vlc_object_find( p_demux, VLC_OBJECT_PLAYLIST, FIND_PARENT );
-        if( !p_playlist )
+        do
         {
-            msg_Err( p_demux, "can't find playlist" );
-            i_ret = VLC_ENOOBJ;
-        }
+            if( xml_ReaderRead( p_xml_reader ) != 1 )
+            {
+                msg_Err( p_demux, "can't read xml stream" );
+                i_ret = VLC_EGENERIC;
+            }
+        } while( i_ret == VLC_SUCCESS &&
+                 xml_ReaderNodeType( p_xml_reader ) != XML_READER_STARTELEM );
     }
-    /* ... and its current item (to convert it to a node) */
-    if ( i_ret == VLC_SUCCESS )
+    /* checking root node name */
+    if( i_ret == VLC_SUCCESS )
     {
-        b_play = E_(FindItem)( p_demux, p_playlist, &p_current );
-        playlist_ItemToNode( p_playlist, p_current );
-        p_current->input.i_type = ITEM_TYPE_PLAYLIST;
-        /* parse the playlist node */
-        i_ret = parse_playlist_node( p_demux, p_playlist, p_current,
-                                     p_xml_reader, "playlist" );
-        /* true/false - success/egeneric mapping */
-        i_ret = ( i_ret==VLC_TRUE ? VLC_SUCCESS : VLC_EGENERIC );
-
-        if( b_play )
+        psz_name = xml_ReaderName( p_xml_reader );
+        if( !psz_name || strcmp( psz_name, "playlist" ) )
         {
-            playlist_Control( p_playlist, PLAYLIST_VIEWPLAY,
-                              p_playlist->status.i_view,
-                              p_playlist->status.p_item, NULL );
+            msg_Err( p_demux, "invalid root node name: %s", psz_name );
+            i_ret = VLC_EGENERIC;
         }
+        FREE_NAME();
     }
 
-    if ( p_playlist )
-        vlc_object_release( p_playlist );
-    if ( p_xml_reader )
+    i_ret = parse_playlist_node( p_demux, p_playlist, p_current_input,
+                                 p_xml_reader, "playlist" );
+    HANDLE_PLAY_AND_RELEASE;
+    if( p_xml_reader )
         xml_ReaderDelete( p_xml, p_xml_reader );
-    if ( p_xml )
+    if( p_xml )
         xml_Delete( p_xml );
-
-    return i_ret;
+    return -1; /* Needed for correct operation of go back */
 }
 
 /** \brief dummy function for demux callback interface */
-int xspf_import_Control( demux_t *p_demux, int i_query, va_list args )
+static int Control( demux_t *p_demux, int i_query, va_list args )
 {
     return VLC_EGENERIC;
 }
@@ -156,7 +136,7 @@ int xspf_import_Control( demux_t *p_demux, int i_query, va_list args )
  * \brief parse the root node of a XSPF playlist
  * \param p_demux demuxer instance
  * \param p_playlist playlist instance
- * \param p_item current playlist node
+ * \param p_input_item current input item
  * \param p_xml_reader xml reader instance
  * \param psz_element name of element to parse
  */
@@ -171,7 +151,7 @@ static vlc_bool_t parse_playlist_node COMPLEX_INTERFACE
     xml_elem_hnd_t pl_elements[] =
         { {"title",        SIMPLE_CONTENT,  {.smpl = set_item_info} },
           {"creator",      SIMPLE_CONTENT,  {.smpl = set_item_info} },
-          {"annotation",   SIMPLE_CONTENT,  {NULL} },
+          {"annotation",   SIMPLE_CONTENT,  {.smpl = set_item_info} },
           {"info",         SIMPLE_CONTENT,  {NULL} },
           {"location",     SIMPLE_CONTENT,  {NULL} },
           {"identifier",   SIMPLE_CONTENT,  {NULL} },
@@ -181,32 +161,36 @@ static vlc_bool_t parse_playlist_node COMPLEX_INTERFACE
           {"attribution",  COMPLEX_CONTENT, {.cmplx = skip_element} },
           {"link",         SIMPLE_CONTENT,  {NULL} },
           {"meta",         SIMPLE_CONTENT,  {NULL} },
-          {"extension",    COMPLEX_CONTENT, {.cmplx = skip_element} },
+          {"extension",    COMPLEX_CONTENT, {.cmplx = parse_extension_node} },
           {"trackList",    COMPLEX_CONTENT, {.cmplx = parse_tracklist_node} },
           {NULL,           UNKNOWN_CONTENT, {NULL} }
         };
 
     /* read all playlist attributes */
-    while ( xml_ReaderNextAttr( p_xml_reader ) == VLC_SUCCESS )
+    while( xml_ReaderNextAttr( p_xml_reader ) == VLC_SUCCESS )
     {
-        psz_name = xml_ReaderName ( p_xml_reader );
-        psz_value = xml_ReaderValue ( p_xml_reader );
-        if ( !psz_name || !psz_value )
+        psz_name = xml_ReaderName( p_xml_reader );
+        psz_value = xml_ReaderValue( p_xml_reader );
+        if( !psz_name || !psz_value )
         {
             msg_Err( p_demux, "invalid xml stream @ <playlist>" );
             FREE_ATT();
             return VLC_FALSE;
         }
         /* attribute: version */
-        if ( !strcmp( psz_name, "version" ) )
+        if( !strcmp( psz_name, "version" ) )
         {
             b_version_found = VLC_TRUE;
-            if ( strcmp( psz_value, "0" ) && strcmp( psz_value, "1" ) )
+            if( strcmp( psz_value, "0" ) && strcmp( psz_value, "1" ) )
                 msg_Warn( p_demux, "unsupported XSPF version" );
         }
         /* attribute: xmlns */
-        else if ( !strcmp ( psz_name, "xmlns" ) )
+        else if( !strcmp( psz_name, "xmlns" ) )
             ;
+        else if( !strcmp( psz_name, "xml:base" ) )
+        {
+            p_demux->p_sys->psz_base = decode_URI_duplicate( psz_value );
+        }
         /* unknown attribute */
         else
             msg_Warn( p_demux, "invalid <playlist> attribute:\"%s\"", psz_name);
@@ -214,21 +198,21 @@ static vlc_bool_t parse_playlist_node COMPLEX_INTERFACE
         FREE_ATT();
     }
     /* attribute version is mandatory !!! */
-    if ( !b_version_found )
+    if( !b_version_found )
         msg_Warn( p_demux, "<playlist> requires \"version\" attribute" );
 
     /* parse the child elements - we only take care of <trackList> */
-    while ( xml_ReaderRead( p_xml_reader ) == 1 )
+    while( xml_ReaderRead( p_xml_reader ) == 1 )
     {
         i_node = xml_ReaderNodeType( p_xml_reader );
-        switch ( i_node )
+        switch( i_node )
         {
             case XML_READER_NONE:
                 break;
             case XML_READER_STARTELEM:
                 /*  element start tag  */
                 psz_name = xml_ReaderName( p_xml_reader );
-                if ( !psz_name || !*psz_name )
+                if( !psz_name || !*psz_name )
                 {
                     msg_Err( p_demux, "invalid xml stream" );
                     FREE_ATT();
@@ -238,7 +222,7 @@ static vlc_bool_t parse_playlist_node COMPLEX_INTERFACE
                 for( p_handler = pl_elements;
                      p_handler->name && strcmp( psz_name, p_handler->name );
                      p_handler++ );
-                if ( !p_handler->name )
+                if( !p_handler->name )
                 {
                     msg_Err( p_demux, "unexpected element <%s>", psz_name );
                     FREE_ATT();
@@ -246,13 +230,13 @@ static vlc_bool_t parse_playlist_node COMPLEX_INTERFACE
                 }
                 FREE_NAME();
                 /* complex content is parsed in a separate function */
-                if ( p_handler->type == COMPLEX_CONTENT )
+                if( p_handler->type == COMPLEX_CONTENT )
                 {
-                    if ( p_handler->pf_handler.cmplx( p_demux,
-                                                      p_playlist,
-                                                      p_item,
-                                                      p_xml_reader,
-                                                      p_handler->name ) )
+                    if( p_handler->pf_handler.cmplx( p_demux,
+                                                     p_playlist,
+                                                     p_input_item,
+                                                     p_xml_reader,
+                                                     p_handler->name ) )
                     {
                         p_handler = NULL;
                         FREE_ATT();
@@ -269,7 +253,7 @@ static vlc_bool_t parse_playlist_node COMPLEX_INTERFACE
                 /* simple element content */
                 FREE_ATT();
                 psz_value = xml_ReaderValue( p_xml_reader );
-                if ( !psz_value )
+                if( !psz_value )
                 {
                     msg_Err( p_demux, "invalid xml stream" );
                     FREE_ATT();
@@ -280,21 +264,21 @@ static vlc_bool_t parse_playlist_node COMPLEX_INTERFACE
             case XML_READER_ENDELEM:
                 /* element end tag */
                 psz_name = xml_ReaderName( p_xml_reader );
-                if ( !psz_name )
+                if( !psz_name )
                 {
                     msg_Err( p_demux, "invalid xml stream" );
                     FREE_ATT();
                     return VLC_FALSE;
                 }
                 /* leave if the current parent node <playlist> is terminated */
-                if ( !strcmp( psz_name, psz_element ) )
+                if( !strcmp( psz_name, psz_element ) )
                 {
                     FREE_ATT();
                     return VLC_TRUE;
                 }
                 /* there MUST have been a start tag for that element name */
-                if ( !p_handler || !p_handler->name
-                     || strcmp( p_handler->name, psz_name ))
+                if( !p_handler || !p_handler->name
+                    || strcmp( p_handler->name, psz_name ))
                 {
                     msg_Err( p_demux, "there's no open element left for <%s>",
                              psz_name );
@@ -302,9 +286,9 @@ static vlc_bool_t parse_playlist_node COMPLEX_INTERFACE
                     return VLC_FALSE;
                 }
 
-                if ( p_handler->pf_handler.smpl )
+                if( p_handler->pf_handler.smpl )
                 {
-                    p_handler->pf_handler.smpl( p_item, p_handler->name,
+                    p_handler->pf_handler.smpl( p_input_item, p_handler->name,
                                                 psz_value );
                 }
                 FREE_ATT();
@@ -332,19 +316,19 @@ static vlc_bool_t parse_tracklist_node COMPLEX_INTERFACE
     int i_ntracks = 0;
 
     /* now parse the <track>s */
-    while ( xml_ReaderRead( p_xml_reader ) == 1 )
+    while( xml_ReaderRead( p_xml_reader ) == 1 )
     {
         i_node = xml_ReaderNodeType( p_xml_reader );
-        if ( i_node == XML_READER_STARTELEM )
+        if( i_node == XML_READER_STARTELEM )
         {
             psz_name = xml_ReaderName( p_xml_reader );
-            if ( !psz_name )
+            if( !psz_name )
             {
                 msg_Err( p_demux, "unexpected end of xml data" );
                 FREE_NAME();
                 return VLC_FALSE;
             }
-            if ( strcmp( psz_name, "track") )
+            if( strcmp( psz_name, "track") )
             {
                 msg_Err( p_demux, "unexpected child of <trackList>: <%s>",
                          psz_name );
@@ -354,23 +338,23 @@ static vlc_bool_t parse_tracklist_node COMPLEX_INTERFACE
             FREE_NAME();
 
             /* parse the track data in a separate function */
-            if ( parse_track_node( p_demux, p_playlist, p_item, p_xml_reader,
-                                   "track" ) == VLC_TRUE )
+            if( parse_track_node( p_demux, p_playlist, p_input_item,
+                                   p_xml_reader,"track" ) == VLC_TRUE )
                 i_ntracks++;
         }
-        else if ( i_node == XML_READER_ENDELEM )
+        else if( i_node == XML_READER_ENDELEM )
             break;
     }
 
     /* the <trackList> has to be terminated */
-    if ( xml_ReaderNodeType( p_xml_reader ) != XML_READER_ENDELEM )
+    if( xml_ReaderNodeType( p_xml_reader ) != XML_READER_ENDELEM )
     {
         msg_Err( p_demux, "there's a missing </trackList>" );
         FREE_NAME();
         return VLC_FALSE;
     }
     psz_name = xml_ReaderName( p_xml_reader );
-    if ( !psz_name || strcmp( psz_name, "trackList" ) )
+    if( !psz_name || strcmp( psz_name, "trackList" ) )
     {
         msg_Err( p_demux, "expected: </trackList>, found: </%s>", psz_name );
         FREE_NAME();
@@ -389,7 +373,7 @@ static vlc_bool_t parse_tracklist_node COMPLEX_INTERFACE
  */
 static vlc_bool_t parse_track_node COMPLEX_INTERFACE
 {
-    playlist_item_t *p_new=NULL;
+    input_item_t *p_new_input = NULL;
     int i_node;
     char *psz_name=NULL;
     char *psz_value=NULL;
@@ -400,7 +384,7 @@ static vlc_bool_t parse_track_node COMPLEX_INTERFACE
           {"identifier",   SIMPLE_CONTENT,  {NULL} },
           {"title",        SIMPLE_CONTENT,  {.smpl = set_item_info} },
           {"creator",      SIMPLE_CONTENT,  {.smpl = set_item_info} },
-          {"annotation",   SIMPLE_CONTENT,  {NULL} },
+          {"annotation",   SIMPLE_CONTENT,  {.smpl = set_item_info} },
           {"info",         SIMPLE_CONTENT,  {NULL} },
           {"image",        SIMPLE_CONTENT,  {NULL} },
           {"album",        SIMPLE_CONTENT,  {.smpl = set_item_info} },
@@ -412,10 +396,10 @@ static vlc_bool_t parse_track_node COMPLEX_INTERFACE
           {NULL,           UNKNOWN_CONTENT, {NULL} }
         };
 
-    while ( xml_ReaderRead( p_xml_reader ) == 1 )
+    while( xml_ReaderRead( p_xml_reader ) == 1 )
     {
         i_node = xml_ReaderNodeType( p_xml_reader );
-        switch ( i_node )
+        switch( i_node )
         {
             case XML_READER_NONE:
                 break;
@@ -423,7 +407,7 @@ static vlc_bool_t parse_track_node COMPLEX_INTERFACE
             case XML_READER_STARTELEM:
                 /*  element start tag  */
                 psz_name = xml_ReaderName( p_xml_reader );
-                if ( !psz_name || !*psz_name )
+                if( !psz_name || !*psz_name )
                 {
                     msg_Err( p_demux, "invalid xml stream" );
                     FREE_ATT();
@@ -433,7 +417,7 @@ static vlc_bool_t parse_track_node COMPLEX_INTERFACE
                 for( p_handler = track_elements;
                      p_handler->name && strcmp( psz_name, p_handler->name );
                      p_handler++ );
-                if ( !p_handler->name )
+                if( !p_handler->name )
                 {
                     msg_Err( p_demux, "unexpected element <%s>", psz_name );
                     FREE_ATT();
@@ -441,9 +425,9 @@ static vlc_bool_t parse_track_node COMPLEX_INTERFACE
                 }
                 FREE_NAME();
                 /* complex content is parsed in a separate function */
-                if ( p_handler->type == COMPLEX_CONTENT )
+                if( p_handler->type == COMPLEX_CONTENT )
                 {
-                    if ( !p_new )
+                    if( !p_new_input )
                     {
                         msg_Err( p_demux,
                                  "at <%s> level no new item has been allocated",
@@ -451,11 +435,11 @@ static vlc_bool_t parse_track_node COMPLEX_INTERFACE
                         FREE_ATT();
                         return VLC_FALSE;
                     }
-                    if ( p_handler->pf_handler.cmplx( p_demux,
-                                                      p_playlist,
-                                                      p_new,
-                                                      p_xml_reader,
-                                                      p_handler->name ) )
+                    if( p_handler->pf_handler.cmplx( p_demux,
+                                                     p_playlist,
+                                                     p_new_input,
+                                                     p_xml_reader,
+                                                     p_handler->name ) )
                     {
                         p_handler = NULL;
                         FREE_ATT();
@@ -472,7 +456,7 @@ static vlc_bool_t parse_track_node COMPLEX_INTERFACE
                 /* simple element content */
                 FREE_ATT();
                 psz_value = xml_ReaderValue( p_xml_reader );
-                if ( !psz_value )
+                if( !psz_value )
                 {
                     msg_Err( p_demux, "invalid xml stream" );
                     FREE_ATT();
@@ -483,21 +467,41 @@ static vlc_bool_t parse_track_node COMPLEX_INTERFACE
             case XML_READER_ENDELEM:
                 /* element end tag */
                 psz_name = xml_ReaderName( p_xml_reader );
-                if ( !psz_name )
+                if( !psz_name )
                 {
                     msg_Err( p_demux, "invalid xml stream" );
                     FREE_ATT();
                     return VLC_FALSE;
                 }
                 /* leave if the current parent node <track> is terminated */
-                if ( !strcmp( psz_name, psz_element ) )
+                if( !strcmp( psz_name, psz_element ) )
                 {
                     FREE_ATT();
+                    input_ItemAddSubItem( p_input_item, p_new_input );
+                    if( p_demux->p_sys->i_identifier <
+                        p_demux->p_sys->i_tracklist_entries )
+                    {
+                        p_demux->p_sys->pp_tracklist[
+                            p_demux->p_sys->i_identifier ] = p_new_input;
+                    }
+                    else
+                    {
+                        if( p_demux->p_sys->i_identifier >
+                            p_demux->p_sys->i_tracklist_entries )
+                        {
+                            p_demux->p_sys->i_tracklist_entries =
+                                p_demux->p_sys->i_identifier;
+                        }
+                        INSERT_ELEM( p_demux->p_sys->pp_tracklist,
+                                     p_demux->p_sys->i_tracklist_entries,
+                                     p_demux->p_sys->i_tracklist_entries,
+                                     p_new_input );
+                    }
                     return VLC_TRUE;
                 }
                 /* there MUST have been a start tag for that element name */
-                if ( !p_handler || !p_handler->name
-                     || strcmp( p_handler->name, psz_name ))
+                if( !p_handler || !p_handler->name
+                    || strcmp( p_handler->name, psz_name ))
                 {
                     msg_Err( p_demux, "there's no open element left for <%s>",
                              psz_name );
@@ -506,21 +510,42 @@ static vlc_bool_t parse_track_node COMPLEX_INTERFACE
                 }
 
                 /* special case: location */
-                if ( !strcmp( p_handler->name, "location" ) )
+                if( !strcmp( p_handler->name, "location" ) )
                 {
+                    char *psz_uri=NULL;
                     /* there MUST NOT be an item */
-                    if ( p_new )
+                    if( p_new_input )
                     {
-                        msg_Err( p_demux,
-                                 "a new item has just been created <%s>",
+                        msg_Err( p_demux, "item <%s> already created",
                                  psz_name );
                         FREE_ATT();
                         return VLC_FALSE;
                     }
-                    /* create it now */
-                    if ( insert_new_item( p_playlist, p_item,
-                                          &p_new, psz_value ) )
+                    psz_uri = decode_URI_duplicate( psz_value );
+
+                    if( psz_uri )
                     {
+                        if( p_demux->p_sys->psz_base &&
+                            !strstr( psz_uri, "://" ) )
+                        {
+                           char* psz_tmp = malloc(
+                                   strlen(p_demux->p_sys->psz_base) +
+                                   strlen(psz_uri) +1 );
+                           if( !psz_tmp )
+                           {
+                               msg_Err( p_demux, "out of memory");
+                               return VLC_FALSE;
+                           }
+                           sprintf( psz_tmp, "%s%s",
+                                    p_demux->p_sys->psz_base, psz_uri );
+                           free( psz_uri );
+                           psz_uri = psz_tmp;
+                        }
+                        p_new_input = input_ItemNewExt( p_playlist, psz_uri,
+                                                        NULL, 0, NULL, -1 );
+                        free( psz_uri );
+                        input_ItemCopyOptions( p_input_item, p_new_input );
+                        psz_uri = NULL;
                         FREE_ATT();
                         p_handler = NULL;
                     }
@@ -530,20 +555,24 @@ static vlc_bool_t parse_track_node COMPLEX_INTERFACE
                         return VLC_FALSE;
                     }
                 }
+                else if( !strcmp( p_handler->name, "identifier" ) )
+                {
+                    p_demux->p_sys->i_identifier = atoi( psz_value );
+                }
                 else
                 {
                     /* there MUST be an item */
-                    if ( !p_new )
+                    if( !p_new_input )
                     {
-                        msg_Err( p_demux,
-                                 "an item hasn't been created yet <%s>",
+                        msg_Err( p_demux, "item not yet created at <%s>",
                                  psz_name );
                         FREE_ATT();
                         return VLC_FALSE;
                     }
-                    if ( p_handler->pf_handler.smpl )
+                    if( p_handler->pf_handler.smpl )
                     {
-                        p_handler->pf_handler.smpl( p_new, p_handler->name,
+                        p_handler->pf_handler.smpl( p_new_input,
+                                                    p_handler->name,
                                                     psz_value );
                         FREE_ATT();
                     }
@@ -571,58 +600,241 @@ static vlc_bool_t parse_track_node COMPLEX_INTERFACE
 static vlc_bool_t set_item_info SIMPLE_INTERFACE
 {
     /* exit if setting is impossible */
-    if ( !psz_name || !psz_value || !p_item )
+    if( !psz_name || !psz_value || !p_input )
         return VLC_FALSE;
 
+
     /* re-convert xml special characters inside psz_value */
-    resolve_xml_special_chars ( psz_value );
+    resolve_xml_special_chars( psz_value );
 
     /* handle each info element in a separate "if" clause */
-    if ( !strcmp( psz_name, "title" ) )
+    if( !strcmp( psz_name, "title" ) )
     {
-        if ( playlist_ItemSetName ( p_item, (char *)psz_value ) == VLC_SUCCESS )
-            return VLC_TRUE;
-        return VLC_FALSE;
+        p_input->psz_name = strdup( (char*)psz_value );
     }
-    else if ( !strcmp( psz_name, "creator" ) )
+    else if( !strcmp( psz_name, "creator" ) )
     {
-        if ( vlc_input_item_AddInfo( &(p_item->input),
-                                     _(VLC_META_INFO_CAT), _(VLC_META_ARTIST),
-                                     "%s", psz_value ) == VLC_SUCCESS )
-            return VLC_TRUE;
-        return VLC_FALSE;
-
+        input_item_SetArtist( p_input, psz_value );
     }
-    else if ( !strcmp( psz_name, "album" ) )
+    else if( !strcmp( psz_name, "album" ) )
     {
-        if ( vlc_input_item_AddInfo( &(p_item->input),
-                                     _(VLC_META_INFO_CAT),
-                                     _(VLC_META_COLLECTION),
-                                     "%s", psz_value ) == VLC_SUCCESS )
-            return VLC_TRUE;
-        return VLC_FALSE;
+        input_item_SetAlbum( p_input, psz_value );
 
-    } else if ( !strcmp( psz_name, "trackNum" ) )
+    }
+    else if( !strcmp( psz_name, "trackNum" ) )
+    {
+        input_item_SetTrackNum( p_input, psz_value );
+    }
+    else if( !strcmp( psz_name, "duration" ) )
     {
         long i_num = atol( psz_value );
-        if ( i_num > 0
-             && vlc_input_item_AddInfo( &(p_item->input),
-                                         _(VLC_META_INFO_CAT),
-                                         _(VLC_META_SEQ_NUM),
-                                         "%s", psz_value ) == VLC_SUCCESS )
-                return VLC_TRUE;
-        return VLC_FALSE;
+        input_item_SetDuration( p_input, (mtime_t) i_num*1000 );
+    }
+    else if( !strcmp( psz_name, "annotation" ) )
+    {
+        input_item_SetDescription( p_input, psz_value );
+    }
+    return VLC_TRUE;
+}
+
 
-    } else if ( !strcmp( psz_name, "duration" ) )
+/**
+ * \brief parse the extension node of a XSPF playlist
+ */
+static vlc_bool_t parse_extension_node COMPLEX_INTERFACE
+{
+    char *psz_name = NULL;
+    char *psz_value = NULL;
+    char *psz_title = NULL;
+    int i_node;
+    xml_elem_hnd_t *p_handler = NULL;
+
+    xml_elem_hnd_t pl_elements[] =
+        { {"node",  COMPLEX_CONTENT, {.cmplx = parse_extension_node} },
+          {"item",  COMPLEX_CONTENT, {.cmplx = parse_extitem_node} },
+          {NULL,    UNKNOWN_CONTENT, {NULL} }
+        };
+
+    /* read all extension node attributes */
+    while( xml_ReaderNextAttr( p_xml_reader ) == VLC_SUCCESS )
     {
-        long i_num = atol( psz_value );
-        if ( i_num > 0
-             && playlist_ItemSetDuration( p_item, i_num*1000 ) == VLC_SUCCESS )
-                return VLC_TRUE;
+        psz_name = xml_ReaderName( p_xml_reader );
+        psz_value = xml_ReaderValue( p_xml_reader );
+        if( !psz_name || !psz_value )
+        {
+            msg_Err( p_demux, "invalid xml stream @ <node>" );
+            FREE_ATT();
+            return VLC_FALSE;
+        }
+        /* attribute: title */
+        if( !strcmp( psz_name, "title" ) )
+        {
+            psz_title = unescape_URI_duplicate( psz_value );
+        }
+        /* unknown attribute */
+        else
+            msg_Warn( p_demux, "invalid <node> attribute:\"%s\"", psz_name);
+
+        FREE_ATT();
+    }
+
+    /* attribute title is mandatory except for <extension> */
+    if( !strcmp( psz_element, "node" ) && !psz_title )
+    {
+        msg_Warn( p_demux, "<node> requires \"title\" attribute" );
         return VLC_FALSE;
+    }
+    if( psz_title ) free( psz_title );
 
+    /* parse the child elements */
+    while( xml_ReaderRead( p_xml_reader ) == 1 )
+    {
+        i_node = xml_ReaderNodeType( p_xml_reader );
+        switch( i_node )
+        {
+            case XML_READER_NONE:
+                break;
+            case XML_READER_STARTELEM:
+                /*  element start tag  */
+                psz_name = xml_ReaderName( p_xml_reader );
+                if( !psz_name || !*psz_name )
+                {
+                    msg_Err( p_demux, "invalid xml stream" );
+                    FREE_ATT();
+                    return VLC_FALSE;
+                }
+                /* choose handler */
+                for( p_handler = pl_elements;
+                     p_handler->name && strcmp( psz_name, p_handler->name );
+                     p_handler++ );
+                if( !p_handler->name )
+                {
+                    msg_Err( p_demux, "unexpected element <%s>", psz_name );
+                    FREE_ATT();
+                    return VLC_FALSE;
+                }
+                FREE_NAME();
+                /* complex content is parsed in a separate function */
+                if( p_handler->type == COMPLEX_CONTENT )
+                {
+                    if( p_handler->pf_handler.cmplx( p_demux,
+                                                     p_playlist,
+                                                     p_input_item,
+                                                     p_xml_reader,
+                                                     p_handler->name ) )
+                    {
+                        p_handler = NULL;
+                        FREE_ATT();
+                    }
+                    else
+                    {
+                        FREE_ATT();
+                        return VLC_FALSE;
+                    }
+                }
+                break;
+
+            case XML_READER_TEXT:
+                /* simple element content */
+                FREE_ATT();
+                psz_value = xml_ReaderValue( p_xml_reader );
+                if( !psz_value )
+                {
+                    msg_Err( p_demux, "invalid xml stream" );
+                    FREE_ATT();
+                    return VLC_FALSE;
+                }
+                break;
+
+            case XML_READER_ENDELEM:
+                /* element end tag */
+                psz_name = xml_ReaderName( p_xml_reader );
+                if( !psz_name )
+                {
+                    msg_Err( p_demux, "invalid xml stream" );
+                    FREE_ATT();
+                    return VLC_FALSE;
+                }
+                /* leave if the current parent node is terminated */
+                if( !strcmp( psz_name, psz_element ) )
+                {
+                    FREE_ATT();
+                    return VLC_TRUE;
+                }
+                /* there MUST have been a start tag for that element name */
+                if( !p_handler || !p_handler->name
+                    || strcmp( p_handler->name, psz_name ))
+                {
+                    msg_Err( p_demux, "there's no open element left for <%s>",
+                             psz_name );
+                    FREE_ATT();
+                    return VLC_FALSE;
+                }
+
+                if( p_handler->pf_handler.smpl )
+                {
+                    p_handler->pf_handler.smpl( p_input_item, p_handler->name,
+                                                psz_value );
+                }
+                FREE_ATT();
+                p_handler = NULL;
+                break;
+
+            default:
+                /* unknown/unexpected xml node */
+                msg_Err( p_demux, "unexpected xml node %i", i_node );
+                FREE_ATT();
+                return VLC_FALSE;
+        }
+        FREE_NAME();
     }
+    return VLC_FALSE;
+}
 
+/**
+ * \brief parse the extension item node of a XSPF playlist
+ */
+static vlc_bool_t parse_extitem_node COMPLEX_INTERFACE
+{
+    char *psz_name = NULL;
+    char *psz_value = NULL;
+    int i_href = -1;
+
+    /* read all extension item attributes */
+    while( xml_ReaderNextAttr( p_xml_reader ) == VLC_SUCCESS )
+    {
+        psz_name = xml_ReaderName( p_xml_reader );
+        psz_value = xml_ReaderValue( p_xml_reader );
+        if( !psz_name || !psz_value )
+        {
+            msg_Err( p_demux, "invalid xml stream @ <item>" );
+            FREE_ATT();
+            return VLC_FALSE;
+        }
+        /* attribute: href */
+        if( !strcmp( psz_name, "href" ) )
+        {
+            i_href = atoi( psz_value );
+        }
+        /* unknown attribute */
+        else
+            msg_Warn( p_demux, "invalid <item> attribute:\"%s\"", psz_name);
+
+        FREE_ATT();
+    }
+
+    /* attribute href is mandatory */
+    if( i_href < 0 )
+    {
+        msg_Warn( p_demux, "<item> requires \"href\" attribute" );
+        return VLC_FALSE;
+    }
+
+    if( i_href > p_demux->p_sys->i_tracklist_entries )
+    {
+        msg_Warn( p_demux, "invalid \"href\" attribute" );
+        return VLC_FALSE;
+    }
     return VLC_TRUE;
 }
 
@@ -633,14 +845,14 @@ static vlc_bool_t skip_element COMPLEX_INTERFACE
 {
     char *psz_endname;
 
-    while ( xml_ReaderRead( p_xml_reader ) == 1 )
+    while( xml_ReaderRead( p_xml_reader ) == 1 )
     {
-        if ( xml_ReaderNodeType( p_xml_reader ) == XML_READER_ENDELEM )
+        if( xml_ReaderNodeType( p_xml_reader ) == XML_READER_ENDELEM )
         {
             psz_endname = xml_ReaderName( p_xml_reader );
-            if ( !psz_endname )
+            if( !psz_endname )
                 return VLC_FALSE;
-            if ( !strcmp( psz_element, psz_endname ) )
+            if( !strcmp( psz_element, psz_endname ) )
             {
                 free( psz_endname );
                 return VLC_TRUE;
@@ -651,32 +863,3 @@ static vlc_bool_t skip_element COMPLEX_INTERFACE
     }
     return VLC_FALSE;
 }
-
-/**
- * \brief creates a new playlist item from the given mrl
- */
-static vlc_bool_t insert_new_item( playlist_t *p_pl, playlist_item_t *p_cur,
-                                   playlist_item_t **pp_new, char *psz_location )
-{
-    char *psz_uri=NULL;
-    psz_uri = unescape_URI_duplicate( psz_location );
-
-    if ( psz_uri )
-    {
-        *pp_new = playlist_ItemNew( p_pl, psz_uri, NULL );
-        free( psz_uri );
-        psz_uri = NULL;
-    }
-
-    if ( !*pp_new )
-        return VLC_FALSE;
-
-    playlist_NodeAddItem( p_pl,  *pp_new,         p_cur->pp_parents[0]->i_view,
-                          p_cur, PLAYLIST_APPEND, PLAYLIST_END );
-
-    playlist_CopyParents( p_cur, *pp_new );
-
-    vlc_input_item_CopyOptions( &p_cur->input, &((*pp_new)->input) );
-
-    return VLC_TRUE;
-}