[MPlayer-cvslog] r32689 - trunk/libmpcodecs/ad_speex.c
reimar
subversion at mplayerhq.hu
Sat Dec 11 18:45:41 CET 2010
Author: reimar
Date: Sat Dec 11 18:45:41 2010
New Revision: 32689
Log:
Fix possible memory leaks on error.
Modified:
trunk/libmpcodecs/ad_speex.c
Modified: trunk/libmpcodecs/ad_speex.c
==============================================================================
--- trunk/libmpcodecs/ad_speex.c Sat Dec 11 17:36:17 2010 (r32688)
+++ trunk/libmpcodecs/ad_speex.c Sat Dec 11 18:45:41 2010 (r32689)
@@ -65,7 +65,7 @@ static int init(sh_audio_t *sh) {
const SpeexStereoState st_st = SPEEX_STEREO_STATE_INIT; // hack
if (!sh->wf || sh->wf->cbSize < 80) {
mp_msg(MSGT_DECAUDIO, MSGL_FATAL, "Missing extradata!\n");
- return 0;
+ goto err_out;
}
ctx->hdr = speex_packet_to_header((char *)&sh->wf[1], sh->wf->cbSize);
if (!ctx->hdr && sh->wf->cbSize == 0x72 && hdr[0] == 1 && hdr[1] == 0) {
@@ -87,7 +87,7 @@ static int init(sh_audio_t *sh) {
}
if (!ctx->hdr) {
mp_msg(MSGT_DECAUDIO, MSGL_FATAL, "Invalid extradata!\n");
- return 0;
+ goto err_out;
}
if (ctx->hdr->nb_channels != 1 && ctx->hdr->nb_channels != 2) {
mp_msg(MSGT_DECAUDIO, MSGL_WARN, "Invalid number of channels (%i), "
@@ -119,6 +119,12 @@ static int init(sh_audio_t *sh) {
sh->sample_format = AF_FORMAT_S16_NE;
sh->context = ctx;
return 1;
+
+err_out:
+ if (ctx)
+ free(ctx->hdr);
+ free(ctx);
+ return 0;
}
static void uninit(sh_audio_t *sh) {
More information about the MPlayer-cvslog
mailing list