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 );
}
/**
if( !psz_name || strcmp( psz_name, "playlist" ) )
{
msg_Err( p_demux, "invalid root node name: %s", psz_name );
+ free( psz_name );
goto end;
}
free( psz_name );
*/
static bool parse_track_node COMPLEX_INTERFACE
{
- int i_node;
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} },
}
/* 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:
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;
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_new_input,
p_xml_reader,
p_handler->name ) )
{
p_handler = NULL;
- FREE_ATT();
}
else
{
- FREE_ATT();
return false;
}
}
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;
if( !psz_name )
{
msg_Err( p_demux, "invalid xml stream" );
- FREE_ATT();
- return false;
+ goto end;
}
/* leave if the current parent node <track> is terminated */
if( !strcmp( psz_name, psz_element ) )
{
- FREE_ATT();
+ free( psz_name );
+ free( psz_value );
/* Make sure we have a URI */
char *psz_uri = input_item_GetURI( p_new_input );
}
free( psz_uri );
- if( p_demux->p_sys->i_track_id < 0 )
+ if( p_sys->i_track_id < 0 )
{
input_item_AddSubItem( p_input_item, p_new_input );
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 */
{
msg_Err( p_demux, "there's no open element left for <%s>",
psz_name );
- FREE_ATT();
- return false;
+ goto end;
}
/* special case: location */
* Last, psz_base should default to the XSPF resource
* location if missing (not the current working directory).
* -- Courmisch */
- if( p_demux->p_sys->psz_base && !strstr( psz_value, "://" ) )
+ if( p_sys->psz_base && !strstr( psz_value, "://" ) )
{
char* psz_tmp;
- if( asprintf( &psz_tmp, "%s%s",
- p_demux->p_sys->psz_base, psz_value ) == -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_value );
input_item_CopyOptions( p_input_item, p_new_input );
- FREE_ATT();
- p_handler = NULL;
}
else
{
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;
}
/**