[FFmpeg-devel] [PATCH] mov.c: read fragment start dts from fragmented mp4

[Michael Niedermayer]

On Wed, Oct 08, 2014 at 03:03:50PM +0200, Michael Niedermayer wrote:
[...]
> > @@ -3565,6 +3678,9 @@ static int mov_read_header(AVFormatContext *s)
> >      else
> >          atom.size = INT64_MAX;
> >  
> > +    if (pb->seekable) {
> > +        mov_read_mfra(s);
> > +    }
> 
> mov_read_mfra() requires multiple seeks, these are quite expensive if
> the file is not local but accessed over the network.
> Is it possible to detect (most) files which do not have mfra atoms
> reliably without having to try to read the mfra atom so as to avoid
> doing these potentially expensive seeks ?

also mov_read_mfra() is missing a seek back to 0 / the previous
position

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 2
"100% positive feedback" - "All either got their money back or didnt complain"
"Best seller ever, very honest" - "Seller refunded buyer after failed scam"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20141008/1225e225/attachment.asc>