Hi,

On Tue, Jul 12, 2011 at 12:41 PM, Anton Khirnov <an...@khirnov.net> wrote:
> On Tue, 12 Jul 2011 12:18:37 -0700, "Ronald S. Bultje" <rsbul...@gmail.com> 
> wrote:
>> On Tue, Jul 12, 2011 at 9:32 AM, Anton Khirnov <an...@khirnov.net> wrote:
>> >     ts_scale = grow_array(ts_scale, sizeof(*ts_scale), &nb_ts_scale, 
>> > stream + 1);
>> >     ts_scale[stream] = scale;
>>
>> This is a poor man's attempt to prevent us from failing because the
>> array grows too big. An alloc check would be nice.
>
> grow_array() already does that.

Then we fail to check the return value... If grow_array fails, the
above crashes.

Ronald
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to