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

Andreas Cadhalpun andreas.cadhalpun at googlemail.com
Sun Apr 19 00:20:01 CEST 2015


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.

Best regards,
Andreas



More information about the ffmpeg-devel mailing list