[FFmpeg-devel] [PATCH]lavf/mov: Do not fail hard for more mov atoms

Michael Niedermayer michael at niedermayer.cc
Mon Jan 14 11:05:48 EET 2019


On Sun, Jan 13, 2019 at 11:14:52PM +0100, Carl Eugen Hoyos wrote:
> Hi!
> 
> Attached patch fixes ticket #7679.
> 
> Please comment, Carl Eugen

>  mov.c |   12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> a2759589f6ab06b09057969490ffc13291965144  0001-lavf-mov-Do-not-fail-hard-for-more-invalid-atoms.patch
> From ea6afa36d5ceb6e027176f051e7886f0648e3ac2 Mon Sep 17 00:00:00 2001
> From: Carl Eugen Hoyos <ceffmpeg at gmail.com>
> Date: Sun, 13 Jan 2019 23:07:06 +0100
> Subject: [PATCH] lavf/mov: Do not fail hard for more invalid atoms.
> 
> This is what several other players do and what FFmpeg already does for the sidx atom.
> 
> Fixes ticket #7679.
> ---
>  libavformat/mov.c |   12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)

If a atom is ignored/discarded it should not have changed state of other
things. This patch would break out after some fields are updated

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship: All citizens are under surveillance, all their steps and
actions recorded, for the politicians to enforce control.
Democracy: All politicians are under surveillance, all their steps and
actions recorded, for the citizens to enforce control.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20190114/10831f49/attachment.sig>


More information about the ffmpeg-devel mailing list