From 1a3ec4b7e6c02174160363879f1004aaeb3c0331 Mon Sep 17 00:00:00 2001 From: nicodvb Date: Sun, 12 Mar 2006 22:14:00 +0000 Subject: [PATCH] unified failure case in muxer_new_muxer() git-svn-id: svn://svn.mplayerhq.hu/mplayer/trunk@17840 b3059339-0415-0410-9bf9-f77b7e298cf2 --- libmpdemux/muxer.c | 32 ++++++++++---------------------- 1 file changed, 10 insertions(+), 22 deletions(-) diff --git a/libmpdemux/muxer.c b/libmpdemux/muxer.c index 3f956bed50..c59d3b1467 100644 --- a/libmpdemux/muxer.c +++ b/libmpdemux/muxer.c @@ -19,51 +19,39 @@ #include "stheader.h" muxer_t *muxer_new_muxer(int type,FILE *f){ - muxer_t* muxer=malloc(sizeof(muxer_t)); - memset(muxer,0,sizeof(muxer_t)); + muxer_t* muxer=calloc(1,sizeof(muxer_t)); if(!muxer) return NULL; muxer->file = f; switch (type) { case MUXER_TYPE_MPEG: if(! muxer_init_muxer_mpeg(muxer)) - { - free(muxer); - return NULL; - } + goto fail; break; case MUXER_TYPE_RAWVIDEO: if(! muxer_init_muxer_rawvideo(muxer)) - { - free(muxer); - return NULL; - } + goto fail; break; case MUXER_TYPE_RAWAUDIO: if(! muxer_init_muxer_rawaudio(muxer)) - { - free(muxer); - return NULL; - } + goto fail; break; #if defined(USE_LIBAVFORMAT) || defined(USE_LIBAVFORMAT_SO) case MUXER_TYPE_LAVF: if(! muxer_init_muxer_lavf(muxer)) - { - free(muxer); - return NULL; - } + goto fail; break; #endif case MUXER_TYPE_AVI: default: if(! muxer_init_muxer_avi(muxer)) - { - free(muxer); - return NULL; - } + goto fail; } return muxer; + +fail: + free(muxer); + return NULL; } /* buffer frames until we either: