Merge commit 'bad4aad4037f59ba0ad656164be9ab8f7a0fa2d4'
authorClément Bœsch <u@pkh.me>
Thu, 23 Mar 2017 10:26:20 +0000 (11:26 +0100)
committerClément Bœsch <u@pkh.me>
Thu, 23 Mar 2017 10:26:32 +0000 (11:26 +0100)
* commit 'bad4aad4037f59ba0ad656164be9ab8f7a0fa2d4':
  avidec: Do not special case palette on big-endian

This commit is a noop, see 64cafe340bd5ddfe704efa95cd9f21471ca12a12

Merged-by: Clément Bœsch <u@pkh.me>
1  2 
libavformat/avidec.c

diff --combined libavformat/avidec.c
@@@ -2,29 -2,27 +2,29 @@@
   * AVI demuxer
   * Copyright (c) 2001 Fabrice Bellard
   *
 - * This file is part of Libav.
 + * This file is part of FFmpeg.
   *
 - * Libav is free software; you can redistribute it and/or
 + * FFmpeg is free software; you can redistribute it and/or
   * modify it under the terms of the GNU Lesser General Public
   * License as published by the Free Software Foundation; either
   * version 2.1 of the License, or (at your option) any later version.
   *
 - * Libav is distributed in the hope that it will be useful,
 + * FFmpeg is distributed in the hope that it will be useful,
   * but WITHOUT ANY WARRANTY; without even the implied warranty of
   * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
   * Lesser General Public License for more details.
   *
   * You should have received a copy of the GNU Lesser General Public
 - * License along with Libav; if not, write to the Free Software
 + * License along with FFmpeg; if not, write to the Free Software
   * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
   */
  
  #include <inttypes.h>
  
 +#include "libavutil/avassert.h"
  #include "libavutil/avstring.h"
  #include "libavutil/bswap.h"
 +#include "libavutil/opt.h"
  #include "libavutil/dict.h"
  #include "libavutil/internal.h"
  #include "libavutil/intreadwrite.h"
@@@ -35,9 -33,9 +35,9 @@@
  #include "internal.h"
  #include "isom.h"
  #include "riff.h"
 -
 -#undef NDEBUG
 -#include <assert.h>
 +#include "libavcodec/bytestream.h"
 +#include "libavcodec/exif.h"
 +#include "libavcodec/internal.h"
  
  typedef struct AVIStream {
      int64_t frame_offset;   /* current frame (video) or byte (audio) counter
      AVFormatContext *sub_ctx;
      AVPacket sub_pkt;
      uint8_t *sub_buffer;
 +
 +    int64_t seek_pos;
  } AVIStream;
  
  typedef struct AVIContext {
 +    const AVClass *class;
      int64_t riff_end;
      int64_t movi_end;
      int64_t fsize;
 +    int64_t io_fsize;
      int64_t movi_list;
      int64_t last_pkt_pos;
      int index_loaded;
      int stream_index;
      DVDemuxContext *dv_demux;
      int odml_depth;
 +    int use_odml;
  #define MAX_ODML_DEPTH 1000
 +    int64_t dts_max;
  } AVIContext;
  
 +
 +static const AVOption options[] = {
 +    { "use_odml", "use odml index", offsetof(AVIContext, use_odml), AV_OPT_TYPE_BOOL, {.i64 = 1}, -1, 1, AV_OPT_FLAG_DECODING_PARAM},
 +    { NULL },
 +};
 +
 +static const AVClass demuxer_class = {
 +    .class_name = "avi",
 +    .item_name  = av_default_item_name,
 +    .option     = options,
 +    .version    = LIBAVUTIL_VERSION_INT,
 +    .category   = AV_CLASS_CATEGORY_DEMUXER,
 +};
 +
 +
  static const char avi_headers[][8] = {
      { 'R', 'I', 'F', 'F', 'A', 'V', 'I', ' '  },
      { 'R', 'I', 'F', 'F', 'A', 'V', 'I', 'X'  },
@@@ -118,8 -95,8 +118,8 @@@ static int avi_load_index(AVFormatConte
  static int guess_ni_flag(AVFormatContext *s);
  
  #define print_tag(str, tag, size)                        \
 -    av_log(NULL, AV_LOG_TRACE, "%s: tag=%c%c%c%c size=0x%x\n",        \
 -            str, tag & 0xff,                             \
 +    av_log(NULL, AV_LOG_TRACE, "pos:%"PRIX64" %s: tag=%c%c%c%c size=0x%x\n", \
 +            avio_tell(pb), str, tag & 0xff,              \
              (tag >> 8) & 0xff,                           \
              (tag >> 16) & 0xff,                          \
              (tag >> 24) & 0xff,                          \
@@@ -138,7 -115,7 +138,7 @@@ static inline int get_duration(AVIStrea
  static int get_riff(AVFormatContext *s, AVIOContext *pb)
  {
      AVIContext *avi = s->priv_data;
 -    char header[8];
 +    char header[8] = {0};
      int i;
  
      /* check RIFF header */
      return 0;
  }
  
 -static int read_braindead_odml_indx(AVFormatContext *s, int frame_num)
 +static int read_odml_index(AVFormatContext *s, int frame_num)
  {
      AVIContext *avi     = s->priv_data;
      AVIOContext *pb     = s->pb;
 -    int longs_pre_entry = avio_rl16(pb);
 +    int longs_per_entry = avio_rl16(pb);
      int index_sub_type  = avio_r8(pb);
      int index_type      = avio_r8(pb);
      int entries_in_use  = avio_rl32(pb);
      int64_t filesize = avi->fsize;
  
      av_log(s, AV_LOG_TRACE,
 -            "longs_pre_entry:%d index_type:%d entries_in_use:%d "
 -            "chunk_id:%X base:%16"PRIX64"\n",
 -            longs_pre_entry,
 +            "longs_per_entry:%d index_type:%d entries_in_use:%d "
 +            "chunk_id:%X base:%16"PRIX64" frame_num:%d\n",
 +            longs_per_entry,
              index_type,
              entries_in_use,
              chunk_id,
 -            base);
 +            base,
 +            frame_num);
  
      if (stream_id >= s->nb_streams || stream_id < 0)
          return AVERROR_INVALIDDATA;
  
      avio_rl32(pb);
  
 -    if (index_type && longs_pre_entry != 2)
 +    if (index_type && longs_per_entry != 2)
          return AVERROR_INVALIDDATA;
      if (index_type > 1)
          return AVERROR_INVALIDDATA;
  
              av_log(s, AV_LOG_TRACE, "pos:%"PRId64", len:%X\n", pos, len);
  
 -            if (pb->eof_reached)
 +            if (avio_feof(pb))
                  return AVERROR_INVALIDDATA;
  
              if (last_pos == pos || pos == base - 8)
                  avi->non_interleaved = 1;
 -            if (last_pos != pos && (len || !ast->sample_size))
 +            if (last_pos != pos && len)
                  av_add_index_entry(st, pos, ast->cum_len, len, 0,
                                     key ? AVINDEX_KEYFRAME : 0);
  
              avio_rl32(pb);       /* size */
              duration = avio_rl32(pb);
  
 -            if (pb->eof_reached)
 +            if (avio_feof(pb))
                  return AVERROR_INVALIDDATA;
  
              pos = avio_tell(pb);
                  return AVERROR_INVALIDDATA;
              }
  
 -            avio_seek(pb, offset + 8, SEEK_SET);
 +            if (avio_seek(pb, offset + 8, SEEK_SET) < 0)
 +                return -1;
              avi->odml_depth++;
 -            read_braindead_odml_indx(s, frame_num);
 +            read_odml_index(s, frame_num);
              avi->odml_depth--;
              frame_num += duration;
  
 -            avio_seek(pb, pos, SEEK_SET);
 +            if (avio_seek(pb, pos, SEEK_SET) < 0) {
 +                av_log(s, AV_LOG_ERROR, "Failed to restore position after reading index\n");
 +                return -1;
 +            }
 +
          }
      }
 -    avi->index_loaded = 1;
 +    avi->index_loaded = 2;
      return 0;
  }
  
