[FFmpeg-devel] [PATCH] mov: Set negative Sample_duration in STTS to 1

[Baptiste Coudurier]

On 08/12/2011 04:27 PM, Thierry Foucu wrote:
> ---
>  libavformat/mov.c |    5 +++++
>  1 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/libavformat/mov.c b/libavformat/mov.c
> index fdb29a0..96a9928 100644
> --- a/libavformat/mov.c
> +++ b/libavformat/mov.c
> @@ -1471,6 +1471,11 @@ static int mov_read_stts(MOVContext *c, AVIOContext *pb, MOVAtom atom)
>  
>          sample_count=avio_rb32(pb);
>          sample_duration = avio_rb32(pb);
> +        /* sample_duration < 0 is invalid based on the spec */
> +        if (sample_duration < 0) {
> +            av_log(c->fc, AV_LOG_ERROR, "Invalid SampleDelta in STTS %d", sample_duration);
> +            sample_duration = 1;
> +        }
>          sc->stts_data[i].count= sample_count;
>          sc->stts_data[i].duration= sample_duration;
>  

Looks good.

-- 
Baptiste COUDURIER
Key fingerprint          8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                           http://www.ffmpeg.org