X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=modules%2Faccess%2Fcdda%2Faccess.c;h=d5e3d819befc366b35543dd8a895a8d237152e4f;hb=313c62d2df64a4a6b24023c15e8140b7ce54d3eb;hp=215863b9d02fbb5b24fe2252e83d477888459aec;hpb=c3dd5600a873b420b4c6613c89b1a85c39ac16cf;p=vlc diff --git a/modules/access/cdda/access.c b/modules/access/cdda/access.c index 215863b9d0..d5e3d819be 100644 --- a/modules/access/cdda/access.c +++ b/modules/access/cdda/access.c @@ -30,7 +30,6 @@ #include "cdda.h" /* private structures. Also #includes vlc things */ #include "info.h" /* headers for meta info retrieval */ #include "access.h" -#include /* Has to come *after* cdda.h */ #include "vlc_keys.h" #include @@ -55,7 +54,7 @@ access_t *p_cdda_input = NULL; /***************************************************************************** * Local prototypes *****************************************************************************/ -static int CDDARead( access_t *, uint8_t *, int ); +static ssize_t CDDARead( access_t *, uint8_t *, size_t ); static block_t *CDDAReadBlocks( access_t * p_access ); static int CDDASeek( access_t * p_access, int64_t i_pos ); static int CDDAControl( access_t *p_access, int i_query, @@ -228,7 +227,7 @@ static block_t * CDDAReadBlocks( access_t * p_access ) /* Return only the dummy RIFF header we created in Open/Init */ p_block = block_New( p_access, sizeof( WAVEHEADER ) ); memcpy( p_block->p_buffer, &p_cdda->waveheader, sizeof(WAVEHEADER) ); - p_cdda->b_header = VLC_TRUE; + p_cdda->b_header = true; return p_block; } @@ -246,7 +245,7 @@ static block_t * CDDAReadBlocks( access_t * p_access ) if( go_on ) { dbg_print( (INPUT_DBG_LSN), "EOF"); - p_access->info.b_eof = VLC_TRUE; + p_access->info.b_eof = true; return NULL; } @@ -283,7 +282,7 @@ static block_t * CDDAReadBlocks( access_t * p_access ) { msg_Err( p_access, "cannot get a new block of size: %i", i_blocks * CDIO_CD_FRAMESIZE_RAW ); - intf_UserFatal( p_access, VLC_FALSE, _("CD reading failed"), + intf_UserFatal( p_access, false, _("CD reading failed"), _("VLC could not get a new block of size: %i."), i_blocks * CDIO_CD_FRAMESIZE_RAW ); return NULL; @@ -306,8 +305,8 @@ static block_t * CDDAReadBlocks( access_t * p_access ) msg_Err( p_access, "%s%s\n", psz_mes ? psz_mes: "", psz_err ? psz_err: "" ); - if( psz_err ) free( psz_err ); - if( psz_mes ) free( psz_mes ); + free( psz_err ); + free( psz_mes ); if( !p_readbuf ) { msg_Err( p_access, "paranoia read error on frame %i\n", @@ -354,8 +353,8 @@ static block_t * CDDAReadBlocks( access_t * p_access ) /***************************************************************************** * CDDARead: Handler for audio control reads the CD-DA. *****************************************************************************/ -static int -CDDARead( access_t * p_access, uint8_t *p_buffer, int i_len ) +static ssize_t +CDDARead( access_t * p_access, uint8_t *p_buffer, size_t i_len ) { cdda_data_t *p_cdda = (cdda_data_t *) p_access->p_sys; @@ -389,7 +388,7 @@ CDDARead( access_t * p_access, uint8_t *p_buffer, int i_len ) if( p_cdda->i_track >= p_cdda->i_first_track + p_cdda->i_titles - 1 ) { dbg_print( (INPUT_DBG_LSN), "EOF"); - p_access->info.b_eof = VLC_TRUE; + p_access->info.b_eof = true; return 0; } p_access->info.i_update |= INPUT_UPDATE_TITLE; @@ -400,7 +399,7 @@ CDDARead( access_t * p_access, uint8_t *p_buffer, int i_len ) { char *psz_title = CDDAFormatTitle( p_access, p_cdda->i_track ); input_Control( p_cdda->p_input, INPUT_SET_NAME, psz_title ); - free(psz_title); + free( psz_title ); } else { @@ -505,7 +504,7 @@ static int CDDASeek( access_t * p_access, int64_t i_pos ) } p_access->info.i_pos = i_pos; - p_access->info.b_eof = VLC_FALSE; + p_access->info.b_eof = false; return VLC_SUCCESS; } @@ -563,7 +562,7 @@ int CDDAOpen( vlc_object_t *p_this ) cdda_data_t *p_cdda = NULL; CdIo_t *p_cdio; track_t i_track = 1; - vlc_bool_t b_single_track = false; + bool b_single_track = false; int i_rc = VLC_EGENERIC; p_access->p_sys = NULL; @@ -598,6 +597,7 @@ int CDDAOpen( vlc_object_t *p_this ) if( !psz_source || !*psz_source ) { + free( psz_source ); /* No device/track given. Continue only when this plugin was selected */ if( !p_this->b_force ) @@ -606,6 +606,7 @@ int CDDAOpen( vlc_object_t *p_this ) psz_source = var_CreateGetString( p_this, "cd-audio" ); if( !psz_source || !*psz_source ) { + free( psz_source ); /* Scan for a CD-ROM drive with a CD-DA in it. */ char **ppsz_drives = cdio_get_devices_with_cap( NULL, CDIO_FS_AUDIO, false ); @@ -628,15 +629,13 @@ int CDDAOpen( vlc_object_t *p_this ) if( !(p_cdio = cdio_open( psz_source, DRIVER_UNKNOWN )) ) { msg_Warn( p_access, "could not open %s", psz_source ); - if ( psz_source ) - free( psz_source ); + free( psz_source ); return VLC_EGENERIC; } p_cdda = calloc( 1, sizeof(cdda_data_t) ); if( p_cdda == NULL ) { - msg_Err( p_access, "out of memory" ); free( psz_source ); return VLC_ENOMEM; } @@ -657,7 +656,7 @@ int CDDAOpen( vlc_object_t *p_this ) #endif p_cdda->psz_source = strdup( psz_source ); - p_cdda->b_header = VLC_FALSE; + p_cdda->b_header = false; p_cdda->p_cdio = p_cdio; p_cdda->i_tracks = 0; p_cdda->i_titles = 0; @@ -717,7 +716,7 @@ int CDDAOpen( vlc_object_t *p_this ) } p_access->info.i_update = 0; - p_access->info.b_eof = VLC_FALSE; + p_access->info.b_eof = false; p_access->info.i_title = 0; p_access->info.i_seekpoint = 0; @@ -774,6 +773,7 @@ int CDDAOpen( vlc_object_t *p_this ) } } } + free( psz_paranoia ); } #endif @@ -805,11 +805,12 @@ int CDDAOpen( vlc_object_t *p_this ) var_Create( p_access, MODULE_STRING "-caching", VLC_VAR_INTEGER|VLC_VAR_DOINHERIT ); vlc_object_release( p_cdda->p_input ); + free( psz_source ); return VLC_SUCCESS; error: cdio_destroy( p_cdda->p_cdio ); - if( psz_source) free( psz_source ); + free( psz_source ); if( p_cdda ) { if ( p_cdda->p_input ) @@ -857,8 +858,8 @@ void CDDAClose (vlc_object_t *p_this ) cdio_cddap_close_no_free_cdio( p_cdda->paranoia_cd ); #endif - if( p_cdda->psz_mcn ) free( p_cdda->psz_mcn ); - if( p_cdda->psz_source ) free( p_cdda->psz_source ); + free( p_cdda->psz_mcn ); + free( p_cdda->psz_source ); #if LIBCDDB_VERSION_NUM >= 1 libcddb_shutdown(); @@ -907,8 +908,8 @@ static int CDDAControl( access_t *p_access, int i_query, va_list args ) case ACCESS_CAN_CONTROL_PACE: { - vlc_bool_t *pb_bool = (vlc_bool_t*)va_arg( args, vlc_bool_t* ); - *pb_bool = p_cdda->b_audio_ctl ? VLC_FALSE : VLC_TRUE; + bool *pb_bool = (bool*)va_arg( args, bool* ); + *pb_bool = p_cdda->b_audio_ctl ? false : true; dbg_print( INPUT_DBG_META, "can control pace? %d", *pb_bool); return VLC_SUCCESS; } @@ -923,8 +924,8 @@ static int CDDAControl( access_t *p_access, int i_query, va_list args ) dbg_print( INPUT_DBG_META, "can pause?"); common: { - 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; } @@ -1006,7 +1007,7 @@ static int CDDAControl( access_t *p_access, int i_query, va_list args ) char *psz_title = CDDAFormatTitle( p_access, i_track ); input_Control( p_cdda->p_input, INPUT_SET_NAME, psz_title ); - free(psz_title); + free( psz_title ); p_cdda->i_track = i_track; i_last_lsn = cdio_get_track_lsn( p_cdda->p_cdio, CDIO_CDROM_LEADOUT_TRACK );