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

Michael Niedermayer michaelni at gmx.at
Sun Apr 19 00:49:35 CEST 2015


On Sun, Apr 19, 2015 at 12:20:01AM +0200, Andreas Cadhalpun wrote:
> On 18.04.2015 23:34, Michael Niedermayer wrote:
> > On Sat, Apr 18, 2015 at 10:28:53PM +0200, Andreas Cadhalpun wrote:
> >> I don't think an assert would be good here. If you want to protect against
> >> future code changes, I would rather leave the error return.
> >> But I'd also be fine with dropping this check altogether.
> >> What do you prefer?
> > 
> > whatever you prefer, the important part is to add checks where they
> > are needed
> 
> OK, then lets keep both checks as in the second patch I sent.

applied

thanks

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

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- 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/20150419/b68c5c22/attachment.asc>


More information about the ffmpeg-devel mailing list