avformat/mov: remove redundant assignment
authorGanesh Ajjanagadde <gajjanagadde@gmail.com>
Mon, 16 Nov 2015 01:58:46 +0000 (20:58 -0500)
committerGanesh Ajjanagadde <gajjanagadde@gmail.com>
Mon, 16 Nov 2015 12:42:48 +0000 (07:42 -0500)
This is possibly undefined behavior based on sequence point rules, but I
have not studied the spec at that level of detail.

Fixes: CID 1338321.

Reviewed-by: Michael Niedermayer <michael@niedermayer.cc>
Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com>
libavformat/mov.c

index 7ab2808..0eb7272 100644 (file)
@@ -4936,7 +4936,7 @@ static int mov_seek_fragment(AVFormatContext *s, AVStream *st, int64_t timestamp
 
     for (i = 0; i < mov->fragment_index_count; i++) {
         if (mov->fragment_index_data[i]->track_id == st->id) {
-            MOVFragmentIndex *index = index = mov->fragment_index_data[i];
+            MOVFragmentIndex *index = mov->fragment_index_data[i];
             for (j = index->item_count - 1; j >= 0; j--) {
                 if (index->items[j].time <= timestamp) {
                     if (index->items[j].headers_read)