[FFmpeg-devel] [PATCH] libavformat/mov: Elimitate double reading of COVR metadata if MOV_EXPORT_ALL_METADATA is enabled

[Michael Niedermayer]

On Wed, May 14, 2014 at 04:52:46PM -0700, Alex Sukhanov wrote:
> On Wed, May 14, 2014 at 2:39 PM, Michael Niedermayer <michaelni at gmx.at>wrote:
> 
> > On Wed, May 14, 2014 at 12:10:16PM -0700, Alex Sukhanov wrote:
> > > Problem: ffmpeg tries to read COVR atom data twice if
> > MOV_EXPORT_ALL_METADATA is enabled.
> > > If COVR atom is the last in the stream, a parsing of such file fails.
> > >
> > > Solution: just return immediatelly after mov_read_covr
> >
> > how can the issue be reproduced ?
> >
> > [...]
> > --
> > Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> >
> > If you think the mosad wants you dead since a long time then you are either
> > wrong or dead since a long time.
> >
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel at ffmpeg.org
> > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> >
> Hi Michael,
> 
> I shared this file privately with you.

thanks, bugfix confirmed

patch applied

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

I know you won't believe me, but the highest form of Human Excellence is
to question oneself and others. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140515/ede6f9e8/attachment.asc>