X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=modules%2Fdemux%2Fflac.c;h=5203cffdda19a9b95174ec5120d1f1ad8c2928db;hb=0c028c292be73c6eaabd6a95daa7ea3d5dac413e;hp=8ab99d8bd4c40447758ab233f499bc3a626e8602;hpb=626e56bfc8c041fcf1db5873a7a2824d6f4f688a;p=vlc diff --git a/modules/demux/flac.c b/modules/demux/flac.c index 8ab99d8bd4..5203cffdda 100644 --- a/modules/demux/flac.c +++ b/modules/demux/flac.c @@ -25,7 +25,12 @@ /***************************************************************************** * Preamble *****************************************************************************/ -#include +#ifdef HAVE_CONFIG_H +# include "config.h" +#endif + +#include +#include #include #include #include @@ -39,14 +44,14 @@ static int Open ( vlc_object_t * ); static void Close ( vlc_object_t * ); -vlc_module_begin(); - set_description( _("FLAC demuxer") ); - set_capability( "demux2", 155 ); - set_category( CAT_INPUT ); - set_subcategory( SUBCAT_INPUT_DEMUX ); - set_callbacks( Open, Close ); - add_shortcut( "flac" ); -vlc_module_end(); +vlc_module_begin () + set_description( N_("FLAC demuxer") ) + set_capability( "demux", 155 ) + set_category( CAT_INPUT ) + set_subcategory( SUBCAT_INPUT_DEMUX ) + set_callbacks( Open, Close ) + add_shortcut( "flac" ) +vlc_module_end () /***************************************************************************** * Local prototypes @@ -58,11 +63,12 @@ static int ReadMeta( demux_t *, uint8_t **pp_streaminfo, int *pi_streaminfo ); struct demux_sys_t { - vlc_bool_t b_start; + bool b_start; es_out_id_t *p_es; /* Packetizer */ decoder_t *p_packetizer; + vlc_meta_t *p_meta; audio_replay_gain_t replay_gain; @@ -93,11 +99,11 @@ struct demux_sys_t static int Open( vlc_object_t * p_this ) { demux_t *p_demux = (demux_t*)p_this; - module_t *p_id3; demux_sys_t *p_sys; - const byte_t *p_peek; + const uint8_t *p_peek; uint8_t *p_streaminfo; int i_streaminfo; + es_format_t fmt; /* Have a peep at the show. */ if( stream_Peek( p_demux->s, &p_peek, 4 ) < 4 ) return VLC_EGENERIC; @@ -114,7 +120,7 @@ static int Open( vlc_object_t * p_this ) p_demux->pf_demux = Demux; p_demux->pf_control = Control; p_demux->p_sys = p_sys = malloc( sizeof( demux_sys_t ) ); - p_sys->b_start = VLC_TRUE; + p_sys->b_start = true; p_sys->p_meta = NULL; memset( &p_sys->replay_gain, 0, sizeof(p_sys->replay_gain) ); p_sys->i_length = 0; @@ -135,56 +141,19 @@ static int Open( vlc_object_t * p_this ) } /* Load the FLAC packetizer */ - INIT_APACKETIZER( p_sys->p_packetizer, 'f', 'l', 'a', 'c' ); - /* Store STREAMINFO for the decoder and packetizer */ p_streaminfo[4] |= 0x80; /* Fake this as the last metadata block */ - p_sys->p_packetizer->fmt_in.i_extra = i_streaminfo; - p_sys->p_packetizer->fmt_in.p_extra = p_streaminfo; + es_format_Init( &fmt, AUDIO_ES, VLC_FOURCC( 'f', 'l', 'a', 'c' ) ); + fmt.i_extra = i_streaminfo; + fmt.p_extra = p_streaminfo; - p_sys->p_packetizer->p_module = - module_Need( p_sys->p_packetizer, "packetizer", NULL, 0 ); - if( !p_sys->p_packetizer->p_module ) + p_sys->p_packetizer = demux_PacketizerNew( p_demux, &fmt, "flac" ); + if( !p_sys->p_packetizer ) { - if( p_sys->p_packetizer->fmt_in.p_extra ) - free( p_sys->p_packetizer->fmt_in.p_extra ); - vlc_object_destroy( p_sys->p_packetizer ); - - msg_Err( p_demux, "cannot find flac packetizer" ); + free( p_sys ); return VLC_EGENERIC; } - /* Parse possible id3 header */ - if( !var_CreateGetBool( p_demux, "meta-preparsed" ) ) - { - p_demux->p_private = malloc( sizeof( demux_meta_t ) ); - if( !p_demux->p_private ) - return VLC_ENOMEM; - if( ( p_id3 = module_Need( p_demux, "meta reader", NULL, 0 ) ) ) - { - module_Unneed( p_demux, p_id3 ); - demux_meta_t *p_demux_meta = (demux_meta_t *)p_demux->p_private; - vlc_meta_t *p_meta = p_demux_meta->p_meta; - - if( !p_sys->p_meta ) - { - p_sys->p_meta = p_meta; - } - else if( p_meta ) - { - vlc_meta_Merge( p_sys->p_meta, p_meta ); - vlc_meta_Delete( p_meta ); - } - int i; - for( i = 0; i < p_demux_meta->i_attachments; i++ ) - TAB_APPEND_CAST( (input_attachment_t**), - p_sys->i_attachments, p_sys->attachments, - p_demux_meta->attachments[p_demux_meta->i_attachments] ); - TAB_CLEAN( p_demux_meta->i_attachments, p_demux_meta->attachments ); - } - free( p_demux->p_private ); - } - if( p_sys->i_cover_idx < p_sys->i_attachments ) { char psz_url[128]; @@ -206,7 +175,6 @@ static void Close( vlc_object_t * p_this ) demux_t *p_demux = (demux_t*)p_this; demux_sys_t *p_sys = p_demux->p_sys; - var_Destroy( p_demux, "meta-preparsed" ); TAB_CLEAN( p_sys->i_seekpoint, p_sys->seekpoint ); int i; @@ -214,14 +182,9 @@ static void Close( vlc_object_t * p_this ) free( p_sys->attachments[i] ); TAB_CLEAN( p_sys->i_attachments, p_sys->attachments); - /* Unneed module */ - module_Unneed( p_sys->p_packetizer, p_sys->p_packetizer->p_module ); - - if( p_sys->p_packetizer->fmt_in.p_extra ) - free( p_sys->p_packetizer->fmt_in.p_extra ); - /* Delete the decoder */ - vlc_object_destroy( p_sys->p_packetizer ); + demux_PacketizerDestroy( p_sys->p_packetizer ); + if( p_sys->p_meta ) vlc_meta_Delete( p_sys->p_meta ); free( p_sys ); @@ -241,7 +204,7 @@ static int Demux( demux_t *p_demux ) return 0; p_block_in->i_pts = p_block_in->i_dts = p_sys->b_start ? 1 : 0; - p_sys->b_start = VLC_FALSE; + p_sys->b_start = false; while( (p_block_out = p_sys->p_packetizer->pf_packetize( p_sys->p_packetizer, &p_block_in )) ) @@ -254,18 +217,20 @@ static int Demux( demux_t *p_demux ) if( p_sys->p_es == NULL ) { - p_sys->p_packetizer->fmt_out.b_packetized = VLC_TRUE; + p_sys->p_packetizer->fmt_out.b_packetized = true; p_sys->p_packetizer->fmt_out.audio_replay_gain = p_sys->replay_gain; p_sys->p_es = es_out_Add( p_demux->out, &p_sys->p_packetizer->fmt_out); } + p_sys->i_pts = p_block_out->i_dts; + + /* Correct timestamp */ + p_block_out->i_pts += p_sys->i_time_offset; + p_block_out->i_dts += p_sys->i_time_offset; + /* set PCR */ - if( p_block_out->i_dts >= p_sys->i_pts_start ) - es_out_Control( p_demux->out, ES_OUT_SET_PCR, p_block_out->i_dts ); - else - es_out_Control( p_demux->out, ES_OUT_RESET_PCR ); + es_out_Control( p_demux->out, ES_OUT_SET_PCR, p_block_out->i_dts ); - p_sys->i_pts = p_block_out->i_dts; es_out_Send( p_demux->out, p_sys->p_es, p_block_out ); p_block_out = p_next; @@ -312,11 +277,8 @@ static int64_t ControlGetTime( demux_t *p_demux ) static int ControlSetTime( demux_t *p_demux, int64_t i_time ) { demux_sys_t *p_sys = p_demux->p_sys; - int64_t i_next_time; - int64_t i_next_offset; int64_t i_delta_time; - int64_t i_delta_offset; - vlc_bool_t b_seekable; + bool b_seekable; int i; /* */ @@ -331,33 +293,43 @@ static int ControlSetTime( demux_t *p_demux, int64_t i_time ) if( p_sys->seekpoint[i]->i_time_offset <= i_time ) break; } - if( i+1 < p_sys->i_seekpoint ) - { - i_next_time = p_sys->seekpoint[i+1]->i_time_offset; - i_next_offset = p_sys->seekpoint[i+1]->i_byte_offset; - } - else - { - i_next_time = p_sys->i_length; - i_next_offset = stream_Size(p_demux->s)-p_sys->i_data_pos; - } i_delta_time = i_time - p_sys->seekpoint[i]->i_time_offset; - i_delta_offset = (i_next_offset - p_sys->seekpoint[i]->i_byte_offset) * i_delta_time / - (p_sys->seekpoint[i+1]->i_time_offset-p_sys->seekpoint[i]->i_time_offset); /* XXX We do exact seek if it's not too far away(45s) */ - if( i_delta_time < 45*I64C(1000000) ) + if( i_delta_time < 45*INT64_C(1000000) ) { if( stream_Seek( p_demux->s, p_sys->seekpoint[i]->i_byte_offset+p_sys->i_data_pos ) ) return VLC_EGENERIC; + p_sys->i_time_offset = p_sys->seekpoint[i]->i_time_offset - p_sys->i_pts; p_sys->i_pts_start = p_sys->i_pts+i_delta_time; - es_out_Control( p_demux->out, ES_OUT_SET_NEXT_DISPLAY_TIME, p_sys->p_es, p_sys->i_pts_start ); + es_out_Control( p_demux->out, ES_OUT_SET_NEXT_DISPLAY_TIME, p_sys->i_pts_start + p_sys->i_time_offset ); } else { + int64_t i_delta_offset; + int64_t i_next_time; + int64_t i_next_offset; + + if( i+1 < p_sys->i_seekpoint ) + { + i_next_time = p_sys->seekpoint[i+1]->i_time_offset; + i_next_offset = p_sys->seekpoint[i+1]->i_byte_offset; + } + else + { + i_next_time = p_sys->i_length; + i_next_offset = stream_Size(p_demux->s)-p_sys->i_data_pos; + } + + i_delta_offset = 0; + if( i_next_time-p_sys->seekpoint[i]->i_time_offset > 0 ) + i_delta_offset = (i_next_offset - p_sys->seekpoint[i]->i_byte_offset) * i_delta_time / + (i_next_time-p_sys->seekpoint[i]->i_time_offset); + if( stream_Seek( p_demux->s, p_sys->seekpoint[i]->i_byte_offset+p_sys->i_data_pos + i_delta_offset ) ) return VLC_EGENERIC; + p_sys->i_pts_start = p_sys->i_pts; p_sys->i_time_offset = (p_sys->seekpoint[i]->i_time_offset+i_delta_time) - p_sys->i_pts; } @@ -375,6 +347,12 @@ static int Control( demux_t *p_demux, int i_query, va_list args ) vlc_meta_Merge( p_meta, p_demux->p_sys->p_meta ); return VLC_SUCCESS; } + else if( i_query == DEMUX_HAS_UNSUPPORTED_META ) + { + bool *pb_bool = (bool*)va_arg( args, bool* ); + *pb_bool = true; + return VLC_SUCCESS; + } else if( i_query == DEMUX_GET_LENGTH ) { int64_t *pi64 = (int64_t*)va_arg( args, int64_t * ); @@ -425,7 +403,7 @@ static int Control( demux_t *p_demux, int i_query, va_list args ) return VLC_SUCCESS; } - return demux2_vaControlHelper( p_demux->s, p_sys->i_data_pos, -1, + return demux_vaControlHelper( p_demux->s, p_sys->i_data_pos, -1, 8*0, 1, i_query, args ); } @@ -442,7 +420,7 @@ static inline int Get24bBE( const uint8_t *p ) return (p[0] << 16)|(p[1] << 8)|(p[2]); } -static void ParseStreamInfo( demux_t *p_demux, int *pi_rate, int64_t *pi_count, uint8_t *p_data, int i_data ); +static void ParseStreamInfo( int *pi_rate, int64_t *pi_count, uint8_t *p_data ); static void ParseSeekTable( demux_t *p_demux, const uint8_t *p_data, int i_data, int i_sample_rate ); static void ParseComment( demux_t *, const uint8_t *p_data, int i_data ); @@ -453,7 +431,7 @@ static int ReadMeta( demux_t *p_demux, uint8_t **pp_streaminfo, int *pi_streami demux_sys_t *p_sys = p_demux->p_sys; int i_peek; const uint8_t *p_peek; - vlc_bool_t b_last; + bool b_last; int i_sample_rate; int64_t i_sample_count; seekpoint_t *s; @@ -484,9 +462,9 @@ static int ReadMeta( demux_t *p_demux, uint8_t **pp_streaminfo, int *pi_streami } /* */ - ParseStreamInfo( p_demux, &i_sample_rate, &i_sample_count, *pp_streaminfo, *pi_streaminfo ); + ParseStreamInfo( &i_sample_rate, &i_sample_count, *pp_streaminfo ); if( i_sample_rate > 0 ) - p_sys->i_length = i_sample_count * I64C(1000000)/i_sample_rate; + p_sys->i_length = i_sample_count * INT64_C(1000000)/i_sample_rate; /* Be sure we have seekpoint 0 */ s = vlc_seekpoint_New(); @@ -535,12 +513,12 @@ static int ReadMeta( demux_t *p_demux, uint8_t **pp_streaminfo, int *pi_streami return VLC_SUCCESS; } -static void ParseStreamInfo( demux_t *p_demux, int *pi_rate, int64_t *pi_count, uint8_t *p_data, int i_data ) +static void ParseStreamInfo( int *pi_rate, int64_t *pi_count, uint8_t *p_data ) { const int i_skip = 4+4; *pi_rate = GetDWBE(&p_data[i_skip+4+6]) >> 12; - *pi_count = GetQWBE(&p_data[i_skip+4+6]) & ((I64C(1)<<36)-1); + *pi_count = GetQWBE(&p_data[i_skip+4+6]) & ((INT64_C(1)<<36)-1); } static void ParseSeekTable( demux_t *p_demux, const uint8_t *p_data, int i_data, @@ -563,7 +541,7 @@ static void ParseSeekTable( demux_t *p_demux, const uint8_t *p_data, int i_data, continue; s = vlc_seekpoint_New(); - s->i_time_offset = i_sample * I64C(1000000)/i_sample_rate; + s->i_time_offset = i_sample * INT64_C(1000000)/i_sample_rate; s->i_byte_offset = GetQWBE( &p_data[4+18*i+8] ); /* Check for duplicate entry */ @@ -631,7 +609,7 @@ static void ParseComment( demux_t *p_demux, const uint8_t *p_data, int i_data ) if( n <= 0 ) continue; - psz = strndup( p_data, n ); + psz = strndup( (const char*)p_data, n ); RM(n); EnsureUTF8( psz ); @@ -643,7 +621,8 @@ static void ParseComment( demux_t *p_demux, const uint8_t *p_data, int i_data ) if( oldval ) \ { \ char * newval; \ - asprintf( &newval, "%s,%s", oldval, &psz[strlen(txt)] ); \ + if( asprintf( &newval, "%s,%s", oldval, &psz[strlen(txt)] ) == -1 ) \ + newval = NULL; \ vlc_meta_Set( p_sys->p_meta, vlc_meta_ ## var, newval ); \ free( newval ); \ } \ @@ -662,7 +641,6 @@ static void ParseComment( demux_t *p_demux, const uint8_t *p_data, int i_data ) { /* generic (PERFORMER/LICENSE/ORGANIZATION/LOCATION/CONTACT/ISRC, * undocumented tags and replay gain ) */ - audio_replay_gain_t *r = &p_sys->replay_gain; char *p = strchr( psz, '=' ); *p++ = '\0'; vlc_meta_AddExtra( p_sys->p_meta, psz, p ); @@ -702,17 +680,17 @@ static void ParsePicture( demux_t *p_demux, const uint8_t *p_data, int i_data ) i_type = GetDWBE( p_data ); RM(4); i_len = GetDWBE( p_data ); RM(4); - if( i_data < i_len + 4 ) + if( i_len < 0 || i_data < i_len + 4 ) goto error; - psz_mime = strndup( p_data, i_len ); RM(i_len); + psz_mime = strndup( (const char*)p_data, i_len ); RM(i_len); i_len = GetDWBE( p_data ); RM(4); - if( i_data < i_len + 4*4 + 4) + if( i_len < 0 || i_data < i_len + 4*4 + 4) goto error; - psz_description = strndup( p_data, i_len ); RM(i_len); + psz_description = strndup( (const char*)p_data, i_len ); RM(i_len); EnsureUTF8( psz_description ); RM(4*4); i_len = GetDWBE( p_data ); RM(4); - if( i_len > i_data ) + if( i_len < 0 || i_len > i_data ) goto error; msg_Dbg( p_demux, "FLAC: Picture type=%d mime=%s description='%s' file length=%d", @@ -728,17 +706,15 @@ static void ParsePicture( demux_t *p_demux, const uint8_t *p_data, int i_data ) p_data, i_data ); TAB_APPEND( p_sys->i_attachments, p_sys->attachments, p_attachment ); - if( i_type >= 0 && i_type < sizeof(pi_cover_score)/sizeof(pi_cover_score[0]) && + if( i_type >= 0 && (unsigned int)i_type < sizeof(pi_cover_score)/sizeof(pi_cover_score[0]) && p_sys->i_cover_score < pi_cover_score[i_type] ) { p_sys->i_cover_idx = p_sys->i_attachments-1; p_sys->i_cover_score = pi_cover_score[i_type]; } error: - if( psz_mime ) - free( psz_mime ); - if( psz_description ) - free( psz_description ); + free( psz_mime ); + free( psz_description ); } #undef RM