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

Thilo Borgmann thilo.borgmann at mail.de
Sun Apr 19 20:01:28 CEST 2015


Am 19.04.15 um 00:49 schrieb Michael Niedermayer:
> 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

Thanks!

-Thilo



More information about the ffmpeg-devel mailing list