X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=modules%2Fdemux%2Fflac.c;h=2459c9501895a536d2885edd7dc5869c7101f85b;hb=68c0144c4cfb489e2b3e53700881b06f3dbb4384;hp=10637978c47f8256cd2f2263be72b392298d47d3;hpb=5818e881a9a236842999da61aebaa36b450d350a;p=vlc diff --git a/modules/demux/flac.c b/modules/demux/flac.c index 10637978c4..2459c95018 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 @@ -40,8 +45,8 @@ static int Open ( vlc_object_t * ); static void Close ( vlc_object_t * ); vlc_module_begin(); - set_description( _("FLAC demuxer") ); - set_capability( "demux2", 155 ); + set_description( N_("FLAC demuxer") ); + set_capability( "demux", 155 ); set_category( CAT_INPUT ); set_subcategory( SUBCAT_INPUT_DEMUX ); set_callbacks( Open, Close ); @@ -58,7 +63,7 @@ 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 */ @@ -95,7 +100,7 @@ static int Open( vlc_object_t * p_this ) { demux_t *p_demux = (demux_t*)p_this; demux_sys_t *p_sys; - const byte_t *p_peek; + const uint8_t *p_peek; uint8_t *p_streaminfo; int i_streaminfo; @@ -114,7 +119,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; @@ -146,9 +151,8 @@ static int Open( vlc_object_t * p_this ) module_Need( p_sys->p_packetizer, "packetizer", NULL, 0 ); if( !p_sys->p_packetizer->p_module ) { - 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 ); + free( p_sys->p_packetizer->fmt_in.p_extra ); + vlc_object_release( p_sys->p_packetizer ); msg_Err( p_demux, "cannot find flac packetizer" ); return VLC_EGENERIC; @@ -185,11 +189,10 @@ static void Close( vlc_object_t * p_this ) /* 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 ); + free( p_sys->p_packetizer->fmt_in.p_extra ); /* Delete the decoder */ - vlc_object_destroy( p_sys->p_packetizer ); + vlc_object_release( p_sys->p_packetizer ); if( p_sys->p_meta ) vlc_meta_Delete( p_sys->p_meta ); free( p_sys ); @@ -209,7 +212,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 )) ) @@ -222,18 +225,23 @@ 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 ) + if( p_block_out->i_dts >= p_sys->i_pts_start + p_sys->i_time_offset ) 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 ); - 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; @@ -280,11 +288,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; /* */ @@ -299,33 +304,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->p_es, 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; } @@ -345,8 +360,8 @@ static int Control( demux_t *p_demux, int i_query, va_list args ) } else if( i_query == DEMUX_HAS_UNSUPPORTED_META ) { - vlc_bool_t *pb_bool = (vlc_bool_t*)va_arg( args, vlc_bool_t* ); - *pb_bool = VLC_TRUE; + bool *pb_bool = (bool*)va_arg( args, bool* ); + *pb_bool = true; return VLC_SUCCESS; } else if( i_query == DEMUX_GET_LENGTH ) @@ -399,7 +414,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 ); } @@ -416,7 +431,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 ); @@ -427,7 +442,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; @@ -458,9 +473,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(); @@ -509,12 +524,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, @@ -537,7 +552,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 */ @@ -605,7 +620,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 ); @@ -617,7 +632,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 ); \ } \ @@ -675,17 +691,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", @@ -701,17 +717,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