[FFmpeg-cvslog] avfilter/vf_detelecine: Fix uninitialized array index error

[Himangi Saraogi]

ffmpeg | branch: master | Himangi Saraogi <himangi774 at gmail.com> | Tue Mar 31 13:01:46 2015 +0530| [0c9abf211931b1a5e3b5d1ebfac3d2214103ae2f] | committer: Michael Niedermayer

avfilter/vf_detelecine: Fix uninitialized array index error

Fixes the defect CID 1292301

Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

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

 libavfilter/vf_detelecine.c |   16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/libavfilter/vf_detelecine.c b/libavfilter/vf_detelecine.c
index 218a8c8..8ad7ae3 100644
--- a/libavfilter/vf_detelecine.c
+++ b/libavfilter/vf_detelecine.c
@@ -265,12 +265,16 @@ static int filter_frame(AVFilterLink *inlink, AVFrame *inpicref)
                 out = 1;
             } else if (len == 1) {
                 // fill in the EARLIER field from the new pic
-                av_image_copy_plane(s->frame->data[i] + s->frame->linesize[i] * s->first_field,
-                                    s->frame->linesize[i] * 2,
-                                    inpicref->data[i] + inpicref->linesize[i] * s->first_field,
-                                    inpicref->linesize[i] * 2,
-                                    s->stride[i],
-                                    (s->planeheight[i] - s->first_field + 1) / 2);
+                for (i = 0; i < s->nb_planes; i++) {
+                    av_image_copy_plane(s->frame->data[i] +
+                                        s->frame->linesize[i] * s->first_field,
+                                        s->frame->linesize[i] * 2,
+                                        inpicref->data[i] +
+                                        inpicref->linesize[i] * s->first_field,
+                                        inpicref->linesize[i] * 2, s->stride[i],
+                                        (s->planeheight[i] - s->first_field + 1) / 2);
+                 }
+
                 // TODO: not sure about the other field
 
                 len--;