[FFmpeg-cvslog] pthread_frame: flush all threads on flush, not just the first one

Anton Khirnov git at videolan.org
Tue Mar 4 01:52:38 CET 2014


ffmpeg | branch: release/1.2 | Anton Khirnov <anton at khirnov.net> | Thu Feb 27 21:36:33 2014 +0100| [2256b2a3c1a6eae15dbfca0b95a64d911d618f40] | committer: Carl Eugen Hoyos

pthread_frame: flush all threads on flush, not just the first one

avcodec_flush_buffers() must release all internally held references
according to its documentation, for which all the threads need to be
flushed.

Bug-Id: vlc/9665
(cherry picked from commit d1f9563d502037239185c11578cc614bdf0c5870)

Conflicts:
	libavcodec/pthread_frame.c

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=2256b2a3c1a6eae15dbfca0b95a64d911d618f40
---

 libavcodec/pthread.c |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/libavcodec/pthread.c b/libavcodec/pthread.c
index 1ca72b4..7a57c2e 100644
--- a/libavcodec/pthread.c
+++ b/libavcodec/pthread.c
@@ -912,8 +912,6 @@ void ff_thread_flush(AVCodecContext *avctx)
     if (fctx->prev_thread) {
         if (fctx->prev_thread != &fctx->threads[0])
             update_context_from_thread(fctx->threads[0].avctx, fctx->prev_thread->avctx, 0);
-        if (avctx->codec->flush)
-            avctx->codec->flush(fctx->threads[0].avctx);
     }
 
     fctx->next_decoding = fctx->next_finished = 0;
@@ -925,6 +923,9 @@ void ff_thread_flush(AVCodecContext *avctx)
         p->got_frame = 0;
 
         release_delayed_buffers(p);
+
+        if (avctx->codec->flush)
+            avctx->codec->flush(fctx->threads[0].avctx);
     }
 }
 



More information about the ffmpeg-cvslog mailing list