deinterlace: fix infinite loop on EOS with non-default methods or fields

Fixes problem of infinite loop in gst_deinterlace_reset_history.
Last field in the history was never deinterlaced because idx becomes negative.

Happens e.g. with method=scalerbob fields=bottom or
method=greedyl fields=top

https://bugzilla.gnome.org/show_bug.cgi?id=695644
https://bugzilla.gnome.org/show_bug.cgi?id=693173

diff --git a/gst/deinterlace/gstdeinterlace.c b/gst/deinterlace/gstdeinterlace.c
index 240cbdb..9f59463 100644 (file)
--- a/gst/deinterlace/gstdeinterlace.c
+++ b/gst/deinterlace/gstdeinterlace.c
@@ -1708,6 +1708,8 @@ restart:
 
   if (!flushing && self->cur_field_idx < 1) {
     goto need_more;
+  } else if (self->cur_field_idx < 0 && flushing) {
+    self->cur_field_idx++;
   }
 
   if (self->fields == GST_DEINTERLACE_ALL || IS_TELECINE (interlacing_mode))