[FFmpeg-devel] [PATCH] avformat/mxfdec: drop invalid index table segments when sorting them

[Tomas Härdin]

fre 2018-07-13 klockan 23:43 +0200 skrev Marton Balint:
> This way we can guess a proper index instead of using an invalid one.
> 
> Fixes seeking in the sample of ticket #5671.
> 
> > Signed-off-by: Marton Balint <cus at passwd.hu>
> ---
>  libavformat/mxfdec.c | 25 +++++++++++++++++--------
>  1 file changed, 17 insertions(+), 8 deletions(-)
> 
> diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c
> index 996969d1ff..b032914388 100644
> --- a/libavformat/mxfdec.c
> +++ b/libavformat/mxfdec.c
> @@ -1348,9 +1348,22 @@ static int mxf_get_sorted_table_segments(MXFContext *mxf, int *nb_sorted_segment
>          return AVERROR(ENOMEM);
>      }
>  
> -    for (i = j = 0; i < mxf->metadata_sets_count; i++)
> -        if (mxf->metadata_sets[i]->type == IndexTableSegment)
> -            unsorted_segments[j++] = (MXFIndexTableSegment*)mxf->metadata_sets[i];
> +    for (i = nb_segments = 0; i < mxf->metadata_sets_count; i++) {
> +        if (mxf->metadata_sets[i]->type == IndexTableSegment) {
> +            MXFIndexTableSegment *s = (MXFIndexTableSegment*)mxf->metadata_sets[i];
> +            if (s->edit_unit_byte_count || s->nb_index_entries)
> +                unsorted_segments[nb_segments++] = s;
> +            else
> +                av_log(mxf->fc, AV_LOG_WARNING, "IndexSID %i segment at %"PRId64" missing EditUnitByteCount and IndexEntryArray\n",
> +                       s->index_sid, s->index_start_position);
> +        }
> +    }
> +
> +    if (!nb_segments) {
> +        av_freep(sorted_segments);
> +        av_free(unsorted_segments);
> +        return AVERROR_INVALIDDATA;
> +    }
>  
>      *nb_sorted_segments = 0;
>  
> @@ -1482,7 +1495,7 @@ static int mxf_edit_unit_absolute_offset(MXFContext *mxf, MXFIndexTable *index_t
>  
>              if (s->edit_unit_byte_count)
>                  offset_temp += s->edit_unit_byte_count * index;
> -            else if (s->nb_index_entries) {
> +            else {
>                  if (s->nb_index_entries == 2 * s->index_duration + 1)
>                      index *= 2;     /* Avid index */
>  
> @@ -1493,10 +1506,6 @@ static int mxf_edit_unit_absolute_offset(MXFContext *mxf, MXFIndexTable *index_t
>                  }
>  
>                  offset_temp = s->stream_offset_entries[index];
> -            } else {
> -                av_log(mxf->fc, AV_LOG_ERROR, "IndexSID %i segment at %"PRId64" missing EditUnitByteCount and IndexEntryArray\n",
> -                       index_table->index_sid, s->index_start_position);
> -                return AVERROR_INVALIDDATA;
>              }
>  
>              if (edit_unit_out)

Looks OK to me

/Tomas