From 579e353b0f05fe90d6c9ba2353aeae3c2acfb2bf Mon Sep 17 00:00:00 2001 From: eugeni Date: Fri, 29 Feb 2008 17:25:41 +0000 Subject: Remove stupid checks of free() argument. git-svn-id: svn://svn.mplayerhq.hu/mplayer/trunk@26117 b3059339-0415-0410-9bf9-f77b7e298cf2 --- libmpdemux/demux_mkv.c | 30 ++++++++++-------------------- libmpdemux/demuxer.c | 14 ++++++-------- 2 files changed, 16 insertions(+), 28 deletions(-) diff --git a/libmpdemux/demux_mkv.c b/libmpdemux/demux_mkv.c index 2a124ba902..6081fb8adb 100644 --- a/libmpdemux/demux_mkv.c +++ b/libmpdemux/demux_mkv.c @@ -927,18 +927,12 @@ demux_mkv_read_trackvideo (demuxer_t *demuxer, mkv_track_t *track) */ static void demux_mkv_free_trackentry(mkv_track_t *track) { - if (track->name) - free (track->name); - if (track->codec_id) - free (track->codec_id); - if (track->language) - free (track->language); - if (track->private_data) - free (track->private_data); - if (track->audio_buf) - free (track->audio_buf); - if (track->audio_timestamp) - free (track->audio_timestamp); + free (track->name); + free (track->codec_id); + free (track->language); + free (track->private_data); + free (track->audio_buf); + free (track->audio_timestamp); demux_mkv_free_encodings(track->encodings, track->num_encodings); free(track); } @@ -2545,14 +2539,10 @@ demux_close_mkv (demuxer_t *demuxer) demux_mkv_free_trackentry(mkv_d->tracks[i]); free (mkv_d->tracks); } - if (mkv_d->indexes) - free (mkv_d->indexes); - if (mkv_d->cluster_positions) - free (mkv_d->cluster_positions); - if (mkv_d->parsed_cues) - free (mkv_d->parsed_cues); - if (mkv_d->parsed_seekhead) - free (mkv_d->parsed_seekhead); + free (mkv_d->indexes); + free (mkv_d->cluster_positions); + free (mkv_d->parsed_cues); + free (mkv_d->parsed_seekhead); free (mkv_d); } } diff --git a/libmpdemux/demuxer.c b/libmpdemux/demuxer.c index 35cf4dc294..28388bb192 100644 --- a/libmpdemux/demuxer.c +++ b/libmpdemux/demuxer.c @@ -251,7 +251,7 @@ sh_sub_t *new_sh_sub_sid(demuxer_t *demuxer, int id, int sid) { void free_sh_sub(sh_sub_t *sh) { mp_msg(MSGT_DEMUXER, MSGL_DBG2, "DEMUXER: freeing sh_sub at %p\n", sh); - if (sh->extradata) free(sh->extradata); + free(sh->extradata); #ifdef USE_ASS if (sh->ass_track) ass_free_track(sh->ass_track); #endif @@ -288,8 +288,8 @@ void free_sh_audio(demuxer_t *demuxer, int id) { sh_audio_t *sh = demuxer->a_streams[id]; demuxer->a_streams[id] = NULL; mp_msg(MSGT_DEMUXER,MSGL_DBG2,"DEMUXER: freeing sh_audio at %p\n",sh); - if(sh->wf) free(sh->wf); - if(sh->codecdata) free(sh->codecdata); + free(sh->wf); + free(sh->codecdata); free(sh->lang); free(sh); } @@ -314,7 +314,7 @@ sh_video_t* new_sh_video_vid(demuxer_t *demuxer,int id,int vid){ void free_sh_video(sh_video_t* sh){ mp_msg(MSGT_DEMUXER,MSGL_DBG2,"DEMUXER: freeing sh_video at %p\n",sh); - if(sh->bih) free(sh->bih); + free(sh->bih); free(sh); } @@ -343,12 +343,10 @@ skip_streamfree: free(demuxer->info[i]); free(demuxer->info); } - if(demuxer->filename) - free(demuxer->filename); + free(demuxer->filename); if (demuxer->chapters) { for (i=0; inum_chapters; i++) - if (demuxer->chapters[i].name) - free(demuxer->chapters[i].name); + free(demuxer->chapters[i].name); free(demuxer->chapters); } if (demuxer->attachments) { -- cgit v1.2.3