mimic: do not release the newly obsolete reference at the end of decoding

Message ID 20161124222954.3FAD65DACC@aruru.libav.org
State New
Headers show

Commit Message

Janne Grunau Nov. 24, 2016, 10:29 p.m.
Module: libav
Branch: master
Commit: a115eb9e750543f1d8bf951414d291069bf396c2

Author:    Anton Khirnov <anton@khirnov.net>
Committer: Anton Khirnov <anton@khirnov.net>
Date:      Mon Jul 25 13:52:59 2016 +0200

mimic: do not release the newly obsolete reference at the end of decoding

The reference frames are used in update_thread_context(), so modifying
them after finish_setup() is a race. The frame in question will be
released during the next decode call.

CC: libav-stable@libav.org

---

 libavcodec/mimic.c |    3 ---
 1 file changed, 3 deletions(-)

Patch

diff --git a/libavcodec/mimic.c b/libavcodec/mimic.c
index 6f43723..d7723a8 100644
--- a/libavcodec/mimic.c
+++ b/libavcodec/mimic.c
@@ -445,9 +445,6 @@  static int mimic_decode_frame(AVCodecContext *avctx, void *data,
     ctx->prev_index = ctx->next_prev_index;
     ctx->cur_index  = ctx->next_cur_index;
 
-    /* Only release frames that aren't used for backreferences anymore */
-    ff_thread_release_buffer(avctx, &ctx->frames[ctx->cur_index]);
-
     return buf_size;
 }