pthread: Fix crash due to fctx->delaying not being cleared.
authorDale Curtis <dalecurtis@chromium.org>
Fri, 13 Apr 2012 00:12:27 +0000 (17:12 -0700)
committerRonald S. Bultje <rsbultje@gmail.com>
Fri, 13 Apr 2012 04:32:01 +0000 (21:32 -0700)
Reproducible with test case and ffplay -threads 2. Stack trace:
http://pastebin.com/PexZ4Uc0

Test case:
http://commondatastorage.googleapis.com/dalecurtis-shared/crash.ogm

Signed-off-by: Dale Curtis <dalecurtis@chromium.org>
Signed-off-by: Ronald S. Bultje <rsbultje@gmail.com>
libavcodec/pthread.c

index ee7bdb5..dc9220a 100644 (file)
@@ -603,11 +603,12 @@ int ff_thread_decode_frame(AVCodecContext *avctx,
      * If we're still receiving the initial packets, don't return a frame.
      */
 
-    if (fctx->delaying && avpkt->size) {
+    if (fctx->delaying) {
         if (fctx->next_decoding >= (avctx->thread_count-1)) fctx->delaying = 0;
 
         *got_picture_ptr=0;
-        return avpkt->size;
+        if (avpkt->size)
+            return avpkt->size;
     }
 
     /*