X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=modules%2Fdemux%2Fplaylist%2Fxspf.c;h=ed6f135f69c88b33944d2f9079e5fb614cff246e;hb=d9f4825a3fa8b2193a14d41d6f01db1eaa6f3f7a;hp=eee5b8502d923bb02297be21891e5b30a469c13a;hpb=a654d4a14edf1a3925cfa731c965652832f01ef2;p=vlc diff --git a/modules/demux/playlist/xspf.c b/modules/demux/playlist/xspf.c index eee5b8502d..ed6f135f69 100644 --- a/modules/demux/playlist/xspf.c +++ b/modules/demux/playlist/xspf.c @@ -63,15 +63,15 @@ int Import_xspf( vlc_object_t *p_this ) void Close_xspf( vlc_object_t *p_this ) { demux_t *p_demux = (demux_t *)p_this; - int i; - for(i = 0; i < p_demux->p_sys->i_tracklist_entries; i++) + demux_sys_t *p_sys = p_demux->p_sys; + for( int i = 0; i < p_sys->i_tracklist_entries; i++ ) { - if(p_demux->p_sys->pp_tracklist[i]) - vlc_gc_decref( p_demux->p_sys->pp_tracklist[i] ); + if( p_sys->pp_tracklist[i] ) + vlc_gc_decref( p_sys->pp_tracklist[i] ); } - free( p_demux->p_sys->pp_tracklist ); - free( p_demux->p_sys->psz_base ); - free( p_demux->p_sys ); + free( p_sys->pp_tracklist ); + free( p_sys->psz_base ); + free( p_sys ); } /** @@ -79,71 +79,61 @@ void Close_xspf( vlc_object_t *p_this ) */ int Demux( demux_t *p_demux ) { - int i_ret = 1; - xml_t *p_xml = NULL; + int i_ret = -1; xml_reader_t *p_xml_reader = NULL; char *psz_name = NULL; - INIT_PLAYLIST_STUFF; + input_item_t *p_current_input = GetCurrentItem(p_demux); p_demux->p_sys->pp_tracklist = NULL; p_demux->p_sys->i_tracklist_entries = 0; p_demux->p_sys->i_track_id = -1; p_demux->p_sys->psz_base = NULL; /* create new xml parser from stream */ - p_xml = xml_Create( p_demux ); - if( !p_xml ) - i_ret = -1; - else - { - p_xml_reader = xml_ReaderCreate( p_xml, p_demux->s ); - if( !p_xml_reader ) - i_ret = -1; - } + p_xml_reader = xml_ReaderCreate( p_demux, p_demux->s ); + if( !p_xml_reader ) + goto end; /* locating the root node */ - if( i_ret == 1 ) + do { - do + if( xml_ReaderRead( p_xml_reader ) != 1 ) { - if( xml_ReaderRead( p_xml_reader ) != 1 ) - { - msg_Err( p_demux, "can't read xml stream" ); - i_ret = -1; - } - } while( i_ret == VLC_SUCCESS && - xml_ReaderNodeType( p_xml_reader ) != XML_READER_STARTELEM ); - } + msg_Err( p_demux, "can't read xml stream" ); + goto end; + } + } while( xml_ReaderNodeType( p_xml_reader ) != XML_READER_STARTELEM ); + /* checking root node name */ - if( i_ret == 1 ) + psz_name = xml_ReaderName( p_xml_reader ); + if( !psz_name || strcmp( psz_name, "playlist" ) ) { - 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 = -1; - } - FREE_NAME(); + msg_Err( p_demux, "invalid root node name: %s", psz_name ); + free( psz_name ); + goto end; } + free( psz_name ); + + input_item_node_t *p_subitems = + input_item_node_Create( p_current_input ); - if( i_ret == 1 ) - i_ret = parse_playlist_node( p_demux, p_current_input, - p_xml_reader, "playlist" ) ? 0 : -1; + i_ret = parse_playlist_node( p_demux, p_subitems, + p_xml_reader, "playlist" ) ? 0 : -1; - int i; - for( i = 0 ; i < p_demux->p_sys->i_tracklist_entries ; i++ ) + for( int i = 0 ; i < p_demux->p_sys->i_tracklist_entries ; i++ ) { input_item_t *p_new_input = p_demux->p_sys->pp_tracklist[i]; if( p_new_input ) { - input_item_AddSubItem( p_current_input, p_new_input ); + input_item_node_AppendItem( p_subitems, p_new_input ); } } - HANDLE_PLAY_AND_RELEASE; + input_item_node_PostAndDelete( p_subitems ); + +end: + vlc_gc_decref(p_current_input); if( p_xml_reader ) - xml_ReaderDelete( p_xml, p_xml_reader ); - if( p_xml ) - xml_Delete( p_xml ); + xml_ReaderDelete( p_xml_reader ); return i_ret; /* Needed for correct operation of go back */ } @@ -163,10 +153,12 @@ static int Control( demux_t *p_demux, int i_query, va_list args ) */ static bool parse_playlist_node COMPLEX_INTERFACE { + input_item_t *p_input_item = p_input_node->p_item; char *psz_name = NULL; char *psz_value = NULL; bool b_version_found = false; int i_node; + bool b_ret = false; xml_elem_hnd_t *p_handler = NULL; xml_elem_hnd_t pl_elements[] = @@ -195,8 +187,7 @@ static bool parse_playlist_node COMPLEX_INTERFACE if( !psz_name || !psz_value ) { msg_Err( p_demux, "invalid xml stream @ " ); - FREE_ATT(); - return false; + goto end; } /* attribute: version */ if( !strcmp( psz_name, "version" ) ) @@ -210,19 +201,22 @@ static bool parse_playlist_node COMPLEX_INTERFACE ; else if( !strcmp( psz_name, "xml:base" ) ) { - p_demux->p_sys->psz_base = decode_URI_duplicate( psz_value ); + p_demux->p_sys->psz_base = strdup( psz_value ); } /* unknown attribute */ else msg_Warn( p_demux, "invalid attribute:\"%s\"", psz_name); - FREE_ATT(); + free( psz_name ); + free( psz_value ); } /* attribute version is mandatory !!! */ if( !b_version_found ) msg_Warn( p_demux, " requires \"version\" attribute" ); /* parse the child elements - we only take care of */ + psz_name = NULL; + psz_value = NULL; while( xml_ReaderRead( p_xml_reader ) == 1 ) { i_node = xml_ReaderNodeType( p_xml_reader ); @@ -236,8 +230,7 @@ static bool parse_playlist_node COMPLEX_INTERFACE if( !psz_name || !*psz_name ) { msg_Err( p_demux, "invalid xml stream" ); - FREE_ATT(); - return false; + goto end; } /* choose handler */ for( p_handler = pl_elements; @@ -246,24 +239,22 @@ static bool parse_playlist_node COMPLEX_INTERFACE if( !p_handler->name ) { msg_Err( p_demux, "unexpected element <%s>", psz_name ); - FREE_ATT(); - return false; + goto end; } FREE_NAME(); /* complex content is parsed in a separate function */ if( p_handler->type == COMPLEX_CONTENT ) { + FREE_VALUE(); if( p_handler->pf_handler.cmplx( p_demux, - p_input_item, + p_input_node, p_xml_reader, p_handler->name ) ) { p_handler = NULL; - FREE_ATT(); } else { - FREE_ATT(); return false; } } @@ -271,13 +262,12 @@ static bool parse_playlist_node COMPLEX_INTERFACE case XML_READER_TEXT: /* simple element content */ - FREE_ATT(); + free( psz_value ); psz_value = xml_ReaderValue( p_xml_reader ); if( !psz_value ) { msg_Err( p_demux, "invalid xml stream" ); - FREE_ATT(); - return false; + goto end; } break; @@ -287,14 +277,13 @@ static bool parse_playlist_node COMPLEX_INTERFACE if( !psz_name ) { msg_Err( p_demux, "invalid xml stream" ); - FREE_ATT(); - return false; + goto end; } /* leave if the current parent node is terminated */ if( !strcmp( psz_name, psz_element ) ) { - FREE_ATT(); - return true; + b_ret = true; + goto end; } /* there MUST have been a start tag for that element name */ if( !p_handler || !p_handler->name @@ -302,8 +291,7 @@ static bool parse_playlist_node COMPLEX_INTERFACE { msg_Err( p_demux, "there's no open element left for <%s>", psz_name ); - FREE_ATT(); - return false; + goto end; } if( p_handler->pf_handler.smpl ) @@ -318,12 +306,14 @@ static bool parse_playlist_node COMPLEX_INTERFACE default: /* unknown/unexpected xml node */ msg_Err( p_demux, "unexpected xml node %i", i_node ); - FREE_ATT(); - return false; + goto end; } - FREE_NAME(); } - return false; + +end: + free( psz_name ); + free( psz_value ); + return b_ret; } /** @@ -332,35 +322,34 @@ static bool parse_playlist_node COMPLEX_INTERFACE static bool parse_tracklist_node COMPLEX_INTERFACE { VLC_UNUSED(psz_element); - char *psz_name = NULL; - int i_node; + char *psz_name; int i_ntracks = 0; /* now parse the s */ while( xml_ReaderRead( p_xml_reader ) == 1 ) { - i_node = xml_ReaderNodeType( p_xml_reader ); + int i_node = xml_ReaderNodeType( p_xml_reader ); if( i_node == XML_READER_STARTELEM ) { - psz_name = xml_ReaderName( p_xml_reader ); - if( !psz_name ) + char *psz_eltname = xml_ReaderName( p_xml_reader ); + if( !psz_eltname ) { msg_Err( p_demux, "unexpected end of xml data" ); - FREE_NAME(); + free( psz_eltname ); return false; } - if( strcmp( psz_name, "track") ) + if( strcmp( psz_eltname, "track") ) { msg_Err( p_demux, "unexpected child of : <%s>", - psz_name ); - FREE_NAME(); + psz_eltname ); + free( psz_eltname ); return false; } - FREE_NAME(); + free( psz_eltname ); /* parse the track data in a separate function */ - if( parse_track_node( p_demux, p_input_item, - p_xml_reader,"track" ) == true ) + if( parse_track_node( p_demux, p_input_node, + p_xml_reader, "track" ) ) i_ntracks++; } else if( i_node == XML_READER_ENDELEM ) @@ -371,20 +360,18 @@ static bool parse_tracklist_node COMPLEX_INTERFACE if( xml_ReaderNodeType( p_xml_reader ) != XML_READER_ENDELEM ) { msg_Err( p_demux, "there's a missing " ); - FREE_NAME(); return false; } psz_name = xml_ReaderName( p_xml_reader ); if( !psz_name || strcmp( psz_name, "trackList" ) ) { msg_Err( p_demux, "expected: , found: ", psz_name ); - FREE_NAME(); + free( psz_name ); return false; } - FREE_NAME(); + free( psz_name ); msg_Dbg( p_demux, "parsed %i tracks successfully", i_ntracks ); - return true; } @@ -394,10 +381,12 @@ static bool parse_tracklist_node COMPLEX_INTERFACE */ static bool parse_track_node COMPLEX_INTERFACE { - int i_node; + input_item_t *p_input_item = p_input_node->p_item; char *psz_name = NULL; char *psz_value = NULL; xml_elem_hnd_t *p_handler = NULL; + demux_sys_t *p_sys = p_demux->p_sys; + bool b_ret = false; xml_elem_hnd_t track_elements[] = { {"location", SIMPLE_CONTENT, {NULL} }, @@ -417,6 +406,7 @@ static bool parse_track_node COMPLEX_INTERFACE }; input_item_t *p_new_input = input_item_New( p_demux, NULL, NULL ); + input_item_node_t *p_new_node = input_item_node_Create( p_new_input ); if( !p_new_input ) { @@ -425,11 +415,11 @@ static bool parse_track_node COMPLEX_INTERFACE } /* reset i_track_id */ - p_demux->p_sys->i_track_id = -1; + p_sys->i_track_id = -1; while( xml_ReaderRead( p_xml_reader ) == 1 ) { - i_node = xml_ReaderNodeType( p_xml_reader ); + int i_node = xml_ReaderNodeType( p_xml_reader ); switch( i_node ) { case XML_READER_NONE: @@ -441,8 +431,7 @@ static bool parse_track_node COMPLEX_INTERFACE if( !psz_name || !*psz_name ) { msg_Err( p_demux, "invalid xml stream" ); - FREE_ATT(); - return false; + goto end; } /* choose handler */ for( p_handler = track_elements; @@ -451,32 +440,24 @@ static bool parse_track_node COMPLEX_INTERFACE if( !p_handler->name ) { msg_Err( p_demux, "unexpected element <%s>", psz_name ); - FREE_ATT(); - return false; + goto end; } FREE_NAME(); /* complex content is parsed in a separate function */ if( p_handler->type == COMPLEX_CONTENT ) { - if( !p_new_input ) - { - msg_Err( p_demux, - "at <%s> level no new item has been allocated", - p_handler->name ); - FREE_ATT(); - return false; - } - if( p_handler->pf_handler.cmplx( p_demux, - p_new_input, - p_xml_reader, - p_handler->name ) ) + FREE_VALUE(); + + bool b_res = p_handler->pf_handler.cmplx( p_demux, + p_new_node, + p_xml_reader, + p_handler->name ); + if( b_res ) { p_handler = NULL; - FREE_ATT(); } else { - FREE_ATT(); return false; } } @@ -484,13 +465,12 @@ static bool parse_track_node COMPLEX_INTERFACE case XML_READER_TEXT: /* simple element content */ - FREE_ATT(); + free( psz_value ); psz_value = xml_ReaderValue( p_xml_reader ); if( !psz_value ) { msg_Err( p_demux, "invalid xml stream" ); - FREE_ATT(); - return false; + goto end; } break; @@ -500,37 +480,43 @@ static bool parse_track_node COMPLEX_INTERFACE if( !psz_name ) { msg_Err( p_demux, "invalid xml stream" ); - FREE_ATT(); - return false; + goto end; } + /* leave if the current parent node is terminated */ if( !strcmp( psz_name, psz_element ) ) { - FREE_ATT(); + free( psz_name ); + free( psz_value ); - if( p_demux->p_sys->i_track_id < 0 ) + /* Make sure we have a URI */ + char *psz_uri = input_item_GetURI( p_new_input ); + if( !psz_uri ) { - input_item_AddSubItem( p_input_item, p_new_input ); + input_item_SetURI( p_new_input, "vlc://nop" ); + } + free( psz_uri ); + + if( p_sys->i_track_id < 0 ) + { + input_item_node_AppendNode( p_input_node, p_new_node ); vlc_gc_decref( p_new_input ); return true; } - if( p_demux->p_sys->i_track_id >= - p_demux->p_sys->i_tracklist_entries ) + if( p_sys->i_track_id >= p_sys->i_tracklist_entries ) { input_item_t **pp; - pp = realloc( p_demux->p_sys->pp_tracklist, - (p_demux->p_sys->i_track_id + 1) * sizeof(*pp) ); + pp = realloc( p_sys->pp_tracklist, + (p_sys->i_track_id + 1) * sizeof(*pp) ); if( !pp ) return false; - p_demux->p_sys->pp_tracklist = pp; - while( p_demux->p_sys->i_track_id >= - p_demux->p_sys->i_tracklist_entries ) - pp[p_demux->p_sys->i_tracklist_entries++] = NULL; + p_sys->pp_tracklist = pp; + while( p_sys->i_track_id >= p_sys->i_tracklist_entries ) + pp[p_sys->i_tracklist_entries++] = NULL; } - p_demux->p_sys->pp_tracklist[ - p_demux->p_sys->i_track_id ] = p_new_input; + p_sys->pp_tracklist[ p_sys->i_track_id ] = p_new_input; return true; } /* there MUST have been a start tag for that element name */ @@ -539,51 +525,40 @@ static bool parse_track_node COMPLEX_INTERFACE { msg_Err( p_demux, "there's no open element left for <%s>", psz_name ); - FREE_ATT(); - return false; + goto end; } /* special case: location */ if( !strcmp( p_handler->name, "location" ) ) { - char *psz_location = psz_value; - if( !strncmp( psz_value, "file://", 7 ) ) - psz_location = decode_URI( psz_value + 7 ); - - if( !psz_location ) - { - FREE_ATT(); - return false; - } - - if( p_demux->p_sys->psz_base && !strstr( psz_value, "://" ) ) + if( psz_value == NULL ) + input_item_SetURI( p_new_input, "vlc://nop" ); + else + /* FIXME: This is broken. Scheme-relative (//...) locations + * and anchors (#...) are not resolved correctly. Also, + * host-relative (/...) and directory-relative locations + * ("relative path" in vernacular) should be resolved. + * Last, psz_base should default to the XSPF resource + * location if missing (not the current working directory). + * -- Courmisch */ + if( p_sys->psz_base && !strstr( psz_value, "://" ) ) { char* psz_tmp; - if( asprintf( &psz_tmp, "%s%s", p_demux->p_sys->psz_base, - psz_location ) == -1 ) + if( asprintf( &psz_tmp, "%s%s", p_sys->psz_base, + psz_value ) == -1 ) { - FREE_ATT(); - return NULL; + goto end; } input_item_SetURI( p_new_input, psz_tmp ); free( psz_tmp ); } else - input_item_SetURI( p_new_input, psz_location ); + input_item_SetURI( p_new_input, psz_value ); input_item_CopyOptions( p_input_item, p_new_input ); - FREE_ATT(); - p_handler = NULL; } else { /* there MUST be an item */ - if( !p_new_input ) - { - msg_Err( p_demux, "item not yet created at <%s>", - psz_name ); - FREE_ATT(); - return false; - } if( p_handler->pf_handler.smpl ) { p_handler->pf_handler.smpl( p_new_input, @@ -599,14 +574,15 @@ static bool parse_track_node COMPLEX_INTERFACE default: /* unknown/unexpected xml node */ msg_Err( p_demux, "unexpected xml node %i", i_node ); - FREE_ATT(); - return false; + goto end; } - FREE_NAME(); } msg_Err( p_demux, "unexpected end of xml data" ); - FREE_ATT(); - return false; + +end: + free( psz_name ); + free( psz_value ); + return b_ret; } /** @@ -618,7 +594,6 @@ static bool set_item_info SIMPLE_INTERFACE if( !psz_name || !psz_value || !p_input ) return false; - /* re-convert xml special characters inside psz_value */ resolve_xml_special_chars( psz_value ); @@ -678,6 +653,7 @@ static bool set_option SIMPLE_INTERFACE */ static bool parse_extension_node COMPLEX_INTERFACE { + input_item_t *p_input_item = p_input_node->p_item; char *psz_name = NULL; char *psz_value = NULL; char *psz_title = NULL; @@ -710,19 +686,22 @@ static bool parse_extension_node COMPLEX_INTERFACE if( !strcmp( psz_name, "title" ) ) { resolve_xml_special_chars( psz_value ); - psz_title = strdup( psz_value ); + psz_title = psz_value; } /* extension attribute: application */ else if( !strcmp( psz_name, "application" ) ) { - psz_application = strdup( psz_value ); + psz_application = psz_value; } /* unknown attribute */ else + { msg_Warn( p_demux, "invalid <%s> attribute:\"%s\"", psz_element, psz_name ); - - FREE_ATT(); + FREE_VALUE(); + } + FREE_NAME(); + psz_value = NULL; } /* attribute title is mandatory except for */ @@ -738,7 +717,8 @@ static bool parse_extension_node COMPLEX_INTERFACE ITEM_TYPE_DIRECTORY ); if( p_new_input ) { - input_item_AddSubItem( p_input_item, p_new_input ); + p_input_node = + input_item_node_AppendItem( p_input_node, p_new_input ); p_input_item = p_new_input; b_release_input_item = true; } @@ -751,15 +731,33 @@ static bool parse_extension_node COMPLEX_INTERFACE msg_Warn( p_demux, " requires \"application\" attribute" ); return false; } + /* Skip the extension if the application is not vlc + This will skip all children of the current node */ else if( strcmp( psz_application, "http://www.videolan.org/vlc/playlist/0" ) ) { msg_Dbg( p_demux, "Skipping \"%s\" extension tag", psz_application ); free( psz_application ); - return false; + /* Skip all children */ + while( xml_ReaderRead( p_xml_reader ) == 1 ) + { + if( xml_ReaderNodeType( p_xml_reader ) == XML_READER_ENDELEM ) + { + char *psz_name = xml_ReaderName( p_xml_reader ); + if( !strcmp( psz_name, "extension" ) ) + { + free( psz_name ); + break; + } + msg_Dbg( p_demux, "\tskipping \"%s\" extension child", psz_name ); + free( psz_name ); + } + } + return true; } } free( psz_application ); + /* parse the child elements */ while( xml_ReaderRead( p_xml_reader ) == 1 ) { @@ -794,7 +792,7 @@ static bool parse_extension_node COMPLEX_INTERFACE if( p_handler->type == COMPLEX_CONTENT ) { if( p_handler->pf_handler.cmplx( p_demux, - p_input_item, + p_input_node, p_xml_reader, p_handler->name ) ) { @@ -885,19 +883,18 @@ static bool parse_extitem_node COMPLEX_INTERFACE { VLC_UNUSED(psz_element); input_item_t *p_new_input = NULL; - char *psz_name = NULL; - char *psz_value = NULL; int i_tid = -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 ); + char *psz_name = xml_ReaderName( p_xml_reader ); + char *psz_value = xml_ReaderValue( p_xml_reader ); if( !psz_name || !psz_value ) { msg_Err( p_demux, "invalid xml stream @ " ); - FREE_ATT(); + free( psz_name ); + free( psz_value ); return false; } /* attribute: href */ @@ -909,7 +906,8 @@ static bool parse_extitem_node COMPLEX_INTERFACE else msg_Warn( p_demux, "invalid attribute:\"%s\"", psz_name); - FREE_ATT(); + free( psz_name ); + free( psz_value ); } /* attribute href is mandatory */ @@ -928,7 +926,7 @@ static bool parse_extitem_node COMPLEX_INTERFACE p_new_input = p_demux->p_sys->pp_tracklist[ i_tid ]; if( p_new_input ) { - input_item_AddSubItem( p_input_item, p_new_input ); + input_item_node_AppendItem( p_input_node, p_new_input ); vlc_gc_decref( p_new_input ); p_demux->p_sys->pp_tracklist[i_tid] = NULL; } @@ -945,14 +943,13 @@ static bool parse_extitem_node COMPLEX_INTERFACE */ static bool skip_element COMPLEX_INTERFACE { - VLC_UNUSED(p_demux); VLC_UNUSED(p_input_item); - char *psz_endname; + VLC_UNUSED(p_demux); VLC_UNUSED(p_input_node); while( xml_ReaderRead( p_xml_reader ) == 1 ) { if( xml_ReaderNodeType( p_xml_reader ) == XML_READER_ENDELEM ) { - psz_endname = xml_ReaderName( p_xml_reader ); + char *psz_endname = xml_ReaderName( p_xml_reader ); if( !psz_endname ) return false; if( !strcmp( psz_element, psz_endname ) )