[FFmpeg-devel] [libav-devel] [PATCH] alsdec: validate time diff index

[Michael Niedermayer]

On Wed, Apr 22, 2015 at 10:59:29AM +0200, Thilo Borgmann wrote:
> Am 21.04.15 um 19:35 schrieb Andreas Cadhalpun:
> > On 21.04.2015 08:14, Thilo Borgmann wrote:
> >> Am 20.04.15 um 23:20 schrieb Andreas Cadhalpun:
> >>> On 19.04.2015 22:20, Luca Barbato wrote:
> >>>> I'd check that `master` is always between `raw_buffer` and the end of it.
> >>>
> >>> You mean something like the attached patch?
> >>>
> >>>> (I'm not sure if `div_blocks` is validated before, same for `offset`)
> >>>
> >>> That should catch problems in those as well.
> >>
> >> Have you tested with fate after applying this patch locally?
> > 
> > Yes, but only on amd64, where fate passes.
> > 
> > Attached is a new patch, which should work everywhere.
> 
> LGTM

applied

thanks

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

Frequently ignored answer#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- 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/20150422/67a9c832/attachment.asc>