@@@ -310,8 -281,7 +310,8 @@@ static int avi_read_tag(AVFormatContex
      value = av_malloc(size + 1);
      if (!value)
          return AVERROR(ENOMEM);
 -    avio_read(pb, value, size);
 +    if (avio_read(pb, value, size) != size)
 +        return AVERROR_INVALIDDATA;
      value[size] = 0;
  
      AV_WL32(key, tag);
@@@ -344,19 -314,18 +344,19 @@@ static void avi_metadata_creation_time(
  
  static void avi_read_nikon(AVFormatContext *s, uint64_t end)
  {
 -    while (avio_tell(s->pb) < end) {
 +    while (avio_tell(s->pb) < end && !avio_feof(s->pb)) {
          uint32_t tag  = avio_rl32(s->pb);
          uint32_t size = avio_rl32(s->pb);
          switch (tag) {
          case MKTAG('n', 'c', 't', 'g'):  /* Nikon Tags */
          {
              uint64_t tag_end = avio_tell(s->pb) + size;
 -            while (avio_tell(s->pb) < tag_end) {
 +            while (avio_tell(s->pb) < tag_end && !avio_feof(s->pb)) {
                  uint16_t tag     = avio_rl16(s->pb);
                  uint16_t size    = avio_rl16(s->pb);
                  const char *name = NULL;
                  char buffer[64]  = { 0 };
 +                size = FFMIN(size, tag_end - avio_tell(s->pb));
                  size -= avio_read(s->pb, buffer,
                                    FFMIN(size, sizeof(buffer) - 1));
                  switch (tag) {
      }
  }
  
 +static int avi_extract_stream_metadata(AVFormatContext *s, AVStream *st)
 +{
 +    GetByteContext gb;
 +    uint8_t *data = st->codecpar->extradata;
 +    int data_size = st->codecpar->extradata_size;
 +    int tag, offset;
 +
 +    if (!data || data_size < 8) {
 +        return AVERROR_INVALIDDATA;
 +    }
 +
 +    bytestream2_init(&gb, data, data_size);
 +
 +    tag = bytestream2_get_le32(&gb);
 +
 +    switch (tag) {
 +    case MKTAG('A', 'V', 'I', 'F'):
 +        // skip 4 byte padding
 +        bytestream2_skip(&gb, 4);
 +        offset = bytestream2_tell(&gb);
 +        bytestream2_init(&gb, data + offset, data_size - offset);
 +
 +        // decode EXIF tags from IFD, AVI is always little-endian
 +        return avpriv_exif_decode_ifd(s, &gb, 1, 0, &st->metadata);
 +        break;
 +    case MKTAG('C', 'A', 'S', 'I'):
 +        avpriv_request_sample(s, "RIFF stream data tag type CASI (%u)", tag);
 +        break;
 +    case MKTAG('Z', 'o', 'r', 'a'):
 +        avpriv_request_sample(s, "RIFF stream data tag type Zora (%u)", tag);
 +        break;
 +    default:
 +        break;
 +    }
 +
 +    return 0;
 +}
 +
 +static int calculate_bitrate(AVFormatContext *s)
 +{
 +    AVIContext *avi = s->priv_data;
 +    int i, j;
 +    int64_t lensum = 0;
 +    int64_t maxpos = 0;
 +
 +    for (i = 0; i<s->nb_streams; i++) {
 +        int64_t len = 0;
 +        AVStream *st = s->streams[i];
 +
 +        if (!st->nb_index_entries)
 +            continue;
 +
 +        for (j = 0; j < st->nb_index_entries; j++)
 +            len += st->index_entries[j].size;
 +        maxpos = FFMAX(maxpos, st->index_entries[j-1].pos);
 +        lensum += len;
 +    }
 +    if (maxpos < avi->io_fsize*9/10) // index does not cover the whole file
 +        return 0;
 +    if (lensum*9/10 > maxpos || lensum < maxpos*9/10) // frame sum and filesize mismatch
 +        return 0;
 +
 +    for (i = 0; i<s->nb_streams; i++) {
 +        int64_t len = 0;
 +        AVStream *st = s->streams[i];
 +        int64_t duration;
 +        int64_t bitrate;
 +
 +        for (j = 0; j < st->nb_index_entries; j++)
 +            len += st->index_entries[j].size;
 +
 +        if (st->nb_index_entries < 2 || st->codecpar->bit_rate > 0)
 +            continue;
 +        duration = st->index_entries[j-1].timestamp - st->index_entries[0].timestamp;
 +        bitrate = av_rescale(8*len, st->time_base.den, duration * st->time_base.num);
 +        if (bitrate <= INT_MAX && bitrate > 0) {
 +            st->codecpar->bit_rate = bitrate;
 +        }
 +    }
 +    return 1;
 +}
 +
  static int avi_read_header(AVFormatContext *s)
  {
      AVIContext *avi = s->priv_data;
      uint64_t list_end   = 0;
      int64_t pos;
      int ret;
 +    AVDictionaryEntry *dict_entry;
  
      avi->stream_index = -1;
  
      if (ret < 0)
          return ret;
  
 -    avi->fsize = avio_size(pb);
 +    av_log(avi, AV_LOG_DEBUG, "use odml:%d\n", avi->use_odml);
 +
 +    avi->io_fsize = avi->fsize = avio_size(pb);
      if (avi->fsize <= 0 || avi->fsize < avi->riff_end)
          avi->fsize = avi->riff_end == 8 ? INT64_MAX : avi->riff_end;
  
      codec_type   = -1;
      frame_period = 0;
      for (;;) {
 -        if (pb->eof_reached)
 +        if (avio_feof(pb))
              goto fail;
          tag  = avio_rl32(pb);
          size = avio_rl32(pb);
              /* AVI header */
              /* using frame_period is bad idea */
              frame_period = avio_rl32(pb);
 -            avio_skip(pb, 4);
 +            avio_rl32(pb); /* max. bytes per second */
              avio_rl32(pb);
              avi->non_interleaved |= avio_rl32(pb) & AVIF_MUSTUSEINDEX;
  
                  ast = s->streams[0]->priv_data;
                  av_freep(&s->streams[0]->codecpar->extradata);
                  av_freep(&s->streams[0]->codecpar);
 +#if FF_API_LAVF_AVCTX
 +FF_DISABLE_DEPRECATION_WARNINGS
 +                av_freep(&s->streams[0]->codec);
 +FF_ENABLE_DEPRECATION_WARNINGS
 +#endif
 +                if (s->streams[0]->info)
 +                    av_freep(&s->streams[0]->info->duration_error);
                  av_freep(&s->streams[0]->info);
 +                if (s->streams[0]->internal)
 +                    av_freep(&s->streams[0]->internal->avctx);
 +                av_freep(&s->streams[0]->internal);
                  av_freep(&s->streams[0]);
                  s->nb_streams = 0;
                  if (CONFIG_DV_DEMUXER) {
                  break;
              }
  
 -            assert(stream_index < s->nb_streams);
 +            av_assert0(stream_index < s->nb_streams);
              ast->handler = handler;
  
              avio_rl32(pb); /* flags */
              st->start_time = 0;
              avio_rl32(pb); /* buffer size */
              avio_rl32(pb); /* quality */
 -            ast->sample_size = avio_rl32(pb); /* sample size */
 +            if (ast->cum_len*ast->scale/ast->rate > 3600) {
 +                av_log(s, AV_LOG_ERROR, "crazy start time, iam scared, giving up\n");
 +                ast->cum_len = 0;
 +            }
 +            ast->sample_size = avio_rl32(pb);
              ast->cum_len    *= FFMAX(1, ast->sample_size);
              av_log(s, AV_LOG_TRACE, "%"PRIu32" %"PRIu32" %d\n",
                      ast->rate, ast->scale, ast->sample_size);
                  codec_type = AVMEDIA_TYPE_VIDEO;
  
                  ast->sample_size = 0;
 +                st->avg_frame_rate = av_inv_q(st->time_base);
                  break;
              case MKTAG('a', 'u', 'd', 's'):
                  codec_type = AVMEDIA_TYPE_AUDIO;
                  codec_type = AVMEDIA_TYPE_DATA;
                  break;
              default:
 -                av_log(s, AV_LOG_ERROR, "unknown stream type %X\n", tag1);
 -                goto fail;
 +                av_log(s, AV_LOG_INFO, "unknown stream type %X\n", tag1);
              }
  
              if (ast->sample_size < 0) {
                  ast->sample_size = 0;
              }
  
 -            if (ast->sample_size == 0)
 +            if (ast->sample_size == 0) {
                  st->duration = st->nb_frames;
 +                if (st->duration > 0 && avi->io_fsize > 0 && avi->riff_end > avi->io_fsize) {
 +                    av_log(s, AV_LOG_DEBUG, "File is truncated adjusting duration\n");
 +                    st->duration = av_rescale(st->duration, avi->io_fsize, avi->riff_end);
 +                }
 +            }
              ast->frame_offset = ast->cum_len;
              avio_skip(pb, size - 12 * 4);
              break;
          case MKTAG('s', 't', 'r', 'f'):
              /* stream header */
 +            if (!size)
 +                break;
              if (stream_index >= (unsigned)s->nb_streams || avi->dv_demux) {
                  avio_skip(pb, size);
              } else {
                  uint64_t cur_pos = avio_tell(pb);
 +                unsigned esize;
                  if (cur_pos < list_end)
                      size = FFMIN(size, list_end - cur_pos);
                  st = s->streams[stream_index];
 +                if (st->codecpar->codec_type != AVMEDIA_TYPE_UNKNOWN) {
 +                    avio_skip(pb, size);
 +                    break;
 +                }
                  switch (codec_type) {
                  case AVMEDIA_TYPE_VIDEO:
                      if (amv_file_format) {
                          avio_skip(pb, size);
                          break;
                      }
 -                    tag1 = ff_get_bmp_header(pb, st);
 +                    tag1 = ff_get_bmp_header(pb, st, &esize);
  
                      if (tag1 == MKTAG('D', 'X', 'S', 'B') ||
                          tag1 == MKTAG('D', 'X', 'S', 'A')) {
                          break;
                      }
  
 -                    if (size > 10 * 4 && size < (1 << 30)) {
 -                        st->codecpar->extradata_size = size - 10 * 4;
 -                        st->codecpar->extradata      = av_malloc(st->codecpar->extradata_size +
 -                                                                 AV_INPUT_BUFFER_PADDING_SIZE);
 -                        if (!st->codecpar->extradata) {
 -                            st->codecpar->extradata_size = 0;
 -                            return AVERROR(ENOMEM);
 +                    if (size > 10 * 4 && size < (1 << 30) && size < avi->fsize) {
 +                        if (esize == size-1 && (esize&1)) {
 +                            st->codecpar->extradata_size = esize - 10 * 4;
 +                        } else
 +                            st->codecpar->extradata_size =  size - 10 * 4;
 +                        if (st->codecpar->extradata) {
 +                            av_log(s, AV_LOG_WARNING, "New extradata in strf chunk, freeing previous one.\n");
 +                            av_freep(&st->codecpar->extradata);
                          }
 -                        avio_read(pb,
 -                                  st->codecpar->extradata,
 -                                  st->codecpar->extradata_size);
 +                        if (ff_get_extradata(s, st->codecpar, pb, st->codecpar->extradata_size) < 0)
 +                            return AVERROR(ENOMEM);
                      }
  
                      // FIXME: check if the encoder really did this correctly
                      /* Extract palette from extradata if bpp <= 8.
                       * This code assumes that extradata contains only palette.
                       * This is true for all paletted codecs implemented in
 -                     * Libav. */
 +                     * FFmpeg. */
                      if (st->codecpar->extradata_size &&
                          (st->codecpar->bits_per_coded_sample <= 8)) {
                          int pal_size = (1 << st->codecpar->bits_per_coded_sample) << 2;
                          pal_size = FFMIN(pal_size, st->codecpar->extradata_size);
                          pal_src  = st->codecpar->extradata +
                                     st->codecpar->extradata_size - pal_size;
 +                        /* Exclude the "BottomUp" field from the palette */
 +                        if (pal_src - st->codecpar->extradata >= 9 &&
 +                            !memcmp(st->codecpar->extradata + st->codecpar->extradata_size - 9, "BottomUp", 9))
 +                            pal_src -= 9;
                          for (i = 0; i < pal_size / 4; i++)
-                             ast->pal[i] = 0xFFU<<24 | AV_RL32(pal_src+4*i);
 -                            ast->pal[i] = AV_RL32(pal_src + 4 * i);
++                            ast->pal[i] = 0xFFU<<24 | AV_RL32(pal_src + 4 * i);
                          ast->has_pal = 1;
                      }
  
                          ast->handler == MKTAG('X', 'V', 'I', 'D'))
                          st->codecpar->codec_tag = MKTAG('X', 'V', 'I', 'D');
  
 -                    // Support "Resolution 1:1" for Avid AVI Codec
 -                    if (tag1 == MKTAG('A', 'V', 'R', 'n') &&
 -                        st->codecpar->extradata_size >= 31   &&
 -                        !memcmp(&st->codecpar->extradata[28], "1:1", 3))
 -                        st->codecpar->codec_id = AV_CODEC_ID_RAWVIDEO;
 +                    if (st->codecpar->codec_tag == MKTAG('V', 'S', 'S', 'H'))
 +                        st->need_parsing = AVSTREAM_PARSE_FULL;
 +                    if (st->codecpar->codec_id == AV_CODEC_ID_RV40)
 +                        st->need_parsing = AVSTREAM_PARSE_NONE;
  
                      if (st->codecpar->codec_tag == 0 && st->codecpar->height > 0 &&
                          st->codecpar->extradata_size < 1U << 30) {
  //                    avio_skip(pb, size - 5 * 4);
                      break;
                  case AVMEDIA_TYPE_AUDIO:
 -                    ret = ff_get_wav_header(s, pb, st->codecpar, size);
 +                    ret = ff_get_wav_header(s, pb, st->codecpar, size, 0);
                      if (ret < 0)
                          return ret;
                      ast->dshow_block_align = st->codecpar->block_align;
                      if (st->codecpar->codec_id == AV_CODEC_ID_AAC &&
                          st->codecpar->extradata_size)
                          st->need_parsing = AVSTREAM_PARSE_NONE;
 +                    // The flac parser does not work with AVSTREAM_PARSE_TIMESTAMPS
 +                    if (st->codecpar->codec_id == AV_CODEC_ID_FLAC)
 +                        st->need_parsing = AVSTREAM_PARSE_NONE;
                      /* AVI files with Xan DPCM audio (wrongly) declare PCM
                       * audio in the header but have Axan as stream_code_tag. */
                      if (ast->handler == AV_RL32("Axan")) {
                          st->codecpar->codec_id  = AV_CODEC_ID_XAN_DPCM;
                          st->codecpar->codec_tag = 0;
 +                        ast->dshow_block_align = 0;
                      }
                      if (amv_file_format) {
                          st->codecpar->codec_id    = AV_CODEC_ID_ADPCM_IMA_AMV;
                          ast->dshow_block_align = 0;
                      }
 +                    if ((st->codecpar->codec_id == AV_CODEC_ID_AAC  ||
 +                         st->codecpar->codec_id == AV_CODEC_ID_FLAC ||
 +                         st->codecpar->codec_id == AV_CODEC_ID_MP2 ) && ast->dshow_block_align <= 4 && ast->dshow_block_align) {
 +                        av_log(s, AV_LOG_DEBUG, "overriding invalid dshow_block_align of %d\n", ast->dshow_block_align);
 +                        ast->dshow_block_align = 0;
 +                    }
 +                    if (st->codecpar->codec_id == AV_CODEC_ID_AAC && ast->dshow_block_align == 1024 && ast->sample_size == 1024 ||
 +                       st->codecpar->codec_id == AV_CODEC_ID_AAC && ast->dshow_block_align == 4096 && ast->sample_size == 4096 ||
 +                       st->codecpar->codec_id == AV_CODEC_ID_MP3 && ast->dshow_block_align == 1152 && ast->sample_size == 1152) {
 +                        av_log(s, AV_LOG_DEBUG, "overriding sample_size\n");
 +                        ast->sample_size = 0;
 +                    }
                      break;
                  case AVMEDIA_TYPE_SUBTITLE:
                      st->codecpar->codec_type = AVMEDIA_TYPE_SUBTITLE;
 -                    st->codecpar->codec_id   = AV_CODEC_ID_PROBE;
 +                    st->request_probe= 1;
 +                    avio_skip(pb, size);
                      break;
                  default:
                      st->codecpar->codec_type = AVMEDIA_TYPE_DATA;
                  }
              }
              break;
 +        case MKTAG('s', 't', 'r', 'd'):
 +            if (stream_index >= (unsigned)s->nb_streams
 +                || s->streams[stream_index]->codecpar->extradata_size
 +                || s->streams[stream_index]->codecpar->codec_tag == MKTAG('H','2','6','4')) {
 +                avio_skip(pb, size);
 +            } else {
 +                uint64_t cur_pos = avio_tell(pb);
 +                if (cur_pos < list_end)
 +                    size = FFMIN(size, list_end - cur_pos);
 +                st = s->streams[stream_index];
 +
 +                if (size<(1<<30)) {
 +                    if (st->codecpar->extradata) {
 +                        av_log(s, AV_LOG_WARNING, "New extradata in strd chunk, freeing previous one.\n");
 +                        av_freep(&st->codecpar->extradata);
 +                    }
 +                    if (ff_get_extradata(s, st->codecpar, pb, size) < 0)
 +                        return AVERROR(ENOMEM);
 +                }
 +
 +                if (st->codecpar->extradata_size & 1) //FIXME check if the encoder really did this correctly
 +                    avio_r8(pb);
 +
 +                ret = avi_extract_stream_metadata(s, st);
 +                if (ret < 0) {
 +                    av_log(s, AV_LOG_WARNING, "could not decoding EXIF data in stream header.\n");
 +                }
 +            }
 +            break;
          case MKTAG('i', 'n', 'd', 'x'):
              pos = avio_tell(pb);
 -            if ((pb->seekable & AVIO_SEEKABLE_NORMAL) &&
 -                !(s->flags & AVFMT_FLAG_IGNIDX) &&
 -                read_braindead_odml_indx(s, 0) < 0 &&
 +            if ((pb->seekable & AVIO_SEEKABLE_NORMAL) && !(s->flags & AVFMT_FLAG_IGNIDX) &&
 +                avi->use_odml &&
 +                read_odml_index(s, 0) < 0 &&
                  (s->error_recognition & AV_EF_EXPLODE))
                  goto fail;
              avio_seek(pb, pos + size, SEEK_SET);
              }
          default:
              if (size > 1000000) {
 +                char tag_buf[32];
 +                av_get_codec_tag_string(tag_buf, sizeof(tag_buf), tag);
                  av_log(s, AV_LOG_ERROR,
                         "Something went wrong during header parsing, "
 -                       "I will ignore it and try to continue anyway.\n");
 +                       "tag %s has size %u, "
 +                       "I will ignore it and try to continue anyway.\n",
 +                       tag_buf, size);
                  if (s->error_recognition & AV_EF_EXPLODE)
                      goto fail;
                  avi->movi_list = avio_tell(pb) - 4;
                  avi->movi_end  = avi->fsize;
                  goto end_of_header;
              }
 +        /* Do not fail for very large idx1 tags */
 +        case MKTAG('i', 'd', 'x', '1'):
              /* skip tag */
              size += (size & 1);
              avio_skip(pb, size);
@@@ -1024,32 -827,17 +1024,32 @@@ fail
  
      if (!avi->index_loaded && (pb->seekable & AVIO_SEEKABLE_NORMAL))
          avi_load_index(s);
 -    avi->index_loaded     = 1;
 +    calculate_bitrate(s);
 +    avi->index_loaded    |= 1;
  
      if ((ret = guess_ni_flag(s)) < 0)
          return ret;
  
 -    avi->non_interleaved |= ret;
 +    avi->non_interleaved |= ret | (s->flags & AVFMT_FLAG_SORT_DTS);
 +
 +    dict_entry = av_dict_get(s->metadata, "ISFT", NULL, 0);
 +    if (dict_entry && !strcmp(dict_entry->value, "PotEncoder"))
 +        for (i = 0; i < s->nb_streams; i++) {
 +            AVStream *st = s->streams[i];
 +            if (   st->codecpar->codec_id == AV_CODEC_ID_MPEG1VIDEO
 +                || st->codecpar->codec_id == AV_CODEC_ID_MPEG2VIDEO)
 +                st->need_parsing = AVSTREAM_PARSE_FULL;
 +        }
 +
      for (i = 0; i < s->nb_streams; i++) {
          AVStream *st = s->streams[i];
          if (st->nb_index_entries)
              break;
      }
 +    // DV-in-AVI cannot be non-interleaved, if set this must be
 +    // a mis-detection.
 +    if (avi->dv_demux)
 +        avi->non_interleaved = 0;
      if (i == s->nb_streams && avi->non_interleaved) {
          av_log(s, AV_LOG_WARNING,
                 "Non-interleaved AVI without index, switching to interleaved\n");
      return 0;
  }
  
 -static int read_gab2_sub(AVStream *st, AVPacket *pkt)
 +static int read_gab2_sub(AVFormatContext *s, AVStream *st, AVPacket *pkt)
  {
      if (pkt->size >= 7 &&
 +        pkt->size < INT_MAX - AVPROBE_PADDING_SIZE &&
          !strcmp(pkt->data, "GAB2") && AV_RL16(pkt->data + 5) == 2) {
          uint8_t desc[256];
          int score      = AVPROBE_SCORE_EXTENSION, ret;
          AVIStream *ast = st->priv_data;
          AVInputFormat *sub_demuxer;
          AVRational time_base;
 +        int size;
          AVIOContext *pb = avio_alloc_context(pkt->data + 7,
                                               pkt->size - 7,
                                               0, NULL, NULL, NULL, NULL);
          avio_rl16(pb);   /* flags? */
          avio_rl32(pb);   /* data size */
  
 -        pd = (AVProbeData) { .buf      = pb->buf_ptr,
 -                             .buf_size = pb->buf_end - pb->buf_ptr };
 -        if (!(sub_demuxer = av_probe_input_format2(&pd, 1, &score)))
 +        size = pb->buf_end - pb->buf_ptr;
 +        pd = (AVProbeData) { .buf      = av_mallocz(size + AVPROBE_PADDING_SIZE),
 +                             .buf_size = size };
 +        if (!pd.buf)
 +            goto error;
 +        memcpy(pd.buf, pb->buf_ptr, size);
 +        sub_demuxer = av_probe_input_format2(&pd, 1, &score);
 +        av_freep(&pd.buf);
 +        if (!sub_demuxer)
              goto error;
  
          if (!(ast->sub_ctx = avformat_alloc_context()))
              goto error;
  
          ast->sub_ctx->pb = pb;
 +
 +        if (ff_copy_whiteblacklists(ast->sub_ctx, s) < 0)
 +            goto error;
 +
          if (!avformat_open_input(&ast->sub_ctx, "", sub_demuxer, NULL)) {
 +            if (ast->sub_ctx->nb_streams != 1)
 +                goto error;
              ff_read_packet(ast->sub_ctx, &ast->sub_pkt);
              avcodec_parameters_copy(st->codecpar, ast->sub_ctx->streams[0]->codecpar);
              time_base = ast->sub_ctx->streams[0]->time_base;
          return 1;
  
  error:
 +        av_freep(&ast->sub_ctx);
          av_freep(&pb);
      }
      return 0;
@@@ -1167,7 -940,7 +1167,7 @@@ static AVStream *get_subtitle_pkt(AVFor
      return sub_st;
  }
  
 -static int get_stream_idx(int *d)
 +static int get_stream_idx(const unsigned *d)
  {
      if (d[0] >= '0' && d[0] <= '9' &&
          d[1] >= '0' && d[1] <= '9') {
      }
  }
  
 +/**
 + *
 + * @param exit_early set to 1 to just gather packet position without making the changes needed to actually read & return the packet
 + */
  static int avi_sync(AVFormatContext *s, int exit_early)
  {
      AVIContext *avi = s->priv_data;
  
  start_sync:
      memset(d, -1, sizeof(d));
 -    for (i = sync = avio_tell(pb); !pb->eof_reached; i++) {
 +    for (i = sync = avio_tell(pb); !avio_feof(pb); i++) {
          int j;
  
          for (j = 0; j < 7; j++)
          size = d[4] + (d[5] << 8) + (d[6] << 16) + (d[7] << 24);
  
          n = get_stream_idx(d + 2);
 -        av_log(s, AV_LOG_TRACE, "%X %X %X %X %X %X %X %X %"PRId64" %u %d\n",
 +        ff_tlog(s, "%X %X %X %X %X %X %X %X %"PRId64" %u %d\n",
                  d[0], d[1], d[2], d[3], d[4], d[5], d[6], d[7], i, size, n);
 -        if (i + (uint64_t)size > avi->fsize || d[0] > 127)
 +        if (i*(avi->io_fsize>0) + (uint64_t)size > avi->fsize || d[0] > 127)
              continue;
  
          // parse ix##
          if ((d[0] == 'i' && d[1] == 'x' && n < s->nb_streams) ||
              // parse JUNK
              (d[0] == 'J' && d[1] == 'U' && d[2] == 'N' && d[3] == 'K') ||
 -            (d[0] == 'i' && d[1] == 'd' && d[2] == 'x' && d[3] == '1')) {
 +            (d[0] == 'i' && d[1] == 'd' && d[2] == 'x' && d[3] == '1') ||
 +            (d[0] == 'i' && d[1] == 'n' && d[2] == 'd' && d[3] == 'x')) {
              avio_skip(pb, size);
              goto start_sync;
          }
              st  = s->streams[n];
              ast = st->priv_data;
  
 +            if (!ast) {
 +                av_log(s, AV_LOG_WARNING, "Skipping foreign stream %d packet\n", n);
 +                continue;
 +            }
 +
              if (s->nb_streams >= 2) {
                  AVStream *st1   = s->streams[1];
                  AVIStream *ast1 = st1->priv_data;
                  // workaround for broken small-file-bug402.avi
 -                if (d[2] == 'w' && d[3] == 'b' && n == 0 &&
 -                    st->codecpar->codec_type  == AVMEDIA_TYPE_VIDEO &&
 -                    st1->codecpar->codec_type == AVMEDIA_TYPE_AUDIO &&
 -                    ast->prefix == 'd' * 256 + 'c' &&
 -                    (d[2] * 256 + d[3] == ast1->prefix ||
 -                     !ast1->prefix_count)) {
 +                if (   d[2] == 'w' && d[3] == 'b'
 +                   && n == 0
 +                   && st ->codecpar->codec_type == AVMEDIA_TYPE_VIDEO
 +                   && st1->codecpar->codec_type == AVMEDIA_TYPE_AUDIO
 +                   && ast->prefix == 'd'*256+'c'
 +                   && (d[2]*256+d[3] == ast1->prefix || !ast1->prefix_count)
 +                  ) {
                      n   = 1;
                      st  = st1;
                      ast = ast1;
                  || st->discard >= AVDISCARD_ALL)) {
                  if (!exit_early) {
                      ast->frame_offset += get_duration(ast, size);
 +                    avio_skip(pb, size);
 +                    goto start_sync;
                  }
 -                avio_skip(pb, size);
 -                goto start_sync;
              }
  
              if (d[2] == 'p' && d[3] == 'c' && size <= 4 * 256 + 4) {
  
                  // b + (g << 8) + (r << 16);
                  for (; k <= last; k++)
 -                    ast->pal[k] = avio_rb32(pb) >> 8;
 +                    ast->pal[k] = 0xFFU<<24 | avio_rb32(pb)>>8;
  
                  ast->has_pal = 1;
                  goto start_sync;
                  ast->packet_size  = size + 8;
                  ast->remaining    = size;
  
 -                if (size || !ast->sample_size) {
 +                if (size) {
                      uint64_t pos = avio_tell(pb) - 8;
                      if (!st->index_entries || !st->nb_index_entries ||
                          st->index_entries[st->nb_index_entries - 1].pos < pos) {
          }
      }
  
 +    if (pb->error)
 +        return pb->error;
      return AVERROR_EOF;
  }
  
@@@ -1368,7 -1128,10 +1368,7 @@@ static int ni_prepare_read(AVFormatCont
          return AVERROR_EOF;
  
      best_ast = best_st->priv_data;
 -    best_ts  = av_rescale_q(best_ts,
 -                            (AVRational) { FFMAX(1, best_ast->sample_size),
 -                                           AV_TIME_BASE },
 -                            best_st->time_base);
 +    best_ts  = best_ast->frame_offset;
      if (best_ast->remaining) {
          i = av_index_search_timestamp(best_st,
                                        best_ts,
      if (i >= 0) {
          int64_t pos = best_st->index_entries[i].pos;
          pos += best_ast->packet_size - best_ast->remaining;
 -        avio_seek(s->pb, pos + 8, SEEK_SET);
 +        if (avio_seek(s->pb, pos + 8, SEEK_SET) < 0)
 +          return AVERROR_EOF;
  
 -        assert(best_ast->remaining <= best_ast->packet_size);
 +        av_assert0(best_ast->remaining <= best_ast->packet_size);
  
          avi->stream_index = best_stream_index;
          if (!best_ast->remaining)
              best_ast->packet_size =
              best_ast->remaining   = best_st->index_entries[i].size;
      }
 +    else
 +        return AVERROR_EOF;
  
      return 0;
  }
@@@ -1443,9 -1203,8 +1443,9 @@@ resync
          err               = av_get_packet(pb, pkt, size);
          if (err < 0)
              return err;
 +        size = err;
  
 -        if (ast->has_pal && pkt->data && pkt->size < (unsigned)INT_MAX / 2) {
 +        if (ast->has_pal && pkt->size < (unsigned)INT_MAX / 2) {
              uint8_t *pal;
              pal = av_packet_new_side_data(pkt,
                                            AV_PKT_DATA_PALETTE,
          if (CONFIG_DV_DEMUXER && avi->dv_demux) {
              AVBufferRef *avbuf = pkt->buf;
              size = avpriv_dv_produce_packet(avi->dv_demux, pkt,
 -                                            pkt->data, pkt->size);
 +                                            pkt->data, pkt->size, pkt->pos);
              pkt->buf    = avbuf;
              pkt->flags |= AV_PKT_FLAG_KEY;
              if (size < 0)
                  av_packet_unref(pkt);
          } else if (st->codecpar->codec_type == AVMEDIA_TYPE_SUBTITLE &&
 -                   !st->codecpar->codec_tag && read_gab2_sub(st, pkt)) {
 +                   !st->codecpar->codec_tag && read_gab2_sub(s, st, pkt)) {
              ast->frame_offset++;
              avi->stream_index = -1;
              ast->remaining    = 0;
                      size);
              pkt->stream_index = avi->stream_index;
  
 -            if (st->codecpar->codec_type == AVMEDIA_TYPE_VIDEO) {
 +            if (st->codecpar->codec_type == AVMEDIA_TYPE_VIDEO && st->index_entries) {
                  AVIndexEntry *e;
                  int index;
 -                assert(st->index_entries);
  
 -                index = av_index_search_timestamp(st, ast->frame_offset, 0);
 +                index = av_index_search_timestamp(st, ast->frame_offset, AVSEEK_FLAG_ANY);
                  e     = &st->index_entries[index];
  
 -                if (index >= 0 && e->timestamp == ast->frame_offset)
 +                if (index >= 0 && e->timestamp == ast->frame_offset) {
 +                    if (index == st->nb_index_entries-1) {
 +                        int key=1;
 +                        uint32_t state=-1;
 +                        if (st->codecpar->codec_id == AV_CODEC_ID_MPEG4) {
 +                            const uint8_t *ptr = pkt->data, *end = ptr + FFMIN(size, 256);
 +                            while (ptr < end) {
 +                                ptr = avpriv_find_start_code(ptr, end, &state);
 +                                if (state == 0x1B6 && ptr < end) {
 +                                    key = !(*ptr & 0xC0);
 +                                    break;
 +                                }
 +                            }
 +                        }
 +                        if (!key)
 +                            e->flags &= ~AVINDEX_KEYFRAME;
 +                    }
                      if (e->flags & AVINDEX_KEYFRAME)
                          pkt->flags |= AV_PKT_FLAG_KEY;
 +                }
              } else {
                  pkt->flags |= AV_PKT_FLAG_KEY;
              }
              ast->packet_size  = 0;
          }
  
 +        if (!avi->non_interleaved && pkt->pos >= 0 && ast->seek_pos > pkt->pos) {
 +            av_packet_unref(pkt);
 +            goto resync;
 +        }
 +        ast->seek_pos= 0;
 +
 +        if (!avi->non_interleaved && st->nb_index_entries>1 && avi->index_loaded>1) {
 +            int64_t dts= av_rescale_q(pkt->dts, st->time_base, AV_TIME_BASE_Q);
 +
 +            if (avi->dts_max - dts > 2*AV_TIME_BASE) {
 +                avi->non_interleaved= 1;
 +                av_log(s, AV_LOG_INFO, "Switching to NI mode, due to poor interleaving\n");
 +            }else if (avi->dts_max < dts)
 +                avi->dts_max = dts;
 +        }
 +
          return 0;
      }
  
@@@ -1563,12 -1290,9 +1563,12 @@@ static int avi_read_idx1(AVFormatContex
      int nb_index_entries, i;
      AVStream *st;
      AVIStream *ast;
 -    unsigned int index, tag, flags, pos, len, first_packet = 1;
 -    unsigned last_pos = -1;
 +    int64_t pos;
 +    unsigned int index, tag, flags, len, first_packet = 1;
 +    int64_t last_pos = -1;
 +    unsigned last_idx = -1;
      int64_t idx1_pos, first_packet_pos = 0, data_offset = 0;
 +    int anykey = 0;
  
      nb_index_entries = size / 16;
      if (nb_index_entries <= 0)
      avi->stream_index = -1;
      avio_seek(pb, idx1_pos, SEEK_SET);
  
 +    if (s->nb_streams == 1 && s->streams[0]->codecpar->codec_tag == AV_RL32("MMES")) {
 +        first_packet_pos = 0;
 +        data_offset = avi->movi_list;
 +    }
 +
      /* Read the entries and sort them in each stream component. */
      for (i = 0; i < nb_index_entries; i++) {
 +        if (avio_feof(pb))
 +            return -1;
 +
          tag   = avio_rl32(pb);
          flags = avio_rl32(pb);
          pos   = avio_rl32(pb);
          len   = avio_rl32(pb);
 -        av_log(s, AV_LOG_TRACE, "%d: tag=0x%x flags=0x%x pos=0x%x len=%d/",
 +        av_log(s, AV_LOG_TRACE, "%d: tag=0x%x flags=0x%x pos=0x%"PRIx64" len=%d/",
                  i, tag, flags, pos, len);
  
          index  = ((tag      & 0xff) - '0') * 10;
          st  = s->streams[index];
          ast = st->priv_data;
  
 -        if (first_packet && first_packet_pos && len) {
 -            data_offset  = first_packet_pos - pos;
 +        /* Skip 'xxpc' palette change entries in the index until a logic
 +         * to process these is properly implemented. */
 +        if ((tag >> 16 & 0xff) == 'p' && (tag >> 24 & 0xff) == 'c')
 +            continue;
 +
 +        if (first_packet && first_packet_pos) {
 +            if (avi->movi_list + 4 != pos || pos + 500 > first_packet_pos)
 +                data_offset  = first_packet_pos - pos;
              first_packet = 0;
          }
          pos += data_offset;
  
          av_log(s, AV_LOG_TRACE, "%d cum_len=%"PRId64"\n", len, ast->cum_len);
  
 -        if (pb->eof_reached)
 -            return AVERROR_INVALIDDATA;
 -
 +        // even if we have only a single stream, we should
 +        // switch to non-interleaved to get correct timestamps
          if (last_pos == pos)
              avi->non_interleaved = 1;
 -        else if (len || !ast->sample_size)
 +        if (last_idx != pos && len) {
              av_add_index_entry(st, pos, ast->cum_len, len, 0,
                                 (flags & AVIIF_INDEX) ? AVINDEX_KEYFRAME : 0);
 +            last_idx= pos;
 +        }
          ast->cum_len += get_duration(ast, len);
          last_pos      = pos;
 +        anykey       |= flags&AVIIF_INDEX;
 +    }
 +    if (!anykey) {
 +        for (index = 0; index < s->nb_streams; index++) {
 +            st = s->streams[index];
 +            if (st->nb_index_entries)
 +                st->index_entries[0].flags |= AVINDEX_KEYFRAME;
 +        }
      }
      return 0;
  }
@@@ -1651,6 -1352,7 +1651,6 @@@ static int check_stream_max_drift(AVFor
      int *idx = av_mallocz_array(s->nb_streams, sizeof(*idx));
      if (!idx)
          return AVERROR(ENOMEM);
 -
      for (min_pos = pos = 0; min_pos != INT64_MAX; pos = min_pos + 1LU) {
          int64_t max_dts = INT64_MIN / 2;
          int64_t min_dts = INT64_MAX / 2;
@@@ -1716,15 -1418,9 +1716,15 @@@ static int guess_ni_flag(AVFormatContex
  
          if (n >= 2) {
              int64_t pos = st->index_entries[0].pos;
 -            avio_seek(s->pb, pos + 4, SEEK_SET);
 +            unsigned tag[2];
 +            avio_seek(s->pb, pos, SEEK_SET);
 +            tag[0] = avio_r8(s->pb);
 +            tag[1] = avio_r8(s->pb);
 +            avio_rl16(s->pb);
              size = avio_rl32(s->pb);
 -            if (pos + size > st->index_entries[1].pos)
 +            if (get_stream_idx(tag) == i && pos + size > st->index_entries[1].pos)
 +                last_start = INT64_MAX;
 +            if (get_stream_idx(tag) == i && size == st->index_entries[0].size + 8)
                  last_start = INT64_MAX;
          }
  
@@@ -1747,19 -1443,16 +1747,19 @@@ static int avi_load_index(AVFormatConte
      AVIOContext *pb = s->pb;
      uint32_t tag, size;
      int64_t pos = avio_tell(pb);
 +    int64_t next;
      int ret     = -1;
  
      if (avio_seek(pb, avi->movi_end, SEEK_SET) < 0)
          goto the_end; // maybe truncated file
      av_log(s, AV_LOG_TRACE, "movi_end=0x%"PRIx64"\n", avi->movi_end);
      for (;;) {
 -        if (pb->eof_reached)
 -            break;
          tag  = avio_rl32(pb);
          size = avio_rl32(pb);
 +        if (avio_feof(pb))
 +            break;
 +        next = avio_tell(pb) + size + (size & 1);
 +
          av_log(s, AV_LOG_TRACE, "tag=%c%c%c%c size=0x%x\n",
                   tag        & 0xff,
                  (tag >>  8) & 0xff,
  
          if (tag == MKTAG('i', 'd', 'x', '1') &&
              avi_read_idx1(s, size) >= 0) {
 +            avi->index_loaded=2;
              ret = 0;
 +        }else if (tag == MKTAG('L', 'I', 'S', 'T')) {
 +            uint32_t tag1 = avio_rl32(pb);
 +
 +            if (tag1 == MKTAG('I', 'N', 'F', 'O'))
 +                ff_read_riff_info(s, size - 4);
 +        }else if (!ret)
              break;
 -        }
  
 -        size += (size & 1);
 -        if (avio_skip(pb, size) < 0)
 +        if (avio_seek(pb, next, SEEK_SET) < 0)
              break; // something is wrong here
      }
  
@@@ -1804,7 -1492,7 +1804,7 @@@ static int avi_read_seek(AVFormatContex
      AVIContext *avi = s->priv_data;
      AVStream *st;
      int i, index;
 -    int64_t pos;
 +    int64_t pos, pos_min;
      AVIStream *ast;
  
      /* Does not matter which stream is requested dv in avi has the
      if (!avi->index_loaded) {
          /* we only load the index on demand */
          avi_load_index(s);
 -        avi->index_loaded = 1;
 +        avi->index_loaded |= 1;
      }
 +    av_assert0(stream_index >= 0);
  
      st    = s->streams[stream_index];
      ast   = st->priv_data;
      index = av_index_search_timestamp(st,
                                        timestamp * FFMAX(ast->sample_size, 1),
                                        flags);
 -    if (index < 0)
 +    if (index < 0) {
 +        if (st->nb_index_entries > 0)
 +            av_log(s, AV_LOG_DEBUG, "Failed to find timestamp %"PRId64 " in index %"PRId64 " .. %"PRId64 "\n",
 +                   timestamp * FFMAX(ast->sample_size, 1),
 +                   st->index_entries[0].timestamp,
 +                   st->index_entries[st->nb_index_entries - 1].timestamp);
          return AVERROR_INVALIDDATA;
 +    }
  
      /* find the position */
      pos       = st->index_entries[index].pos;
          /* offsets. Calling with other stream indexes should have failed */
          /* the av_index_search_timestamp call above.                     */
  
 +        if (avio_seek(s->pb, pos, SEEK_SET) < 0)
 +            return -1;
 +
          /* Feed the DV video stream version of the timestamp to the */
          /* DV demux so it can synthesize correct timestamps.        */
          ff_dv_offset_reset(avi->dv_demux, timestamp);
  
 -        avio_seek(s->pb, pos, SEEK_SET);
          avi->stream_index = -1;
          return 0;
      }
  
 +    pos_min = pos;
      for (i = 0; i < s->nb_streams; i++) {
          AVStream *st2   = s->streams[i];
          AVIStream *ast2 = st2->priv_data;
          if (st2->nb_index_entries <= 0)
              continue;
  
 -//        assert(st2->codecpar->block_align);
 -        assert((int64_t)st2->time_base.num * ast2->rate ==
 -               (int64_t)st2->time_base.den * ast2->scale);
 +//        av_assert1(st2->codecpar->block_align);
          index = av_index_search_timestamp(st2,
                                            av_rescale_q(timestamp,
                                                         st->time_base,
                                                         st2->time_base) *
                                            FFMAX(ast2->sample_size, 1),
 -                                          flags | AVSEEK_FLAG_BACKWARD);
 +                                          flags |
 +                                          AVSEEK_FLAG_BACKWARD |
 +                                          (st2->codecpar->codec_type != AVMEDIA_TYPE_VIDEO ? AVSEEK_FLAG_ANY : 0));
          if (index < 0)
              index = 0;
 +        ast2->seek_pos = st2->index_entries[index].pos;
 +        pos_min = FFMIN(pos_min,ast2->seek_pos);
 +    }
 +    for (i = 0; i < s->nb_streams; i++) {
 +        AVStream *st2 = s->streams[i];
 +        AVIStream *ast2 = st2->priv_data;
  
 -        if (!avi->non_interleaved) {
 -            while (index > 0 && st2->index_entries[index].pos > pos)
 -                index--;
 -            while (index + 1 < st2->nb_index_entries &&
 -                   st2->index_entries[index].pos < pos)
 -                index++;
 -        }
 +        if (ast2->sub_ctx || st2->nb_index_entries <= 0)
 +            continue;
  
 -        av_log(s, AV_LOG_TRACE, "%"PRId64" %d %"PRId64"\n",
 -                timestamp, index, st2->index_entries[index].timestamp);
 -        /* extract the current frame number */
 +        index = av_index_search_timestamp(
 +                st2,
 +                av_rescale_q(timestamp, st->time_base, st2->time_base) * FFMAX(ast2->sample_size, 1),
 +                flags | AVSEEK_FLAG_BACKWARD | (st2->codecpar->codec_type != AVMEDIA_TYPE_VIDEO ? AVSEEK_FLAG_ANY : 0));
 +        if (index < 0)
 +            index = 0;
 +        while (!avi->non_interleaved && index>0 && st2->index_entries[index-1].pos >= pos_min)
 +            index--;
          ast2->frame_offset = st2->index_entries[index].timestamp;
      }
  
      /* do the seek */
 -    avio_seek(s->pb, pos, SEEK_SET);
 +    if (avio_seek(s->pb, pos_min, SEEK_SET) < 0) {
 +        av_log(s, AV_LOG_ERROR, "Seek failed\n");
 +        return -1;
 +    }
      avi->stream_index = -1;
 +    avi->dts_max      = INT_MIN;
      return 0;
  }
  
@@@ -1928,12 -1596,12 +1928,12 @@@ static int avi_read_close(AVFormatConte
                  av_freep(&ast->sub_ctx->pb);
                  avformat_close_input(&ast->sub_ctx);
              }
 -            av_free(ast->sub_buffer);
 +            av_freep(&ast->sub_buffer);
              av_packet_unref(&ast->sub_pkt);
          }
      }
  
 -    av_free(avi->dv_demux);
 +    av_freep(&avi->dv_demux);
  
      return 0;
  }
@@@ -1944,8 -1612,8 +1944,8 @@@ static int avi_probe(AVProbeData *p
  
      /* check file header */
      for (i = 0; avi_headers[i][0]; i++)
 -        if (!memcmp(p->buf,     avi_headers[i],     4) &&
 -            !memcmp(p->buf + 8, avi_headers[i] + 4, 4))
 +        if (AV_RL32(p->buf    ) == AV_RL32(avi_headers[i]    ) &&
 +            AV_RL32(p->buf + 8) == AV_RL32(avi_headers[i] + 4))
              return AVPROBE_SCORE_MAX;
  
      return 0;
@@@ -1961,5 -1629,4 +1961,5 @@@ AVInputFormat ff_avi_demuxer = 
      .read_packet    = avi_read_packet,
      .read_close     = avi_read_close,
      .read_seek      = avi_read_seek,
 +    .priv_class = &demuxer_class,
  };