diff options
| author | Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com> | 2015-02-22 20:43:30 +0100 |
|---|---|---|
| committer | Michael Niedermayer <michaelni@gmx.at> | 2015-02-23 13:35:08 +0100 |
| commit | d0da72ebb04a1812433099c8effd8f753a4f7e61 (patch) | |
| tree | 2231fa91b0aafdc6b7d2edbbd70fec93d755f2d4 | |
| parent | b62a3478e92be87e2b508b9aae6f86b20db9f9e2 (diff) | |
| download | ffmpeg-d0da72ebb04a1812433099c8effd8f753a4f7e61.tar.gz | |
avcodec/a64multienc: use av_frame_ref instead of copying the frame
This fixes freeing the frame buffer twice on cleanup leading to a crash.
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
(cherry picked from commit 39e4ed7c1d8d840be47f6d604704d47a59a9ae5d)
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
| -rw-r--r-- | libavcodec/a64multienc.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/libavcodec/a64multienc.c b/libavcodec/a64multienc.c index d692ebfdef..888b87be1a 100644 --- a/libavcodec/a64multienc.c +++ b/libavcodec/a64multienc.c @@ -317,7 +317,9 @@ static int a64multi_encode_frame(AVCodecContext *avctx, AVPacket *pkt, } else { /* fill up mc_meta_charset with data until lifetime exceeds */ if (c->mc_frame_counter < c->mc_lifetime) { - *p = *pict; + ret = av_frame_ref(p, pict); + if (ret < 0) + return ret; p->pict_type = AV_PICTURE_TYPE_I; p->key_frame = 1; to_meta_with_crop(avctx, p, meta + 32000 * c->mc_frame_counter); |