[FFmpeg-cvslog] libfdk-aacdec: Clean up properly if the init fails
Martin Storsjö
git at videolan.org
Tue Aug 18 09:10:01 CEST 2015
ffmpeg | branch: master | Martin Storsjö <martin at martin.st> | Fri Aug 14 11:33:03 2015 +0300| [f34b152eb7b7e8d2aee57c710a072cf74173fbe1] | committer: Martin Storsjö
libfdk-aacdec: Clean up properly if the init fails
Previously most of the error paths leaked.
Also add FF_CODEC_CAP_INIT_THREADSAFE while adding caps_internal;
this decoder wrapper doesn't have any static data that is initialized.
Signed-off-by: Martin Storsjö <martin at martin.st>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=f34b152eb7b7e8d2aee57c710a072cf74173fbe1
---
libavcodec/libfdk-aacdec.c | 17 ++++++-----------
1 file changed, 6 insertions(+), 11 deletions(-)
diff --git a/libavcodec/libfdk-aacdec.c b/libavcodec/libfdk-aacdec.c
index 9f896ef..5ac8fb7 100644
--- a/libavcodec/libfdk-aacdec.c
+++ b/libavcodec/libfdk-aacdec.c
@@ -256,13 +256,11 @@ static av_cold int fdk_aac_decode_init(AVCodecContext *avctx)
s->anc_buffer = av_malloc(DMX_ANC_BUFFSIZE);
if (!s->anc_buffer) {
av_log(avctx, AV_LOG_ERROR, "Unable to allocate ancillary buffer for the decoder\n");
- ret = AVERROR(ENOMEM);
- goto fail;
+ return AVERROR(ENOMEM);
}
if (aacDecoder_AncDataInit(s->handle, s->anc_buffer, DMX_ANC_BUFFSIZE)) {
av_log(avctx, AV_LOG_ERROR, "Unable to register downmix ancillary buffer in the decoder\n");
- ret = AVERROR_UNKNOWN;
- goto fail;
+ return AVERROR_UNKNOWN;
}
}
}
@@ -307,15 +305,10 @@ static av_cold int fdk_aac_decode_init(AVCodecContext *avctx)
s->decoder_buffer_size = DECODER_BUFFSIZE * DECODER_MAX_CHANNELS;
s->decoder_buffer = av_malloc(s->decoder_buffer_size);
- if (!s->decoder_buffer) {
- ret = AVERROR(ENOMEM);
- goto fail;
- }
+ if (!s->decoder_buffer)
+ return AVERROR(ENOMEM);
return 0;
-fail:
- fdk_aac_decode_close(avctx);
- return ret;
}
static int fdk_aac_decode_frame(AVCodecContext *avctx, void *data,
@@ -389,4 +382,6 @@ AVCodec ff_libfdk_aac_decoder = {
.flush = fdk_aac_decode_flush,
.capabilities = AV_CODEC_CAP_DR1 | AV_CODEC_CAP_CHANNEL_CONF,
.priv_class = &fdk_aac_dec_class,
+ .caps_internal = FF_CODEC_CAP_INIT_THREADSAFE |
+ FF_CODEC_CAP_INIT_CLEANUP,
};
More information about the ffmpeg-cvslog
mailing list