lavfi/fifo: fix flushing when using request_samples
authorAnton Khirnov <anton@khirnov.net>
Sun, 4 Aug 2013 10:10:23 +0000 (12:10 +0200)
committerAnton Khirnov <anton@khirnov.net>
Mon, 5 Aug 2013 08:52:00 +0000 (10:52 +0200)
If any samples are still buffered when request_frame returns EOF, they
won't be returned currently.

libavfilter/fifo.c

index 8d981ce..a58ce9f 100644 (file)
@@ -147,10 +147,14 @@ static int return_audio_frame(AVFilterContext *ctx)
 {
     AVFilterLink *link = ctx->outputs[0];
     FifoContext *s = ctx->priv;
-    AVFrame *head = s->root.next->frame;
+    AVFrame *head = s->root.next ? s->root.next->frame : NULL;
     AVFrame *out;
     int ret;
 
+    /* if head is NULL then we're flushing the remaining samples in out */
+    if (!head && !s->out)
+        return AVERROR_EOF;
+
     if (!s->out &&
         head->nb_samples >= link->request_samples &&
         calc_ptr_alignment(head) >= 32) {
@@ -227,8 +231,11 @@ static int request_frame(AVFilterLink *outlink)
     int ret = 0;
 
     if (!fifo->root.next) {
-        if ((ret = ff_request_frame(outlink->src->inputs[0])) < 0)
+        if ((ret = ff_request_frame(outlink->src->inputs[0])) < 0) {
+            if (ret == AVERROR_EOF && outlink->request_samples)
+                return return_audio_frame(outlink->src);
             return ret;
+        }
     }
 
     if (outlink->request_samples) {