Re: [FFmpeg-devel] [PATCH] lavf/mov: don't read outside frag_index bounds

2017-11-17 Thread Michael Niedermayer
On Fri, Nov 17, 2017 at 12:23:11PM -0800, Dale Curtis wrote: > lgtm, fixes the crash and doesn't regress any of our tests. will apply unless it breaks something in testing thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Opposition brings concord. Out of

Re: [FFmpeg-devel] [PATCH] lavf/mov: don't read outside frag_index bounds

2017-11-17 Thread Dale Curtis
lgtm, fixes the crash and doesn't regress any of our tests. - dale On Fri, Nov 17, 2017 at 8:21 AM, John Stebbins wrote: > Potentially fixes: > https://bugs.chromium.org/p/chromium/issues/detail?id=786269#c1 > > In theory, the crash can be triggered by an invalid stream that has > either tfdt o

[FFmpeg-devel] [PATCH] lavf/mov: don't read outside frag_index bounds

2017-11-17 Thread John Stebbins
Potentially fixes: https://bugs.chromium.org/p/chromium/issues/detail?id=786269#c1 In theory, the crash can be triggered by an invalid stream that has either tfdt or trun outside of the moof --- libavformat/mov.c | 4 1 file changed, 4 insertions(+) diff --git a/libavformat/mov.c b/libavfor