Re: [FFmpeg-devel] [PATCH]lavf/mov: Do not overread iTunes metadata

2015-04-04 Thread Michael Niedermayer
On Sat, Apr 04, 2015 at 11:16:20AM +0200, Carl Eugen Hoyos wrote: > On Saturday 04 April 2015 06:25:36 am Carl Eugen Hoyos wrote: > > Hi! > > > > Attached patch fixes ticket #4425 for me. > > Third try attached after getting some more sleep. > > Carl Eugen probably ok unless the parsing code goe

Re: [FFmpeg-devel] [PATCH]lavf/mov: Do not overread iTunes metadata

2015-04-04 Thread Carl Eugen Hoyos
On Saturday 04 April 2015 06:25:36 am Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #4425 for me. Third try attached after getting some more sleep. Carl Eugen diff --git a/libavformat/mov.c b/libavformat/mov.c index dc31e5a..c63e817 100644 --- a/libavformat/mov.c +++ b/libavformat

Re: [FFmpeg-devel] [PATCH]lavf/mov: Do not overread iTunes metadata

2015-04-03 Thread Carl Eugen Hoyos
On Saturday 04 April 2015 06:25:36 am Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #4425 for me. New patch attached that does not brake fate. Carl Eugen diff --git a/libavformat/mov.c b/libavformat/mov.c index dc31e5a..eda9d42 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.

[FFmpeg-devel] [PATCH]lavf/mov: Do not overread iTunes metadata

2015-04-03 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #4425 for me. Please comment, Carl Eugen diff --git a/libavformat/mov.c b/libavformat/mov.c index dc31e5a..8a4b739 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -342,7 +342,7 @@ retry: if (c->itunes_metadata && atom.size > 8) { int data_si