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

[Michael Niedermayer]

On Fri, Aug 12, 2011 at 06:14:42PM -0700, Baptiste Coudurier wrote:
> 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.

applied, will push when tests pass

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 3
"Rare item" - "Common item with rare defect or maybe just a lie"
"Professional" - "'Toy' made in china, not functional except as doorstop"
"Experts will know" - "The seller hopes you are not an expert"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110813/93ca581b/attachment.asc>