Luca Barbato <[email protected]> writes:

> On 5/16/11 4:58 PM, Mans Rullgard wrote:
>> @@ -180,9 +180,14 @@ av_cold int ff_dct_init(DCTContext *s, int nbits, enum 
>> DCTTransformType inverse)
>>       int n = 1<<  nbits;
>>       int i;
>>
>> +    memset(s, 0, sizeof(*s));
>> +
>
> Why that?

To avoid changing ff_dct_end().  Without zeroing it first, there will be
random pointers in there which are then passed to free().

-- 
Måns Rullgård
[email protected]
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to