[FFmpeg-cvslog] pthread: don't increment index on zero-sized packets.

Aaron Colwell git at videolan.org
Wed Nov 30 02:40:23 CET 2011


ffmpeg | branch: master | Aaron Colwell <acolwell at chromium.org> | Mon Nov 28 07:23:03 2011 -0800| [39a7a5b8ab12bc75306f52e671dfb1497771553b] | committer: Ronald S. Bultje

pthread: don't increment index on zero-sized packets.

The next call to decode() will update from an invalid index, which will
either lead to a memcpy() where dest==src (2 threads), or lead to a
crash (>2 threads).

Signed-off-by: Ronald S. Bultje <rsbultje at gmail.com>

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

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

diff --git a/libavcodec/pthread.c b/libavcodec/pthread.c
index 21e32b5..1364f57 100644
--- a/libavcodec/pthread.c
+++ b/libavcodec/pthread.c
@@ -491,6 +491,7 @@ static int submit_packet(PerThreadContext *p, AVPacket *avpkt)
     }
 
     fctx->prev_thread = p;
+    fctx->next_decoding++;
 
     return 0;
 }
@@ -513,8 +514,6 @@ int ff_thread_decode_frame(AVCodecContext *avctx,
     err = submit_packet(p, avpkt);
     if (err) return err;
 
-    fctx->next_decoding++;
-
     /*
      * If we're still receiving the initial packets, don't return a frame.
      */



More information about the ffmpeg-cvslog mailing list