diff options
author | Michael Niedermayer <michaelni@gmx.at> | 2012-02-11 02:06:24 +0100 |
---|---|---|
committer | Michael Niedermayer <michaelni@gmx.at> | 2012-02-11 02:06:24 +0100 |
commit | ed83f97235b6b3bd2cf62830bba00dfd7327d389 (patch) | |
tree | 1614d17eb7c70185aa56fff3ed0234de0f29c830 /libavcodec/tta.c | |
parent | c6abf214923ae8ee07456c5594f632a10499442b (diff) | |
download | ffmpeg-ed83f97235b6b3bd2cf62830bba00dfd7327d389.tar.gz |
tta: Add some safety precautions to avoid freeing things that have not been allocated.
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
Diffstat (limited to 'libavcodec/tta.c')
-rw-r--r-- | libavcodec/tta.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/libavcodec/tta.c b/libavcodec/tta.c index 0e3bbd224c..6b8e0e9e19 100644 --- a/libavcodec/tta.c +++ b/libavcodec/tta.c @@ -283,7 +283,8 @@ static av_cold int tta_decode_init(AVCodecContext * avctx) s->decode_buffer = av_mallocz(sizeof(int32_t)*s->frame_length*s->channels); if (!s->decode_buffer) return AVERROR(ENOMEM); - } + } else + s->decode_buffer = NULL; s->ch_ctx = av_malloc(avctx->channels * sizeof(*s->ch_ctx)); if (!s->ch_ctx) { av_freep(&s->decode_buffer); @@ -449,7 +450,9 @@ static int tta_decode_frame(AVCodecContext *avctx, void *data, static av_cold int tta_decode_close(AVCodecContext *avctx) { TTAContext *s = avctx->priv_data; - av_free(s->decode_buffer); + if (s->bps < 3) + av_free(s->decode_buffer); + s->decode_buffer = NULL; av_freep(&s->ch_ctx); return 0; |